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)
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

index ba6267a32de2c158500feca23756046f89ce0e1a..76ea0de01df7bff6de11d86f934c7d0764d03a3c 100644 (file)
@@ -26,6 +26,7 @@
  * 
  **************************************************************************/
 
+#include "main/bufferobj.h"
 #include "main/macros.h"
 #include "main/mtypes.h"
 #include "main/pbo.h"
@@ -205,10 +206,14 @@ intel_texsubimage_tiled_memcpy(struct gl_context * ctx,
        texImage->TexObject->Target != GL_TEXTURE_2D ||
        texImage->Level != 0 ||
        pixels == NULL ||
+       _mesa_is_bufferobj(packing->BufferObj) ||
        packing->Alignment > 4 ||
        packing->SkipPixels > 0 ||
        packing->SkipRows > 0 ||
-       packing->RowLength != width)
+       (packing->RowLength != 0 && packing->RowLength != width) ||
+       packing->SwapBytes ||
+       packing->LsbFirst ||
+       packing->Invert)
       return false;
 
    if (for_glTexImage)