The i965 driver fails the following dEQP test on all platforms:
This is because it respects GL_SKIP_DECODE_EXT for all texturing functions, when it's required to ignore it for texelFetch*(). From the EXT_texture_sRGB_decode spec:
"The conversion of sRGB color space components to linear color space is
always performed if the texel lookup function is one of the texelFetch
Otherwise, if the texel lookup function is one of the texture builtin
functions or one of the texture gather functions, the conversion of sRGB
color space components to linear color space is controlled by the
If the TEXTURE_SRGB_DECODE_EXT parameter is DECODE_EXT, the conversion
of sRGB color space components to linear color space is performed.
If the TEXTURE_SRGB_DECODE_EXT parameter is SKIP_DECODE_EXT, the value
is returned without decoding. However, if the texture is also accessed
with a texelFetch function, then the result of texture builtin functions
and/or texture gather functions may be returned with decoding or without
Hi Ken-Thanks for filing. So is this (for sure) a mesa issue? If so, can you help me determine a priority? Thank you.
@Ken-Is this something we should review again as we get closer to Android O?
Discussed the spec at Khronos. Still need to fix this.
The srgb-txf branch of my tree has patches to fix this, I ought to drop the BRW_META_IN_PROGRESS rubbish (which causes more CPU overhead) in favor of whacking the uses_txf bitfield somehow. I'll want to do that before sending them for review.
Nicolai just fixed this on radeonsi today, so I may have to rebase.
The same test is failing in the Android-IA on O.
It would be great if we get the fix quickly as possible.
Kindly let me know the status of the fix.
There are patches to fix the bug here,
but I haven't mailed them out for review yet because Nicolai is imminently pushing patches that conflict with them. I should be able to sort it out tomorrow.
Thank you will try with the patches(In reply to Kenneth Graunke from comment #7)
> There are patches to fix the bug here,
> but I haven't mailed them out for review yet because Nicolai is imminently
> pushing patches that conflict with them. I should be able to sort it out
Author: Kenneth Graunke <email@example.com>
Date: Sat Sep 9 00:20:26 2017 -0700
i965: Ignore GL_SKIP_DECODE_EXT for textures accessed via texelFetch().
Thanks to Nicolai and Jason for reviewing.