mesa: call ctx->Driver.ChooseTextureFormat() only when necessary.
authorBrian Paul <brianp@vmware.com>
Tue, 20 Jul 2010 14:40:19 +0000 (08:40 -0600)
committerBrian Paul <brianp@vmware.com>
Tue, 20 Jul 2010 14:50:32 +0000 (08:50 -0600)
commitbab484a59b21fff84579a492d079d46e27d486dd
tree537f55a743f40c82fa3c2392965ca2009f347023
parent895086467e03b15d376dcdff0d772408dda03f88
mesa: call ctx->Driver.ChooseTextureFormat() only when necessary.

When defining mipmap level 'L' and level L-1 exists and the new level's
internalFormat matches level L-1's internalFormat, then use the same hw
format.  Otherwise, do the regular ctx->Driver.ChooseTextureFormat() call.

This avoids a problem where we end up choosing different hw formats for
different mipmap levels depending on how the levels are defined (glTexImage
vs. glCopyTexImage vs. glGenerateMipmap, etc).

The root problem is the ChooseTextureFormat() implementation in some
drivers uses the user's glTexImage format/type parameters in the choosing
heuristic.  Later mipmap levels might be generated with different calls
(ex: glCopyTexImage()) so we don't always have format/type info and the
driver may choose a different format.

For more background info see the July 2010 mesa-dev thread "Bug in
_mesa_meta_GenerateMipmap"
src/mesa/drivers/common/meta.c
src/mesa/main/teximage.c
src/mesa/main/teximage.h