<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_ASSIGNED "
   title="ASSIGNED - Hangcheck timer too agressive to pass dEQP for SNBGT1"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=93029#c16">Comment # 16</a>
              on <a class="bz_bug_link 
          bz_status_ASSIGNED "
   title="ASSIGNED - Hangcheck timer too agressive to pass dEQP for SNBGT1"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=93029">bug 93029</a>
              from <span class="vcard"><a class="email" href="mailto:mark.a.janes@intel.com" title="Mark Janes <mark.a.janes@intel.com>"> <span class="fn">Mark Janes</span></a>
</span></b>
        <pre>Mika: this bug is not a regression, and does not block a release.  There may be
a point (soon) at which Google will require stricter dEQP conformance, but
based on the list here, the long_running_shaders are not required.

<a href="https://android.googlesource.com/platform/external/deqp/+/4423ddaeef6c6a252152f774b76950f62c412f94/android/cts">https://android.googlesource.com/platform/external/deqp/+/4423ddaeef6c6a252152f774b76950f62c412f94/android/cts</a>

However, the hang detection's intermittent firing on *other* non-hanging dEQP
tests creates instability in the test suite.  The patch you attached to this
bug improves stability, and I haven't seen any spurious dEQP failures using it
on SNB (long-running-tests are disabled in CI).

(In reply to Mika Kuoppala from <a href="show_bug.cgi?id=93029#c15">comment #15</a>)
<span class="quote">> (In reply to Eero Tamminen from <a href="show_bug.cgi?id=93029#c12">comment #12</a>)
> > (In reply to Mark Janes from <a href="show_bug.cgi?id=93029#c10">comment #10</a>)
> > > With Mika's patch, spurious hangchecks can still be generated by invoking
> > > the the tests listed in the comments (long_running_shaders).  These tests
> > > are blacklisted by Chrome's test suite.
> > > 
> > > The good news is that all of the *intermittent* gpu hangs that were
> > > generated by the rest of the suite have been fixed by this patch.
> > 
> > Are the still hanging tests too heavy for a slow GPU like SNB GT1?
> > 
> > What if the hang period would just be increased while running them?

> I would like to understand the tests a little better and try to improve the
> heuristics, before adding a tunable.

> Is this still blocking a release?</span ></pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the QA Contact for the bug.</li>
          <li>You are on the CC list for the bug.</li>
      </ul>
    </body>
</html>