<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [SNA HSW bisected] git commit "sna: Skip sync if we have an idle coherent bo" causes corruption"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=94125#c7">Comment # 7</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [SNA HSW bisected] git commit "sna: Skip sync if we have an idle coherent bo" causes corruption"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=94125">bug 94125</a>
              from <span class="vcard"><a class="email" href="mailto:ccr@tnsp.org" title="Matti Hämäläinen <ccr@tnsp.org>"> <span class="fn">Matti Hämäläinen</span></a>
</span></b>
        <pre>Thanks for the explanation. I realized that as I was using the assert()
conditions from your original patch, they might not trigger, so added some
appropriate printf-debugs and those did show that the short-circuit paths are
working. ( <a href="http://tnsp.org/~ccr/intel-gfx/sna_skip_sync/test_patch.diff">http://tnsp.org/~ccr/intel-gfx/sna_skip_sync/test_patch.diff</a> )

In conclusion, I guess this works as expected now? :)</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>