[Piglit] [PATCH] Mark timer tests as not concurrent

Nicolai Hähnle nhaehnle at gmail.com
Thu Sep 15 16:11:11 UTC 2016


On 13.09.2016 15:38, Emil Velikov wrote:
> On 12 September 2016 at 16:31, Nicolai Hähnle <nhaehnle at gmail.com> wrote:
>> From: Nicolai Hähnle <nicolai.haehnle at amd.com>
>>
>> This reduces the chance of false positives caused by system load.
> From my experience the {arb,ext}-timer-query tests have always been
> inconsistent. Even on a consecutive, non-concurrent, runs outside
> piglit. System load no affect here, afaict.
>
> What is your observation before/after the patch, please add some
> numbers to the commit message.

Going to add this to the commit message, but posting here as well for 
the record:

On an 8-core test machine with an AMD Polaris, I've observed about a 50% 
false positive rate on these tests when running concurrently. On the 
same system, I have not observed a single failure in 10 runs when 
running non-concurrently.

> As it helps at least some setups:
> Acked-by: Emil Velikov <emil.velikov at collabora.com>

Thanks!

Nicolai

> Thanks
> Emil
>


More information about the Piglit mailing list