Re: [Piglit] [PATCH 1/5] fbo-drawbuffers-none: Add the instructions for running subtests

2014-05-20 Thread Marek Olšák
If you fix the typo that Ilia pointed out, the series is: Reviewed-by: Marek Olšák marek.ol...@amd.com Marek On Mon, May 19, 2014 at 10:51 PM, Anuj Phogat anuj.pho...@gmail.com wrote: Signed-off-by: Anuj Phogat anuj.pho...@gmail.com --- .../arb_framebuffer_object/fbo-drawbuffers-none.c |

[Piglit] [PATCH 11/11] glean/glsl1: Delete duplicated array tests.

2014-05-20 Thread Matt Turner
--- tests/all.py | 14 tests/glean/tglsl1.cpp | 183 - 2 files changed, 197 deletions(-) diff --git a/tests/all.py b/tests/all.py index 1b27a6f..80413c7 100644 --- a/tests/all.py +++ b/tests/all.py @@ -269,20 +269,6 @@

[Piglit] [PATCH 01/11] glean: Remove the getString test.

2014-05-20 Thread Matt Turner
From: Kenneth Graunke kenn...@whitecape.org This test always passes, so all it does is make sure glGetString doesn't crash. But essentially every Piglit test that does extension checks already does that. --- tests/glean/CMakeLists.gl.txt | 1 - tests/glean/tgetstr.cpp | 74

[Piglit] [PATCH 07/11] glean/glsl1: Delete duplicated function tests.

2014-05-20 Thread Matt Turner
--- tests/all.py | 8 --- tests/glean/tglsl1.cpp | 155 - 2 files changed, 163 deletions(-) diff --git a/tests/all.py b/tests/all.py index 984b177..d591d5e 100644 --- a/tests/all.py +++ b/tests/all.py @@ -226,17 +226,9 @@

[Piglit] [PATCH 10/11] glean/glsl1: Delete duplicated matrix tests.

2014-05-20 Thread Matt Turner
--- tests/all.py | 15 --- tests/glean/tglsl1.cpp | 248 - 2 files changed, 263 deletions(-) diff --git a/tests/all.py b/tests/all.py index f9bc763..1b27a6f 100644 --- a/tests/all.py +++ b/tests/all.py @@ -269,21 +269,6 @@

[Piglit] [PATCH 02/11] glean/glsl1: Port illegal shader tests to piglit.

2014-05-20 Thread Matt Turner
From: Kenneth Graunke kenn...@whitecape.org --- tests/glean/tglsl1.cpp | 152 + tests/shaders/glsl1/COPYRIGHT | 31 + tests/shaders/glsl1/break-with-no-loop.frag| 9 ++

[Piglit] [PATCH 05/11] glean/glsl1: Delete duplicated basic arithmetic tests.

2014-05-20 Thread Matt Turner
--- tests/all.py | 12 tests/glean/tglsl1.cpp | 169 - 2 files changed, 181 deletions(-) diff --git a/tests/all.py b/tests/all.py index a80ce62..1d869e2 100644 --- a/tests/all.py +++ b/tests/all.py @@ -165,20 +165,8 @@

[Piglit] [PATCH 00/11] Remove duplicated glean tests.

2014-05-20 Thread Matt Turner
glean contains a number of tests that test things we have coverage for elsewhere in piglit. It also runs its tests nonconcurrently, which is kind of not nice. This patch series is by no means exhaustive or complete. Just a quick pass over glean's glsl1 tests. Reduces the number of glean tests on

[Piglit] [PATCH 04/11] glean/glsl1: Delete duplicated logic operator tests.

2014-05-20 Thread Matt Turner
--- tests/all.py | 10 tests/glean/tglsl1.cpp | 143 - 2 files changed, 153 deletions(-) diff --git a/tests/all.py b/tests/all.py index e80deec..a80ce62 100644 --- a/tests/all.py +++ b/tests/all.py @@ -227,18 +227,8 @@

[Piglit] [PATCH 06/11] glean/glsl1: Delete duplicated equality/inequality tests.

2014-05-20 Thread Matt Turner
--- tests/all.py | 16 tests/glean/tglsl1.cpp | 225 - 2 files changed, 241 deletions(-) diff --git a/tests/all.py b/tests/all.py index 1d869e2..984b177 100644 --- a/tests/all.py +++ b/tests/all.py @@ -199,22 +199,6 @@

[Piglit] [PATCH 09/11] glean/glsl1: Delete duplicated comment tests.

2014-05-20 Thread Matt Turner
--- tests/all.py | 5 tests/glean/tglsl1.cpp | 63 -- 2 files changed, 68 deletions(-) diff --git a/tests/all.py b/tests/all.py index 2f9acba..f9bc763 100644 --- a/tests/all.py +++ b/tests/all.py @@ -260,11 +260,6 @@

[Piglit] [PATCH 03/11] glean/glsl1: Delete duplicated built-in function tests.

2014-05-20 Thread Matt Turner
I've less cross() function, in-place because we don't have an equivalent test for it elsewhere. --- tests/all.py | 31 tests/glean/tglsl1.cpp | 452 - 2 files changed, 483 deletions(-) diff --git a/tests/all.py b/tests/all.py index

[Piglit] [PATCH] arb_draw_instanced: use default window size, set viewport

2014-05-20 Thread Brian Paul
--- tests/spec/arb_draw_instanced/execution/elements.c |5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/tests/spec/arb_draw_instanced/execution/elements.c b/tests/spec/arb_draw_instanced/execution/elements.c index 7cb418b..84eed79 100644 ---

[Piglit] [PATCH 02/10] arb_instanced_arrays: move and rename test

2014-05-20 Thread Brian Paul
Move tests/general/draw-instanced-divisor.c to tests/spec/arb_instanced_arrays/drawarrays.c --- tests/all.py |2 +- tests/general/CMakeLists.gl.txt |1 - tests/general/draw-instanced-divisor.c| 191 -

[Piglit] [PATCH 0/10] rename some tests

2014-05-20 Thread Brian Paul
The names of many piglit tests are pretty inconsistent. Nowadays, most tests get put in a tests/spec/arb_foo_bar/ directory and the executables are prefixed with arb_foo_bar-*. But older tests don't really follow any pattern. Many were just dumped in tests/general/ I'd like to clean up

[Piglit] [PATCH 09/10] arb_draw_elements_base_vertex: move the drawelements test

2014-05-20 Thread Brian Paul
--- tests/all.py |5 +- tests/general/CMakeLists.gl.txt|1 - tests/general/draw-elements-base-vertex.c | 156 .../CMakeLists.gl.txt |1 +

[Piglit] [PATCH 04/10] arb_draw_elements_base_vertex: rename dlist test

2014-05-20 Thread Brian Paul
--- tests/all.py |2 +- .../CMakeLists.gl.txt |2 +- .../dlist-arb_draw_instanced.c | 96 tests/spec/arb_draw_elements_base_vertex/dlist.c | 96 4 files

[Piglit] [PATCH 10/10] arb_draw_elements_base_vertex: rename/move negative index test

2014-05-20 Thread Brian Paul
--- tests/all.py |4 +- tests/general/CMakeLists.gl.txt|4 - tests/general/draw-elements-base-vertex-neg.c | 141 .../CMakeLists.gl.txt |1 +

[Piglit] [PATCH 01/10] arb_draw_instanced: move and rename drawarrays test

2014-05-20 Thread Brian Paul
Move tests/general/draw-instanced.c to tests/spec/arb_draw_instanced/drawarrays.c --- tests/all.py |2 +- tests/general/CMakeLists.gl.txt|1 - tests/general/draw-instanced.c | 183

[Piglit] [PATCH 03/10] arb_instanced_arrays: put extension prefix on executables

2014-05-20 Thread Brian Paul
Prefix the executables with arb_instanced_arrays to be more consistent. --- tests/all.py |6 +++--- tests/spec/arb_instanced_arrays/CMakeLists.gl.txt |4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/all.py b/tests/all.py

[Piglit] [PATCH 07/10] arb_draw_elements_base_vertex: rename multidrawelements test

2014-05-20 Thread Brian Paul
--- tests/all.py |2 +- .../CMakeLists.gl.txt |2 +- .../multi-draw-elements-base-vertex.c | 190 .../multidrawelements.c| 190 4 files

[Piglit] [PATCH 08/10] arb_draw_elements_base_vertex: move bounds checking test

2014-05-20 Thread Brian Paul
--- tests/all.py |2 +- tests/general/CMakeLists.gl.txt|1 - tests/general/draw-elements-base-vertex-bounds.c | 127 .../CMakeLists.gl.txt |1 +

[Piglit] [PATCH 05/10] arb_draw_elements_base_vertex: rename instanced drawelements test

2014-05-20 Thread Brian Paul
--- tests/all.py |2 +- .../CMakeLists.gl.txt |2 +- .../draw-elements-instanced-base-vertex.c | 191 .../drawelements-instanced.c | 191 4 files

Re: [Piglit] [PATCH 02/11] glean/glsl1: Port illegal shader tests to piglit.

2014-05-20 Thread Kenneth Graunke
On 05/20/2014 03:31 PM, Matt Turner wrote: From: Kenneth Graunke kenn...@whitecape.org --- tests/glean/tglsl1.cpp | 152 + tests/shaders/glsl1/COPYRIGHT | 31 + tests/shaders/glsl1/break-with-no-loop.frag|