<div dir="ltr">I'd caught this error in a branch as well, but noticed there were no users of this function so it didn't seem pressing to fix it.<div><br></div><div>If you wanted a little less code:</div><div><span style="font-family:arial,sans-serif;font-size:13px">group[name] = PiglitTest(args, run_concurrent=True)</span><br>
</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Sep 3, 2014 at 4:35 PM, Ilia Mirkin <span dir="ltr"><<a href="mailto:imirkin@alum.mit.edu" target="_blank">imirkin@alum.mit.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Wed, Sep 3, 2014 at 7:28 PM, Tom Stellard <<a href="mailto:tom@stellard.net">tom@stellard.net</a>> wrote:<br>

> On Wed, Sep 03, 2014 at 07:20:10PM -0400, Jan Vesely wrote:<br>
>> Signed-off-by: Jan Vesely <<a href="mailto:jan.vesely@rutgers.edu">jan.vesely@rutgers.edu</a>><br>
><br>
> Reviewed-by: Tom Stellard <<a href="mailto:thomas.stellard@amd.com">thomas.stellard@amd.com</a>><br>
><br>
>> ---<br>
>><br>
>> seemed wrong. Wonder if we can use concurrent tests.<br>
>><br>
><br>
> If you have render-nodes enabled, then you can run the cl tests<br>
> concurrently by adding the -c flag when running piglit.<br>
><br>
> Render-nodes were enabled by default in the 3.16 kernel, so<br>
> I think it may be too early to enable concurrent tests by default.<br>
<br>
</div>Conversely, you can use -1 to force single-threaded runs... seems like<br>
it's not for piglit to worry about that. -c forces all tests to run<br>
concurrently even if the test author said that it shouldn't run<br>
concurrently with other things. Not sure how common that is in the CL<br>
world though.<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
> -Tom<br>
><br>
>>  tests/cl.py | 2 +-<br>
>>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
>><br>
>> diff --git a/tests/cl.py b/tests/cl.py<br>
>> index fd698fc..b59368d 100644<br>
>> --- a/tests/cl.py<br>
>> +++ b/tests/cl.py<br>
>> @@ -21,7 +21,7 @@ def add_plain_test(group, name, args):<br>
>>  def add_concurrent_test(group, name, args):<br>
>>          test = PiglitTest(args)<br>
>>          test.run_concurrent = true;<br>
>> -        group[name] = PiglitTest(args)<br>
>> +        group[name] = test<br>
>><br>
>>  def add_plain_program_tester_test(group, name, path):<br>
>>          add_plain_test(group, name, ['cl-program-tester', path])<br>
>> --<br>
>> 1.9.3<br>
>><br>
>> _______________________________________________<br>
>> Piglit mailing list<br>
>> <a href="mailto:Piglit@lists.freedesktop.org">Piglit@lists.freedesktop.org</a><br>
>> <a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
> _______________________________________________<br>
> Piglit mailing list<br>
> <a href="mailto:Piglit@lists.freedesktop.org">Piglit@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
_______________________________________________<br>
Piglit mailing list<br>
<a href="mailto:Piglit@lists.freedesktop.org">Piglit@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
</div></div></blockquote></div><br></div>