st/mesa: check for incomplete texture in st_finalize_texture()
authorBrian Paul <brianp@vmware.com>
Tue, 27 Jun 2017 15:29:18 +0000 (09:29 -0600)
committerBrian Paul <brianp@vmware.com>
Fri, 30 Jun 2017 19:37:10 +0000 (13:37 -0600)
commitf4d5e55dd1cb4617ee18387d2aff337676b420c4
tree336fcd29b993af68865c0c8ac38b447a05b7b87a
parente54fe78e0e131635a62bcbf49b923a118ca1926f
st/mesa: check for incomplete texture in st_finalize_texture()

Return early from st_finalize_texture() if we have an incomplete
texture.  This avoids trying to create a texture resource with invalid
parameters (too many mipmap levels given the base dimension).

Specifically, the Piglit fbo-incomplete-texture-03 test winds up
calling pipe_screen::resource_create() with width0=32, height0=32 and
last_level=6 because the first five cube faces are 32x32 but the sixth
face is 64x64.  Some drivers handle this, but others (like VMware svga)
do not (generates device errors).

Note that this code is on the path that's usually not taken (we normally
build consistent textures).

No Piglit regressions.

v2: only need to check for base-level completeness since that's what has to
be consistent in order to specify the dimensions for a new gallium texture.
Per Roland.

Reviewed-by: Roland Scheidegger <sroland@vmware.com>
src/mesa/state_tracker/st_cb_texture.c