Re: [Piglit] non-concurrent piglit tests

2015-11-23 Thread Eric Anholt
Ilia Mirkin writes: > On Sun, Nov 22, 2015 at 7:12 PM, Eric Anholt wrote: >> Ilia Mirkin writes: >> >>> It looks like we're up to something like 1K non-concurrent piglit >>> tests... maybe more. Can someone who actually understands

Re: [Piglit] non-concurrent piglit tests

2015-11-22 Thread Eric Anholt
Ilia Mirkin writes: > It looks like we're up to something like 1K non-concurrent piglit > tests... maybe more. Can someone who actually understands the issues > explain what makes a piglit test unreliable when run concurrently with > another test? Then we can go and enable

Re: [Piglit] non-concurrent piglit tests

2015-11-22 Thread Ilia Mirkin
On Sun, Nov 22, 2015 at 7:12 PM, Eric Anholt wrote: > Ilia Mirkin writes: > >> It looks like we're up to something like 1K non-concurrent piglit >> tests... maybe more. Can someone who actually understands the issues >> explain what makes a piglit test

Re: [Piglit] non-concurrent piglit tests

2015-11-21 Thread Marek Olšák
On Fri, Nov 20, 2015 at 11:19 PM, Ilia Mirkin wrote: > Right... so I'm looking for concrete things I can look for in tests to > determine whether the run_concurrent=False is set incorrectly. I know > the *approximate* reasons, but I'd like to be certain and then go grep > it

Re: [Piglit] non-concurrent piglit tests

2015-11-20 Thread Dylan Baker
Any tests that use front buffer rendering cannot be run concurrently. I think that's some other cases. On Nov 20, 2015 12:32, "Ilia Mirkin" wrote: > It looks like we're up to something like 1K non-concurrent piglit > tests... maybe more. Can someone who actually understands

[Piglit] non-concurrent piglit tests

2015-11-20 Thread Ilia Mirkin
It looks like we're up to something like 1K non-concurrent piglit tests... maybe more. Can someone who actually understands the issues explain what makes a piglit test unreliable when run concurrently with another test? Then we can go and enable concurrency on probably 75% of the

Re: [Piglit] non-concurrent piglit tests

2015-11-20 Thread Ilia Mirkin
Right... so I'm looking for concrete things I can look for in tests to determine whether the run_concurrent=False is set incorrectly. I know the *approximate* reasons, but I'd like to be certain and then go grep it all and remove the run_concurrent flag from 75% of those that have it (either by

Re: [Piglit] non-concurrent piglit tests

2015-11-20 Thread Ilia Mirkin
And how do you tell if a test is using front buffer rendering? Is that the only situation, or are there others? On Fri, Nov 20, 2015 at 3:41 PM, Dylan Baker wrote: > Any tests that use front buffer rendering cannot be run concurrently. I > think that's some other cases.

Re: [Piglit] non-concurrent piglit tests

2015-11-20 Thread Dylan Baker
I don't remember. I asked Ken about it when Marek updated a huge swath of tests to run concurrent and I swapped the default flag from non-concurrent to concurrent, but I don't remember all of the details. Front buffer rendering and timer query were two cases where concurrent definitely wasn't

Re: [Piglit] non-concurrent piglit tests

2015-11-20 Thread Dylan Baker
Ah, okay. Never mind then. On Fri, Nov 20, 2015 at 05:19:45PM -0500, Ilia Mirkin wrote: > Right... so I'm looking for concrete things I can look for in tests to > determine whether the run_concurrent=False is set incorrectly. I know > the *approximate* reasons, but I'd like to be certain and then