<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Memory corruption when uploading DXT5 cubemap faces"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=97473#c2">Comment # 2</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Memory corruption when uploading DXT5 cubemap faces"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=97473">bug 97473</a>
              from <span class="vcard"><a class="email" href="mailto:kenneth@whitecape.org" title="Kenneth Graunke <kenneth@whitecape.org>"> <span class="fn">Kenneth Graunke</span></a>
</span></b>
        <pre>There are many solutions to this:

1. Fall back to untiled for compressed textures that are too big to blit.
   ("Better to fall off a performance cliff than hit a spike trap.")  Easy to
implement, and backport to stable releases, but awful.  Need to get the math
right to avoid sending more things than necessary off the performance cliff.

2. Take Chris Wilson's "unlimited GTT map!" patch, which apparently exploits a
Kernel 4.9 feature to eliminate the need for the check.  I'm not sure how this
works yet.  It would be nice to solve it without requiring Kernel 4.9.

3. Teach the BLT code about compressed textures.  Doable.  Probably
backportable.

4. Use BLORP for the map-blit path.  Jason has patches to make it able to do
dumb copies by lying about formats.  We could use this for MapTextureImage.

I'm thinking we want to do #3 (fix BLT, and backport), #2 (make CPU mapping
always work), and longer term #4 (stop using the BLT).</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the QA Contact for the bug.</li>
      </ul>
    </body>
</html>