[Piglit] [PATCH] framework/programs/run.py: Allow comments in test-list files.

2016-10-17 Thread Petri Latvala
IGT testing in Intel's CI is using a static list of tests to run. Commenting out tests and annotating them will help human-readability. Signed-off-by: Petri Latvala --- Possible duplicate, I didn't see this arrive on the mailing list myself framework/programs/run.py | 5 +++-- 1 file changed, 3

Re: [Piglit] [PATCH] fbo-clear-formats: Do one of the 4 clears unscissored.

2016-10-17 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Sat, Oct 15, 2016 at 1:36 AM, Eric Anholt wrote: > I was surprised to find through DEQP that 565 clearing was broken on > vc4, but that was because fbo-clear-formats never hit the hardware > clear path. This gives us a little exposure for that path (enough to

Re: [Piglit] [PATCH v6] cl: Add a test for the predefined macros

2016-10-17 Thread Jan Vesely
On Sat, 2016-10-15 at 14:43 +0200, Niels Ole Salscheider wrote: > v6: Fix OpenCL C version test if test is compiled against OpenCL 1.0 > v5: Use sscanf, fix CL_DEVICE_OPENCL_C_VERSION include guard, fix logic > v4: Test against env->version instead of opencl_version and never > report PIGLIT_SK

Re: [Piglit] [PATCH v2] arb_gpu_shader_fp64: Add tests to check float to double conversion under non-uniform control flow

2016-10-17 Thread Nicolai Hähnle
Reviewed-by: Nicolai Hähnle On 17.10.2016 08:26, Samuel Iglesias Gonsálvez wrote: We found issues with this case while developing i965's Ivybridge patches to support arb_gpu_shader_fp64. Signed-off-by: Samuel Iglesias Gonsálvez --- .../fs-non-uniform-control-flow-f2d.shader_test| 40

[Piglit] [PATCH 2/2] arb_gpu_shader_fp64: add some inout function argument tests

2016-10-17 Thread Nicolai Hähnle
From: Nicolai Hähnle This exposes a bug in st_glsl_to_tgsi. --- .../execution/fs-function-inout-array.shader_test | 43 ++ 1 file changed, 43 insertions(+) create mode 100644 tests/spec/arb_gpu_shader_fp64/execution/fs-function-inout-array.shader_test diff --git a/tests/

[Piglit] [PATCH 1/2] glsl-1.20: test array of structs inout parameter

2016-10-17 Thread Nicolai Hähnle
From: Nicolai Hähnle This exposes a bug in st_glsl_to_tgsi. --- .../fs-function-inout-array-of-structs.shader_test | 55 ++ 1 file changed, 55 insertions(+) create mode 100644 tests/spec/glsl-1.20/execution/fs-function-inout-array-of-structs.shader_test diff --git a/tests

Re: [Piglit] [PATCH v6] cl: Add a test for the predefined macros

2016-10-17 Thread Serge Martin
Le 2016-10-17 16:20, Jan Vesely a écrit : On Sat, 2016-10-15 at 14:43 +0200, Niels Ole Salscheider wrote: v6: Fix OpenCL C version test if test is compiled against OpenCL 1.0 v5: Use sscanf, fix CL_DEVICE_OPENCL_C_VERSION include guard, fix logic v4: Test against env->version instead of opencl

Re: [Piglit] [PATCH 1/4] util: Link to gbm if gbm is used

2016-10-17 Thread Anuj Phogat
On Thu, Oct 13, 2016 at 3:09 PM, Chad Versace wrote: > If PIGLIT_HAS_GBM, then add GBM_LDFLAGS to the linker flags. Fixes > undefined symbol errors at link time for me, using Arch Linux. > --- > tests/util/CMakeLists.txt | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/tests/util/CMake

[Piglit] [PATCH] arb_compute_variable_group_size: add ARB_compute_shader enables

2016-10-17 Thread Nicolai Hähnle
From: Nicolai Hähnle Without those, many of the tests started failing now that Mesa explicitly checks for the compute shader enable in the compiler. --- tests/spec/arb_compute_variable_group_size/compiler/do_nothing.comp | 1 + .../compiler/gl_LocalGroupSizeARB_illegal_use.comp

[Piglit] [PATCH] arb_compute_variable_group_size: Fix require section.

2016-10-17 Thread Matt Turner
--- .../linker/mixed_fixed_variable_local_work_size.shader_test | 2 +- .../linker/no_local_size_specified.shader_test | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/spec/arb_compute_variable_group_size/linker/mixed_fixed_variable

Re: [Piglit] [PATCH] arb_compute_variable_group_size: Fix require section.

2016-10-17 Thread Jordan Justen
Reviewed-by: Jordan Justen On 2016-10-17 12:39:18, Matt Turner wrote: > --- > .../linker/mixed_fixed_variable_local_work_size.shader_test | 2 > +- > .../linker/no_local_size_specified.shader_test | 2 > +- > 2 files changed, 2 insertions(+), 2 deletions(-)

Re: [Piglit] [PATCH] arb_compute_variable_group_size: Fix require section.

2016-10-17 Thread Samuel Pitoiset
Thanks for fixing this. Reviewed-by: Samuel Pitoiset On 10/17/2016 09:39 PM, Matt Turner wrote: --- .../linker/mixed_fixed_variable_local_work_size.shader_test | 2 +- .../linker/no_local_size_specified.shader_test | 2 +- 2 files changed, 2 insertions(+),

Re: [Piglit] [PATCH] arb_compute_variable_group_size: add ARB_compute_shader enables

2016-10-17 Thread Samuel Pitoiset
On 10/17/2016 09:38 PM, Nicolai Hähnle wrote: From: Nicolai Hähnle Without those, many of the tests started failing now that Mesa explicitly checks for the compute shader enable in the compiler. Well, ARB_compute_variable_group_size requires ARB_compute_shader, that's why I didn't check it

Re: [Piglit] [PATCH] arb_compute_variable_group_size: Fix require section.

2016-10-17 Thread Samuel Pitoiset
On 10/17/2016 09:45 PM, Samuel Pitoiset wrote: Thanks for fixing this. Reviewed-by: Samuel Pitoiset Actually, we need to check for both ARB_compute_shader and ARB_compute_variable_group_size since https://cgit.freedesktop.org/mesa/mesa/commit/?id=8785a8ff8948385a913e9bd75e8cdd1092bd750f.

Re: [Piglit] [PATCH] arb_compute_variable_group_size: Fix require section.

2016-10-17 Thread Matt Turner
On Mon, Oct 17, 2016 at 12:54 PM, Samuel Pitoiset wrote: > > > On 10/17/2016 09:45 PM, Samuel Pitoiset wrote: >> >> Thanks for fixing this. >> >> Reviewed-by: Samuel Pitoiset > > > Actually, we need to check for both ARB_compute_shader and > ARB_compute_variable_group_size since > https://cgit.fr

Re: [Piglit] [PATCH] arb_compute_variable_group_size: add ARB_compute_shader enables

2016-10-17 Thread Matt Turner
On Mon, Oct 17, 2016 at 12:38 PM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > Without those, many of the tests started failing now that Mesa explicitly > checks for the compute shader enable in the compiler. > --- The strange thing is that ARB_compute_shader doesn't say anything about an #e

Re: [Piglit] [PATCH] framework/programs/run.py: Allow comments in test-list files.

2016-10-17 Thread Dylan Baker
Quoting Petri Latvala (2016-10-17 02:52:36) > IGT testing in Intel's CI is using a static list of tests to > run. Commenting out tests and annotating them will help > human-readability. > > Signed-off-by: Petri Latvala > --- > > Possible duplicate, I didn't see this arrive on the mailing list my

Re: [Piglit] [PATCH] arb_compute_variable_group_size: Fix require section.

2016-10-17 Thread Samuel Pitoiset
On 10/17/2016 10:33 PM, Matt Turner wrote: On Mon, Oct 17, 2016 at 12:54 PM, Samuel Pitoiset wrote: On 10/17/2016 09:45 PM, Samuel Pitoiset wrote: Thanks for fixing this. Reviewed-by: Samuel Pitoiset Actually, we need to check for both ARB_compute_shader and ARB_compute_variable_grou

Re: [Piglit] [PATCH] arb_compute_variable_group_size: Fix require section.

2016-10-17 Thread Dylan Baker
Quoting Matt Turner (2016-10-17 13:33:11) > On Mon, Oct 17, 2016 at 12:54 PM, Samuel Pitoiset > wrote: > > > > > > On 10/17/2016 09:45 PM, Samuel Pitoiset wrote: > >> > >> Thanks for fixing this. > >> > >> Reviewed-by: Samuel Pitoiset > > > > > > Actually, we need to check for both ARB_compute_sh

[Piglit] [PATCH] arb_compute_variable_group_size: Add GL_ARB_compute_variable_group_size requirement

2016-10-17 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- .../linker/mixed_fixed_variable_local_work_size.shader_test | 1 + .../linker/no_local_size_specified.shader_test | 1 + 2 files changed, 2 insertions(+) diff --git a/tests/spec/arb_compute_variable_grou

Re: [Piglit] [PATCH] arb_compute_variable_group_size: Add GL_ARB_compute_variable_group_size requirement

2016-10-17 Thread Matt Turner
On Mon, Oct 17, 2016 at 6:04 PM, Ian Romanick wrote: > From: Ian Romanick > > Signed-off-by: Ian Romanick I actually sent and committed this patch earlier today. ___ Piglit mailing list Piglit@lists.freedesktop.org https://lists.freedesktop.org/mailma