[Mesa-dev] [PATCH] i965: Drop support for the GL_MESA_ycbcr_texture extension.

Eric Anholt eric at anholt.net
Tue Dec 6 16:22:49 PST 2011


On Tue,  6 Dec 2011 13:13:59 -0800, Kenneth Graunke <kenneth at whitecape.org> wrote:
> We forgot to implement the color conversions in the new fragment shader
> backend, so YCBCR + GLSL has been broken since Mesa 7.10.  Additionally,
> with the fixed function fragment shader rework, YCBCR + FF is broken in
> master.  The only combination that should still work is YCBCR + ARB_fp.
> 
> Presumably all consumers of this extension already have some kind of
> fallback, as the AMD and nVidia binary drivers likely don't support this
> extension.
> 
> At some point, we may wish to implement the GL_APPLE_rgb_422 extension
> so we can still offer subsampling without having to do the color space
> conversion in the driver.
> 
> Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>

I think we can drop i915's support in a separate commit.

The only consumers I know of for this are wined3d and mplayer.  mplayer
appears to be broken -- trying various commandlines to use it didn't
result in using it, and the only command line I found for it (a mailing
list post back in 2008 about benchmarking it) fails.  wined3d has actual
glsl code for doing this stuff.

Comment from an mplayer dev:

<uau> anholt: mplayer/mplayer2 only use that if you explicitly specify a suboption to enable it
<uau> and i doubt many are even aware of that
<uau> so if you're working on mesa and considering removing the format i doubt it'd cause any problems

I agree that if anyone wanted these packing formats, the right thing to
supply would be GL_APPLE_ycbcr_422.

Reviewed-by: Eric Anholt <eric at anholt.net>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20111206/20721f4d/attachment-0001.pgp>


More information about the mesa-dev mailing list