[Piglit] [PATCH v2] Remove ARB_timer_query / EXT_timer_query from quick.py
imirkin at alum.mit.edu
Thu Nov 20 12:21:14 PST 2014
On Thu, Nov 20, 2014 at 3:10 PM, Mark Janes <mark.a.janes at intel.com> wrote:
> Ben Widawsky <ben at bwidawsk.net> writes:
>> On Tue, Nov 18, 2014 at 02:48:09PM -0800, Mark Janes wrote:
>>> Ilia Mirkin <imirkin at alum.mit.edu> writes:
>>> > FWIW, on nv50 and nvc0, EXT_timer_query time-elapsed and
>>> > ARB_timer_query query GL_TIMESTAMP reliably fail, and the rest
>>> > reliably pass. I don't remember ever seeing inconsistent behaviour.
>>> > FWIW I don't really know what these extensions do, or what the tests
>>> > check for, but just thought I'd provide the data point.
>>> We probably see the failure more often because we run test with gbm, so
>>> the CPU is consistently pegged on all cores. Comparing cpu times with
>>> gpu times is likely to be error-prone in this situation.
>>> Since it only affects our automated system, we'll exclude it when we
>>> invoke piglit.
>> I think we should change the "Couldn't find appropriate number of iterations" to
>> a skip. That seems to be the common failure on GBM (for me). All other failures
>> are likely to be real^winteresting failures.
> The failure I've seen on the older machines is "GPU time didn't match
> CPU time". I've already disabled this test in our environment, so it's
> a non-issue.
FWIW "GPU time didn't match CPU time" is the reason for the nv50/nvc0
failures as well:
The gpu time is like 5e-6 while cpu time is like 0.1.
More information about the Piglit