<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - screen corruption using SNA and TearFree on Intel GeminiLake"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=105720#c36">Comment # 36</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - screen corruption using SNA and TearFree on Intel GeminiLake"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=105720">bug 105720</a>
              from <span class="vcard"><a class="email" href="mailto:chris@chris-wilson.co.uk" title="Chris Wilson <chris@chris-wilson.co.uk>"> <span class="fn">Chris Wilson</span></a>
</span></b>
        <pre>So, we simplified the wait_for_shadow() to always do a fullscreen copy onto a
new buffer and by first filling the buffer with a checkerboard we confirmed
that the copy was not occurring.

By using #define DEBUG_NO_BLT 1 and comparing with Option "AccelMethod" "blt",
we were able to conclude this affects the render copy; i.e. gen9_render.c seems
completely suspect.

That the copy fails entirely suggests some sort of pixel kill, or that the
sampler read hits 0. The latter has happened before, when the sample offset was
larger the supported table size.

But there is another possibility, that we fail to submit the sampler table,
which is tucked in at the end of the batch.

Currently we are investigating this angle with

diff --git a/src/sna/sna.h b/src/sna/sna.h
index 6fe1f0d2..af61e4b7 100644
--- a/src/sna/sna.h
+++ b/src/sna/sna.h
@@ -80,7 +80,7 @@ SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 #include "compiler.h"
 #include "debug.h"

-#define DEBUG_NO_BLT 0
+#define DEBUG_NO_BLT 1

 #define DEBUG_FLUSH_BATCH 0

and so far, all seems ok.</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>