<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [IVB bisected] assert_scanout:531 assertion 'width == info.width && height == info.height' failed"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=95401#c8">Comment # 8</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [IVB bisected] assert_scanout:531 assertion 'width == info.width && height == info.height' failed"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=95401">bug 95401</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>(In reply to Chris Bainbridge from <a href="show_bug.cgi?id=95401#c6">comment #6</a>)
<span class="quote">> Created <span class=""><a href="attachment.cgi?id=123795" name="attach_123795" title="sna-Fix-assert_scanout-regression.patch">attachment 123795</a> <a href="attachment.cgi?id=123795&action=edit" title="sna-Fix-assert_scanout-regression.patch">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=95401&attachment=123795'>[review]</a> [review]
> sna-Fix-assert_scanout-regression.patch

> The issue looks like a bug in the bisected commit (the "== 0" at end of
> drmIoctl line was missed), this patch reverts to the previous behaviour.</span >

I didn't even spot it when looking at the commit :( Thanks! The result would be
that we didn't notice the modeset and used the old flip buffer (as the flips
are reported after the change is in effect).

commit ab3ab412a459a95e94707a4e39a18790ee04e6e3
Author: Chris Bainbridge <<a href="mailto:chris.bainbridge@gmail.com">chris.bainbridge@gmail.com</a>>
Date:   Tue May 17 07:26:10 2016 +0100

    sna: Fix increment of modeset serial after applying CRTC

    We track how many mode sets have been made in order to detect stale
    flips (i.e. a sequence that crosses a mode change). This was broken by
    the logic inversion in setcrc in 46caee86db0f ("sna: Fix reporting of
    errno after setcrtc failure")

    References: <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [IVB bisected] assert_scanout:531 assertion 'width == info.width && height == info.height' failed"
   href="show_bug.cgi?id=95401">https://bugs.freedesktop.org/show_bug.cgi?id=95401</a>
    Signed-off-by: Chris Bainbridge <<a href="mailto:chris.bainbridge@gmail.com">chris.bainbridge@gmail.com</a>>
    Signed-off-by: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</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>