[Bug 66663] New: broken on big-endian systems

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=66663

          Priority: medium
            Bug ID: 66663
          Assignee: xorg-driver-ati at lists.x.org
           Summary: broken on big-endian systems
        QA Contact: xorg-team at lists.x.org
          Severity: normal
    Classification: Unclassified
                OS: All
          Reporter: kettenis at openbsd.org
          Hardware: Other
            Status: NEW
           Version: git
         Component: Driver/Radeon
           Product: xorg

Created attachment 82134
  --> https://bugs.freedesktop.org/attachment.cgi?id=82134&action=edit
patch

The current xf86-video-ati is broken on big-endian systems.  The problem is
that
RADEON_TILING_SURFACE isn't set on the front buffer since commits
ef9bfb262db7004bef3704e5d914687e50d3fca4 and
e5bd99faa3b6629a55168386d5dfa936ee4e97ae.

As a result the RADEON_TILING_SWAP_32BIT and RADEON_TILING_SWAP_16BIT flags
don't have any effect, and any direct access to the front buffer will not do
the necessary byte swapping.  The effect is blatantly obvious if you disable
acceleration (option NoAccel "on"), but also happens with (EXA) acceleration
enabled.  With acceleration disabled, the colors are all wrong.  With
acceleration enabled, the background of my xterms is yellow when it should be
white.

The attached patch fixes this, but might reintroduce problems with page
flipping as mentioned in https://bugs.freedesktop.org/show_bug.cgi?id=33738
(the bug report that seems to have triggered the removal of the
RADEON_TILING_SURFACE flag for the front buffer).  But only on big-endian
platforms.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.x.org/archives/xorg-driver-ati/attachments/20130707/5aa50642/attachment.html>