<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body><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> changed
          <a class="bz_bug_link 
          bz_status_NEEDINFO "
   title="NEEDINFO - screen corruption using SNA and TearFree on Intel GeminiLake"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=105720">bug 105720</a>
          <br>
             <table border="1" cellspacing="0" cellpadding="8">
          <tr>
            <th>What</th>
            <th>Removed</th>
            <th>Added</th>
          </tr>

         <tr>
           <td style="text-align:right;">Status</td>
           <td>NEW
           </td>
           <td>NEEDINFO
           </td>
         </tr></table>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEEDINFO "
   title="NEEDINFO - screen corruption using SNA and TearFree on Intel GeminiLake"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=105720#c39">Comment # 39</a>
              on <a class="bz_bug_link 
          bz_status_NEEDINFO "
   title="NEEDINFO - 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>I suspect related to commit 746c8f143afad7aaa66c484485fc39888d437a3f
Author: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>>
Date:   Sun Jun 10 20:43:09 2018 +0100

    drm/i915: Apply batch location restrictions before pinning

    We special case the position of the batch within the GTT to prevent
    negative self-relocation deltas from underflowing. However, that
    restriction is being applied after a trial pin of the batch in its
    current position. Thus we are not rejecting an invalid location if the
    batch has been used before, leading to an assertion if we happen to need
    to rearrange the entire payload. In the worst case, this may cause a GPU
    hang on gen7 or perhaps missing state.

    References: <a class="bz_bug_link 
          bz_status_NEEDINFO "
   title="NEEDINFO - screen corruption using SNA and TearFree on Intel GeminiLake"
   href="show_bug.cgi?id=105720">https://bugs.freedesktop.org/show_bug.cgi?id=105720</a>
    Fixes: 2889caa92321 ("drm/i915: Eliminate lots of iterations over the
execob
jects array")
    Signed-off-by: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>>
    Cc: Joonas Lahtinen <<a href="mailto:joonas.lahtinen@linux.intel.com">joonas.lahtinen@linux.intel.com</a>>
    Cc: Martin Peres <<a href="mailto:martin.peres@linux.intel.com">martin.peres@linux.intel.com</a>>
    Link:
<a href="https://patchwork.freedesktop.org/patch/msgid/20180610194325.13467-2-c">https://patchwork.freedesktop.org/patch/msgid/20180610194325.13467-2-c</a>
<a href="mailto:hris@chris-wilson.co.uk">hris@chris-wilson.co.uk</a>
    Reviewed-by: Joonas Lahtinen <<a href="mailto:joonas.lahtinen@linux.intel.com">joonas.lahtinen@linux.intel.com</a>></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>