Re: [Piglit] [PATCH 1/2] drawoverhead: test more state changes, remove the GL_FRAMEBUFFER_SRGB test

2017-06-14 Thread Brian Paul
For both, Reviewed-by: Brian Paul On 06/14/2017 03:37 PM, Marek Olšák wrote: From: Marek Olšák the sRGB test is unrealistic --- tests/perf/drawoverhead.c | 63 +-- 1 file changed, 61 insertions(+), 2 deletions(-) diff --git a/tests/perf/drawover

Re: [Piglit] Status of Glean test rewrites?

2017-06-14 Thread Dylan Baker
Quoting Timothy Arceri (2017-06-14 18:17:19) > On 15/06/17 10:57, Dylan Baker wrote: > > I have (a now fairly old) series to port either the arb fp or vp tests to > > shader_runner on the list. I have no idea how much work they'd need to > > land, or > > if they're really relavent. > > > > Ian ha

Re: [Piglit] Status of Glean test rewrites?

2017-06-14 Thread Timothy Arceri
On 15/06/17 10:57, Dylan Baker wrote: I have (a now fairly old) series to port either the arb fp or vp tests to shader_runner on the list. I have no idea how much work they'd need to land, or if they're really relavent. Ian had patches (I have no idea how far done) to generate arb_fp and arb_vp

Re: [Piglit] Status of Glean test rewrites?

2017-06-14 Thread Dylan Baker
I have (a now fairly old) series to port either the arb fp or vp tests to shader_runner on the list. I have no idea how much work they'd need to land, or if they're really relavent. Ian had patches (I have no idea how far done) to generate arb_fp and arb_vp tests which would superscede both glean

[Piglit] Status of Glean test rewrites?

2017-06-14 Thread Timothy Arceri
Hi All, I recently got a cpu with more than 2 cores \o/ and it's become even more obvious just how much of a bottle neck the glean tests are. Anyway what is the currently status of the rewrite? Have all the current patches landed or are there still patchs waiting review/yet to be finished?

[Piglit] [PATCH 2/2] drawoverhead: print test numbers

2017-06-14 Thread Marek Olšák
From: Marek Olšák and align all results to the same column --- tests/perf/drawoverhead.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/tests/perf/drawoverhead.c b/tests/perf/drawoverhead.c index 9afd3d0..bedaecb 100644 --- a/tests/perf/drawoverhead.c +++ b/tests/pe

[Piglit] [PATCH 1/2] drawoverhead: test more state changes, remove the GL_FRAMEBUFFER_SRGB test

2017-06-14 Thread Marek Olšák
From: Marek Olšák the sRGB test is unrealistic --- tests/perf/drawoverhead.c | 63 +-- 1 file changed, 61 insertions(+), 2 deletions(-) diff --git a/tests/perf/drawoverhead.c b/tests/perf/drawoverhead.c index 76aed20..9afd3d0 100644 --- a/tests/perf/d

Re: [Piglit] [PATCH 1/7] appveyro: Remove help invocation from cmake

2017-06-14 Thread Jose Fonseca
I see. Fair enough. Reviewed-by: Jose Fonseca Jose On 14/06/17 19:01, Dylan Baker wrote: According to the man page (for 3.8.2): --help,-help,-usage,-h,-H,/? Print usage information and exit. Usage describes the basic command line interface and its opti

Re: [Piglit] [PATCH 1/7] appveyro: Remove help invocation from cmake

2017-06-14 Thread Dylan Baker
According to the man page (for 3.8.2): --help,-help,-usage,-h,-H,/? Print usage information and exit. Usage describes the basic command line interface and its options. And I saw this behavior, without removing the -H option no build direcotry was created and ni

Re: [Piglit] Appveyor: Add support for python tests in addition to native tests

2017-06-14 Thread Jose Fonseca
I got the missing PATCH 3/4. Reviewed-by: Jose Fonseca From: Dylan Baker Sent: Tuesday, June 13, 2017 21:59 To: piglit@lists.freedesktop.org Cc: Jose Fonseca Subject: Appveyor: Add support for python tests in addition to native tests This little series fixes

Re: [Piglit] [PATCH] tests/igt.py: mark segfaulted IGT process as 'crash'

2017-06-14 Thread Martin Peres
Reviewed-by: Martin Peres I will push the patch now. It has waited on the list long-enough! Thanks, Martin On 16/05/17 18:08, Tomi Sarvela wrote: On IGT testing segfaulted processes are lumped together with failed tests in results. This patch separates the two for correctness: test doesn't ha