Re: [Piglit] [PATCH 1/8] README: Fix spelling mistake

2016-04-16 Thread Vinson Lee
On Sat, Apr 16, 2016 at 10:28 AM, Eric Engestrom wrote: > On Mon, Apr 04, 2016 at 11:16:54AM -0700, Dylan Baker wrote: >> Do you have commit access or do you need someone to push these for you? >> >> For the series: >> Reviewed-by: Dylan Baker > > Somehow I completely missed your reply, sorry abo

Re: [Piglit] [PATCH 5/8] arb_shader_image_load_store: add additional coherency test

2016-04-16 Thread Francisco Jerez
Nicolai Hähnle writes: > On 16.04.2016 16:01, Francisco Jerez wrote: >> Jan Vesely writes: >> >>> On Sat, 2016-04-16 at 10:19 -0500, Nicolai Hähnle wrote: On 15.04.2016 17:12, Francisco Jerez wrote: > >> >>> > > For a test doing almost the same thing but

[Piglit] Nearly finished: shader_runner running THOUSANDS of tests per process

2016-04-16 Thread Marek Olšák
Hi, This makes shader_runner very fast. The expected result is 40% decrease in quick.py running time, or a 12x faster piglit run if you run shader tests alone. Branch: https://cgit.freedesktop.org/~mareko/piglit/log/?h=shader-runner Changes: 1) Any number of test files can be specified as comma

Re: [Piglit] [PATCH 5/8] arb_shader_image_load_store: add additional coherency test

2016-04-16 Thread Francisco Jerez
Jan Vesely writes: > On Sat, 2016-04-16 at 10:19 -0500, Nicolai Hähnle wrote: >> On 15.04.2016 17:12, Francisco Jerez wrote: >> > >> > > >> > > > >> > > > > >> > > > > > >> > > > > > For a test doing almost the same thing but not relying on >> > > > > > unspecified >> > > > > > invocation or

Re: [Piglit] [PATCH 5/8] arb_shader_image_load_store: add additional coherency test

2016-04-16 Thread Jan Vesely
On Sat, 2016-04-16 at 10:19 -0500, Nicolai Hähnle wrote: > On 15.04.2016 17:12, Francisco Jerez wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > For a test doing almost the same thing but not relying on > > > > > > unspecified > > > > > > invocation ordering, see > > > > > > "tests/s

Re: [Piglit] [PATCH 5/8] arb_shader_image_load_store: add additional coherency test

2016-04-16 Thread Francisco Jerez
Nicolai Hähnle writes: > On 15.04.2016 17:12, Francisco Jerez wrote: >> For a test doing almost the same thing but not relying on unspecified >> invocation ordering, see >> "tests/spec/arb_shader_image_load_store/shader-mem-barrier.c" -- It >> would be interesting to see whether y

Re: [Piglit] [PATCH 1/8] README: Fix spelling mistake

2016-04-16 Thread Eric Engestrom
On Mon, Apr 04, 2016 at 11:16:54AM -0700, Dylan Baker wrote: > Do you have commit access or do you need someone to push these for you? > > For the series: > Reviewed-by: Dylan Baker Somehow I completely missed your reply, sorry about that. I don't have access, someone else will have to push it :

Re: [Piglit] [PATCH 5/8] arb_shader_image_load_store: add additional coherency test

2016-04-16 Thread Nicolai Hähnle
On 15.04.2016 17:12, Francisco Jerez wrote: For a test doing almost the same thing but not relying on unspecified invocation ordering, see "tests/spec/arb_shader_image_load_store/shader-mem-barrier.c" -- It would be interesting to see whether you can get it to reproduce the GCN coherency bug usin

[Piglit] [PATCH] util: probe pixel rectangles as ubyte if possible (v2)

2016-04-16 Thread Marek Olšák
From: Marek Olšák v2: int -> bool, unify all new functions into one --- tests/util/piglit-util-gl.c | 94 + 1 file changed, 94 insertions(+) diff --git a/tests/util/piglit-util-gl.c b/tests/util/piglit-util-gl.c index e8a47c5..09eb99d 100644 --- a/tes