i965: Check base format to determine whether to use tiled memcpy
authorNeil Roberts <neil@linux.intel.com>
Thu, 19 Nov 2015 14:06:08 +0000 (15:06 +0100)
committerNeil Roberts <neil@linux.intel.com>
Thu, 10 Dec 2015 11:03:49 +0000 (11:03 +0000)
commit3f10774cbab1e803f8aa3d6d24f8f6f98b8128c3
tree4090b653bb3ae48638f0edad44fadf7025246583
parent9a31d9870b0c2113eefd1c2baf347ab8b288f6a6
i965: Check base format to determine whether to use tiled memcpy

The tiled memcpy doesn't work for copying from RGBX to RGBA because it
doesn't override the alpha component to 1.0. Commit 2cebaac479d4 added
a check to disable it for RGBX formats by looking at the TexFormat.
However a lot of the rest of the code base is written with the
assumption that an RGBA texture can be used internally to implement a
GL_RGB texture. If that is done then this check breaks. This patch
makes it instead check the base format of the texture which I think
more directly matches the intention.

Reviewed-by: Jason Ekstrand <jason.ekstrand@intel.com>
src/mesa/drivers/dri/i965/intel_pixel_read.c
src/mesa/drivers/dri/i965/intel_tex_image.c