[Mesa-dev] Wrong colors in 3D apps on big-endian systems

Michel Dänzer michel at daenzer.net
Tue Mar 11 21:35:21 PDT 2014


On Die, 2014-03-11 at 17:51 +0100, Christian Zigotzky wrote:
> On 11.03.2014 16:25, Richard Sandiford wrote:
> > I just mean changing instances of things like PIPE_FORMAT_A8R8G8B8_UNORM
> > to PIPE_FORMAT_ARGB8888_UNORM in the relevant parts of the r600 support
> > (which is I think what Michel also meant -- like I say, I'm not really
> > adding anything new here).  PIPE_FORMAT_ARGB8888_UNORM always has the
> > alpha channel in the low bits of a 32-bit int according to host endianness,
> > whereas PIPE_FORMAT_A8R8G8B8_UNORM always has the alpha channel in the
> > first byte of memory.
> >
> > I can't say for sure which the "revelavant parts" are because I don't
> > know the r600 code.
> >
> > Thanks,
> > Richard
> >
> >
> I have replaced PIPE_FORMAT_A8R8G8B8_UNORM with 
> PIPE_FORMAT_ARGB8888_UNORM in the files r600_state.c and 
> evergreen_state.c.

That's not really a good idea, as the hardware supports both variants of
formats.


Do the attached patches help?


-- 
Earthling Michel Dänzer            |                  http://www.amd.com
Libre software enthusiast          |                Mesa and X developer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: st-dri-packed-formats.diff
Type: text/x-patch
Size: 3306 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20140312/fd975afa/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: r600g-8888-endianness.diff
Type: text/x-patch
Size: 495 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20140312/fd975afa/attachment-0001.bin>


More information about the mesa-dev mailing list