<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body><table border="1" cellspacing="0" cellpadding="8">
        <tr>
          <th>Priority</th>
          <td>medium
          </td>
        </tr>

        <tr>
          <th>Bug ID</th>
          <td><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - broken on big-endian systems"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=66663">66663</a>
          </td>
        </tr>

        <tr>
          <th>Assignee</th>
          <td>xorg-driver-ati@lists.x.org
          </td>
        </tr>

        <tr>
          <th>Summary</th>
          <td>broken on big-endian systems
          </td>
        </tr>

        <tr>
          <th>QA Contact</th>
          <td>xorg-team@lists.x.org
          </td>
        </tr>

        <tr>
          <th>Severity</th>
          <td>normal
          </td>
        </tr>

        <tr>
          <th>Classification</th>
          <td>Unclassified
          </td>
        </tr>

        <tr>
          <th>OS</th>
          <td>All
          </td>
        </tr>

        <tr>
          <th>Reporter</th>
          <td>kettenis@openbsd.org
          </td>
        </tr>

        <tr>
          <th>Hardware</th>
          <td>Other
          </td>
        </tr>

        <tr>
          <th>Status</th>
          <td>NEW
          </td>
        </tr>

        <tr>
          <th>Version</th>
          <td>git
          </td>
        </tr>

        <tr>
          <th>Component</th>
          <td>Driver/Radeon
          </td>
        </tr>

        <tr>
          <th>Product</th>
          <td>xorg
          </td>
        </tr></table>
      <p>
        <div>
        <pre>Created <span class=""><a href="attachment.cgi?id=82134" name="attach_82134" title="patch">attachment 82134</a> <a href="attachment.cgi?id=82134&action=edit" title="patch">[details]</a></span>
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 <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - PageFlip urxvt/xterm screen corruptions"
   href="show_bug.cgi?id=33738">https://bugs.freedesktop.org/show_bug.cgi?id=33738</a>
(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.</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>