<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - xorg-server 1.18.2 freeze desktop on i915"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=94515#c39">Comment # 39</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - xorg-server 1.18.2 freeze desktop on i915"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=94515">bug 94515</a>
              from <span class="vcard"><a class="email" href="mailto:joakim.tjernlund@infinera.com" title="Joakim Tjernlund <joakim.tjernlund@infinera.com>"> <span class="fn">Joakim Tjernlund</span></a>
</span></b>
        <pre>(In reply to Chris Wilson from <a href="show_bug.cgi?id=94515#c35">comment #35</a>)
<span class="quote">> It looks like the Client sent PresentPixmap with a divisor=1<<32, or even
> target_msc of 1<<32.

> I think

> commit f656f6afa288c63968f45d6d32c96a4cf6213117
> Author: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>>
> Date:   Tue Mar 22 22:11:18 2016 +0000

>     sna/present: Report BadValue if target_msc exceeds last known msc by
> 1<<32
>     
>     As we cannot queue a vblank further than 1<<32 into the future, nor can
>     we set a Timer to expire that far into the future, creating such a
>     vblank event is futile. Report BadValue and hope for the best.
>     
>     References: <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - xorg-server 1.18.2 freeze desktop on i915"
   href="show_bug.cgi?id=94515">https://bugs.freedesktop.org/show_bug.cgi?id=94515</a>
>     Signed-off-by: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>>

> should hide the immediate issue with the freeze / busy spin.

> Can someone attach gdb and put a breakpoint on present_pixmap() line 953,
> just after present_queue_vblank() returns an error and print the locals? I
> am most interested in getting the arguments to the function so we can
> compare what the client requested with the computed vblank.</span >

+if (warn_unless(msc - swap->msc < 1ull<<32))
Should that be <= ?

or possibly just cap the time to 1<<32 ?</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>