<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_REOPENED "
   title="REOPENED - [IGT][BDW] kms_frontbuffer_tracking@fbc-*draw* has inconsistent results"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=101623#c7">Comment # 7</a>
              on <a class="bz_bug_link 
          bz_status_REOPENED "
   title="REOPENED - [IGT][BDW] kms_frontbuffer_tracking@fbc-*draw* has inconsistent results"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=101623">bug 101623</a>
              from <span class="vcard"><a class="email" href="mailto:marta.lofstedt@intel.com" title="Marta Löfstedt <marta.lofstedt@intel.com>"> <span class="fn">Marta Löfstedt</span></a>
</span></b>
        <pre>Still no review on my patch. I asked on IRC:
<marta_> I did this IGT patch before vacation:
<a href="https://patchwork.freedesktop.org/patch/164198/">https://patchwork.freedesktop.org/patch/164198/</a>
<marta_> but I have had no comments. 
<marta_> We have flip-floppy results for kms_frontbuffer_tracking for a lot of
fbc related subtests. If I increase time out from 2000ms to 5000ms the tests
appear to always pass.
<marta_> Note, when the test was created the timeout was 5000ms.
<marta_> If anyone has an objection to changing back to 5000ms please speak up!
<marta_> bug: <a class="bz_bug_link 
          bz_status_REOPENED "
   title="REOPENED - [IGT][BDW] kms_frontbuffer_tracking@fbc-*draw* has inconsistent results"
   href="show_bug.cgi?id=101623">https://bugs.freedesktop.org/show_bug.cgi?id=101623</a>
* ThSenior has quit (Remote host closed the connection)
* DJAnonimo (<a href="mailto:shiners@genf196.server4you.de">shiners@genf196.server4you.de</a>) has joined
<danvet> tsa, pls have an eye on shard-kbl runtime
<danvet> and complain if still not good enough yet :-)
<danvet> with the very latest igt that I just pushed ofc
<danvet> marta_, pls ping pzanoni
<danvet> he's the master of all

I CC:ed Paulo Zanoni on the patch. 
To document further the initial timeout when the test was created was 5000ms
then it was changed to 2000ms by this:
commit 64590c7b768dc8d8dd962f812d5ff5a39e7e8b54
Author: Paulo Zanoni <<a href="mailto:paulo.r.zanoni@intel.com">paulo.r.zanoni@intel.com</a>>
Date:   Fri Aug 14 16:08:07 2015 -0300

    kms_frontbuffer_tracking: reduce the FBC wait timeout to 2s

    Just like we did for PSR, let's do it for FBC. FBC gets reenabled in
    just 50ms, so the 5000ms timeout is huge. On the other hand, we only
    pay the 5000ms timeout full price 9 times when running
    kms_frontbuffer_tracking --fbc-only, so this change shouldn't save too
    much time.

    Signed-off-by: Paulo Zanoni <<a href="mailto:paulo.r.zanoni@intel.com">paulo.r.zanoni@intel.com</a>>

It is the interesting to note that the PSR timeout was later changed back to
5000ms again. commit d074b44ab6a7ac14cc36b1dc98df5bcf73c82f95
Author: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com">rodrigo.vivi@intel.com</a>>
Date:   Mon Nov 2 15:54:06 2015 -0800

    kms_frontbuffer_tracking: Increase the time we wait for PSR.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are on the CC list for the bug.</li>
          <li>You are the QA Contact for the bug.</li>
      </ul>
    </body>
</html>