Revert "i965: Use MESA_FORMAT_B8G8R8X8_SRGB for RGB visuals"
authorNeil Roberts <neil@linux.intel.com>
Wed, 16 Dec 2015 14:53:13 +0000 (14:53 +0000)
committerNeil Roberts <neil@linux.intel.com>
Wed, 16 Dec 2015 17:35:33 +0000 (17:35 +0000)
commit61cdb7665f7bd147533cdc5977750d990c2eafd5
treefda5b574e34657390a5f455cec674ce57d910f54
parent8c5310da9d1cbf2272f72d3ed4264544456a4683
Revert "i965: Use MESA_FORMAT_B8G8R8X8_SRGB for RGB visuals"

This reverts commit 839793680f99b8387bee9489733d5071c10f3ace.

The patch was breaking DRI3 because driGLFormatToImageFormat does not
handle MESA_FORMAT_B8G8R8X8_SRGB which ended up making it fail to
create the renderbuffer and it would later crash. It's not trivial to
add this format because there is no __DRI_IMAGE_FORMAT nor
__DRI_IMAGE_FOURCC define for the format either. I'm not sure how
difficult adding this would be and whether adding a new format would
require some sort of new version for DRI. Seeing as this might take a
while to fix I think it makes sense to just revert the patch in the
meantime in order to avoid regressing master.

It is also not handled in intel_gles3_srgb_workaround and there may be
other cases where it breaks.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93388
Acked-by: Jason Ekstrand <jason.ekstrand@intel.com>
src/mesa/drivers/dri/i965/intel_screen.c