Re: [Piglit] [PATCH v2] winsys-framework Default to showing window

2015-11-19 Thread Martin Peres
On 17/11/15 22:01, Dave Airlie wrote: On 18 November 2015 at 05:49, Arthur Huillet wrote: Thanks :) Reviewed by: Arthur Huillet Reviewed-by: Dave Airlie Pushed ___ Piglit mailing

Re: [Piglit] [PATCH 4/4] ext_shader_samples_identical: Simple fragment shader rendering test

2015-11-19 Thread Neil Roberts
Ian Romanick writes: > +# Group EXT_shader_samples_identical > +with profile.group_manager( > +PiglitGLTest, > +grouptools.join('spec', 'EXT_shader_samples_identical')) as g: > +g(['ext_shader_samples_identical', '2']) > +

Re: [Piglit] [PATCH] arb_shader_storage_buffer_object: Test copying a large SSBO item

2015-11-19 Thread Ian Romanick
On 11/19/2015 05:23 PM, Jordan Justen wrote: > Signed-off-by: Jordan Justen > Cc: Kristian Høgsberg > Cc: Samuel Iglesias Gonsalvez > Cc: Iago Toral Quiroga > --- > This fails to link on i965. > >

[Piglit] [PATCH] arb_shader_storage_buffer_object: Test copying a large SSBO item

2015-11-19 Thread Jordan Justen
Signed-off-by: Jordan Justen Cc: Kristian Høgsberg Cc: Samuel Iglesias Gonsalvez Cc: Iago Toral Quiroga --- This fails to link on i965. .../execution/large-field-copy.shader_test | 66

Re: [Piglit] [PATCH 4/4] ext_shader_samples_identical: Simple fragment shader rendering test

2015-11-19 Thread Ian Romanick
On 11/19/2015 09:59 AM, Neil Roberts wrote: > Ian Romanick writes: > >> +# Group EXT_shader_samples_identical >> +with profile.group_manager( >> +PiglitGLTest, >> +grouptools.join('spec', 'EXT_shader_samples_identical')) as g: >> +

[Piglit] [PATCH 8/9] ext_shader_samples_identical: Verify that every function overload exists

2015-11-19 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- .../glsl-1.10/compiler/all-functions.frag | 21 + .../glsl-1.10/compiler/all-functions.vert | 21 + .../glsl-1.50/compiler/all-functions.frag

[Piglit] [PATCH 3/9] all.py: Everything in MSAA_SAMPLE_COUNTS is already even

2015-11-19 Thread Ian Romanick
From: Ian Romanick If this test has problems with odd sample counts, the test should detect that and SKIP. Signed-off-by: Ian Romanick --- tests/all.py | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/tests/all.py

[Piglit] [PATCH 7/9] ext_shader_samples_identical: add test for preprocessor define

2015-11-19 Thread Ian Romanick
From: Ian Romanick v2: Move the changes to tests/spec/CMakeLists.txt to a later patch. Signed-off-by: Ian Romanick --- tests/spec/CMakeLists.txt | 1 - .../glsl-1.10/preprocessor/define.frag| 19

[Piglit] [PATCH 9/9] ext_shader_samples_identical: Simple fragment shader rendering test

2015-11-19 Thread Ian Romanick
From: Ian Romanick Render a simple image. Scan the image. At each texel, render green if textureSamplesIdenticalEXT returns false. If textureSamplesIdenticalEXT returns true, examine each sample. If the samples are all the same color, render blue. Render red

[Piglit] [PATCH 6/9] arb_texture_multisample: Use piglit_draw_rect_from_arrays instead of open-coding it

2015-11-19 Thread Ian Romanick
From: Ian Romanick This requires changing from drawing a triangle to drawing a diamond (quadrilateral) shape. Signed-off-by: Ian Romanick Suggested-by: Neil Roberts --- tests/spec/arb_texture_multisample/texelfetch.c |

[Piglit] [PATCH 5/9] arb_texture_multisample: Use piglit_draw_rect_tex instead of open-coding it

2015-11-19 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick Suggested-by: Neil Roberts --- tests/spec/arb_texture_multisample/texelfetch.c | 22 +- 1 file changed, 1 insertion(+), 21 deletions(-) diff --git

Re: [Piglit] [PATCH v2] Implement piglit tests for EXT_blend_func_extended

2015-11-19 Thread Ilia Mirkin
Pushed, thanks! On Tue, Nov 17, 2015 at 6:57 PM, Ryan Houdek wrote: > --- > tests/all.py | 13 +++ > .../api/CMakeLists.gles2.txt | 5 + > .../api/CMakeLists.gles3.txt | 6 ++ >

[Piglit] [PATCH 2/9] all.py: Use the same MSAA_SAMPLE_COUNTS idiom almost everywhere

2015-11-19 Thread Ian Romanick
From: Ian Romanick I could not figure out how to get for sample_count in (str(x) for x in MSAA_SAMPLE_COUNTS): to work with the (0,) + MSAA_SAMPLE_COUNTS places. Signed-off-by: Ian Romanick Cc: Dylan Baker ---

Re: [Piglit] [PATCH] arb_shader_storage_buffer_object: Test copying a large SSBO item

2015-11-19 Thread Samuel Iglesias Gonsálvez
Reviewed-by: Samuel Iglesias Gonsálvez Thanks for writing a test for this issue, Sam On 20/11/15 02:23, Jordan Justen wrote: > Signed-off-by: Jordan Justen > Cc: Kristian Høgsberg > Cc: Samuel Iglesias Gonsalvez

Re: [Piglit] [PATCH] arb_separate_shader_objects: test mixed explicit and non-explicit locations

2015-11-19 Thread Timothy Arceri
On Thu, 2015-11-19 at 14:21 +0100, Gregory Hainaut wrote: > Hello Timothy, > > > There seems to be 3 separate issues so I think we should have a separate > > tests > > (or at least cleary labebled subtests) for each of them. > > In this case, I propose that we keep your new test. It has a better