Re: [Piglit] [PATCH 1/1] cl: fix adding concurent tests

2014-09-26 Thread Tom Stellard
On Thu, Sep 25, 2014 at 10:19:16PM -0400, Jan Vesely wrote: Didn't notice somebody dropped Tom from the cc list. On Thu, 2014-09-04 at 10:28 -0400, Jan Vesely wrote: On Wed, 2014-09-03 at 20:20 -0700, Dylan Baker wrote: I'd caught this error in a branch as well, but noticed there were no

Re: [Piglit] [PATCH 1/1] cl: fix adding concurent tests

2014-09-26 Thread Jan Vesely
On Fri, 2014-09-26 at 07:22 -0700, Tom Stellard wrote: On Thu, Sep 25, 2014 at 10:19:16PM -0400, Jan Vesely wrote: Didn't notice somebody dropped Tom from the cc list. On Thu, 2014-09-04 at 10:28 -0400, Jan Vesely wrote: On Wed, 2014-09-03 at 20:20 -0700, Dylan Baker wrote: I'd

Re: [Piglit] [PATCH 1/1] cl: fix adding concurent tests

2014-09-26 Thread Tom Stellard
On Fri, Sep 26, 2014 at 01:51:15PM -0400, Jan Vesely wrote: On Fri, 2014-09-26 at 07:22 -0700, Tom Stellard wrote: On Thu, Sep 25, 2014 at 10:19:16PM -0400, Jan Vesely wrote: Didn't notice somebody dropped Tom from the cc list. On Thu, 2014-09-04 at 10:28 -0400, Jan Vesely wrote:

Re: [Piglit] [PATCH 1/1] cl: fix adding concurent tests

2014-09-25 Thread Jan Vesely
Didn't notice somebody dropped Tom from the cc list. On Thu, 2014-09-04 at 10:28 -0400, Jan Vesely wrote: On Wed, 2014-09-03 at 20:20 -0700, Dylan Baker wrote: 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.

[Piglit] [PATCH 1/1] cl: fix adding concurent tests

2014-09-03 Thread Jan Vesely
Signed-off-by: Jan Vesely jan.ves...@rutgers.edu --- seemed wrong. Wonder if we can use concurrent tests. tests/cl.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/cl.py b/tests/cl.py index fd698fc..b59368d 100644 --- a/tests/cl.py +++ b/tests/cl.py @@ -21,7 +21,7 @@

Re: [Piglit] [PATCH 1/1] cl: fix adding concurent tests

2014-09-03 Thread Tom Stellard
On Wed, Sep 03, 2014 at 07:20:10PM -0400, Jan Vesely wrote: Signed-off-by: Jan Vesely jan.ves...@rutgers.edu Reviewed-by: Tom Stellard thomas.stell...@amd.com --- seemed wrong. Wonder if we can use concurrent tests. If you have render-nodes enabled, then you can run the cl tests

Re: [Piglit] [PATCH 1/1] cl: fix adding concurent tests

2014-09-03 Thread Ilia Mirkin
On Wed, Sep 3, 2014 at 7:28 PM, Tom Stellard t...@stellard.net wrote: On Wed, Sep 03, 2014 at 07:20:10PM -0400, Jan Vesely wrote: Signed-off-by: Jan Vesely jan.ves...@rutgers.edu Reviewed-by: Tom Stellard thomas.stell...@amd.com --- seemed wrong. Wonder if we can use concurrent tests.

Re: [Piglit] [PATCH 1/1] cl: fix adding concurent tests

2014-09-03 Thread Dylan Baker
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. If you wanted a little less code: group[name] = PiglitTest(args, run_concurrent=True) On Wed, Sep 3, 2014 at 4:35 PM, Ilia Mirkin imir...@alum.mit.edu wrote: On