intel: Skip texsubimage fastpath for more pixel unpack state (v2)
authorChad Versace <chad.versace@linux.intel.com>
Fri, 12 Oct 2012 18:11:57 +0000 (11:11 -0700)
committerChad Versace <chad.versace@linux.intel.com>
Thu, 18 Oct 2012 15:53:59 +0000 (08:53 -0700)
commit0da72d3502ecc48e9189ebc4cbea446f79ccb61d
tree5fa49e2551e1d1f9bb1b7f44a1ae8edb8cbbc5a7
parent6c281749696da417a88c3d1ee460f642f92a4cee
intel: Skip texsubimage fastpath for more pixel unpack state (v2)

Fixes piglit tests "unpack-teximage2d --pbo=* --format=GL_BGRA" on
Sandybridge+.

The fastpath was checking an incomplete set of pixel unpack state. This
patch adds checks for all the fields of gl_pixelstore_attrib that affect
2D texture uploads.  Also, it begins permitting the case where
GL_UNPACK_ROW_LENGTH is 0.

Ideally, we would just ask a unicorn to JIT this fastpath for us in
a way that safely handles the unpacking state. Until then, it's safer if
only a small set of situations activate the fastpath.

v2: Use _mesa_is_bufferobj(), per Anholt.

Reviewed-by: Eric Anholt <eric@anholt.net>
Signed-off-by: Chad Versace <chad.versace@linux.intel.com>
src/mesa/drivers/dri/intel/intel_tex_subimage.c