[Piglit] [PATCH 7/8] teximage-errors: Only run the depth texture subtests when depth textures are supported

2017-05-30 Thread Ian Romanick
From: Ian Romanick Fixes failure on radeon and probably others. Signed-off-by: Ian Romanick --- tests/texturing/teximage-errors.c | 32 ++-- 1 file changed, 18 insertions(+), 14 deletions(-) diff --git a/tests/texturing/teximage-errors.c b/tests/texturing/teximage

[Piglit] [PATCH 4/8] texsubimage-depth-formats: Require ARB_depth_texture

2017-05-30 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- tests/texturing/texsubimage-depth-formats.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/texturing/texsubimage-depth-formats.c b/tests/texturing/texsubimage-depth-formats.c index aa6f586..ead2811 100644 --- a/tests/texturing/tex

[Piglit] [PATCH 2/8] fbo: Require ARB_depth_texture in more tests

2017-05-30 Thread Ian Romanick
From: Ian Romanick All of these tests fail when run on implementations that support ARB_framebuffer_object but not ARB_depth_texture. Signed-off-by: Ian Romanick --- tests/fbo/fbo-depth-tex1d.c | 1 + tests/fbo/fbo-incomplete-invalid-texture.c

[Piglit] [PATCH 6/8] fbo: Use renderbuffers instead of textures

2017-05-30 Thread Ian Romanick
From: Ian Romanick This allows fbo-blit-stretch to run on platforms that support ARB_framebuffer_object but not ARB_texture_non_power_of_two. Coneptually similar to 7b3f6d5. Signed-off-by: Ian Romanick --- tests/fbo/fbo-blit-stretch.cpp | 26 -- 1 file changed, 16 inse

[Piglit] [PATCH 3/8] getteximage-depth: Require ARB_depth_texture

2017-05-30 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- tests/texturing/getteximage-depth.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/texturing/getteximage-depth.c b/tests/texturing/getteximage-depth.c index 2986962..bc79466 100644 --- a/tests/texturing/getteximage-depth.c +++ b/t

[Piglit] [PATCH 8/8] teximage-errors: Fix comment formatting after previous commit

2017-05-30 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- tests/texturing/teximage-errors.c | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/tests/texturing/teximage-errors.c b/tests/texturing/teximage-errors.c index bb993d3..fe95c82 100644 --- a/tests/texturing/

[Piglit] [PATCH 1/8] fbo: Fix trivial typo in comment

2017-05-30 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- tests/fbo/fbo-incomplete-invalid-texture.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/fbo/fbo-incomplete-invalid-texture.c b/tests/fbo/fbo-incomplete-invalid-texture.c index cd8adaf..ac183b4 100644 --- a/tests/fb

[Piglit] [PATCH 5/8] fbo: Remove support for the checkerboard test pattern

2017-05-30 Thread Ian Romanick
From: Ian Romanick The comment says (correctly) that it should not be used, and removing it make a later change easier. Signed-off-by: Ian Romanick --- tests/fbo/fbo-blit-stretch.cpp | 18 +- 1 file changed, 1 insertion(+), 17 deletions(-) diff --git a/tests/fbo/fbo-blit-stret

Re: [Piglit] [PATCH] test/gleantest: fix missing --quick property from quick profile

2017-05-30 Thread Dylan Baker
Quoting Timothy Arceri (2017-05-18 23:21:13) > --- > framework/test/gleantest.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/framework/test/gleantest.py b/framework/test/gleantest.py > index b2d56f3..0220c1a 100644 > --- a/framework/test/gleantest.py > +++ b/framework

[Piglit] [PATCH] glsl-1.50: add linker test for unused in out blocks

2017-05-30 Thread Józef Kucia
This test exposes a Mesa GLSL linker bug. The test fails with the following error message: error: Input block `blk' is not an output of the previous stage Section 4.3.4 (Inputs) of the GLSL 1.50 spec says: "Only the input variables that are actually read need to be written by the previ

Re: [Piglit] [PATCH] arb_texture_view: add simple test for rendering to R32UI texture view

2017-05-30 Thread Marek Olšák
Pushed, thanks! Marek On Tue, May 30, 2017 at 3:28 PM, Józef Kucia wrote: > This test exposes a Radeonsi driver bug. > > The test works on Nvidia binary driver and Mesa Intel (Broadwell). > It fails on Radeonsi with Cape Verde GPU. > --- > tests/all.py | 1 + >

[Piglit] [PATCH] arb_texture_view: add simple test for rendering to R32UI texture view

2017-05-30 Thread Józef Kucia
This test exposes a Radeonsi driver bug. The test works on Nvidia binary driver and Mesa Intel (Broadwell). It fails on Radeonsi with Cape Verde GPU. --- tests/all.py | 1 + tests/spec/arb_texture_view/CMakeLists.gl.txt | 1 + tests/spec/arb_texture_view/rende

Re: [Piglit] [PATCH] arb_get_program_binary: check correctly got_error

2017-05-30 Thread Lionel Landwerlin
Reviewed-by: Lionel Landwerlin On 29/05/17 16:18, Juan A. Suarez Romero wrote: Seems it was checking if *not* error got, instead of the other way around. Suggested-by: Fabio Lagalla Signed-off-by: Juan A. Suarez Romero --- tests/spec/arb_get_program_binary/retrievable_hint.c | 2 +- 1 file

Re: [Piglit] [PATCH] arb_shader_image_load_store: set KHR_no_error compatibility

2017-05-30 Thread Timothy Arceri
Looks good to me: Reviewed-by: Timothy Arceri On 26/05/17 06:21, Samuel Pitoiset wrote: Signed-off-by: Samuel Pitoiset --- tests/spec/arb_shader_image_load_store/atomicity.c | 1 + tests/spec/arb_shader_image_load_store/bitcast.c| 1 + tests/spec/arb_shader_image_load