[Piglit] [PATCH 1/5] OES_geometry_shader: Add preprocessor define test for EXT versions

2016-06-29 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- generated_tests/gen_extensions_defined.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/generated_tests/gen_extensions_defined.py b/generated_tests/gen_extensions_defined.py index

[Piglit] [PATCH 2/5] OES_shader_io_blocks: Add preprocessor define test for EXT version

2016-06-29 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- generated_tests/gen_extensions_defined.py | 1 + 1 file changed, 1 insertion(+) diff --git a/generated_tests/gen_extensions_defined.py b/generated_tests/gen_extensions_defined.py index

[Piglit] [PATCH 4/5] shader_runner: Allow SSO in OpenGL ES too

2016-06-29 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- tests/shaders/shader_runner.c | 21 + 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/tests/shaders/shader_runner.c b/tests/shaders/shader_runner.c index

[Piglit] [PATCH 5/5] OES_geometry_shader: Basic SSO IO validation tests

2016-06-29 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- .../user-defined-gs-input-in-block.shader_test | 70 ++ .../user-defined-gs-input-not-in-block.shader_test | 65 2 files changed, 135

[Piglit] [PATCH 3/5] shader_runner: Trivial whitespace fixes

2016-06-29 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- tests/shaders/shader_runner.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/shaders/shader_runner.c b/tests/shaders/shader_runner.c index 2f90fee..a979b46

[Piglit] [PATCH 09/11] mesa_shader_integer_functions: Clone arb_gpu_shader5 indirect sampler array access compiler test

2016-06-29 Thread Ian Romanick
From: Ian Romanick GL_MESA_shader_integer_functions does not add non-constant indexing of sampler arrays. cd tests/spec/mesa_shader_integer_functions/compiler cp ../../arb_gpu_shader5/compiler/indirect-projGradOffset-shadow.frag . sed --in-place -e

[Piglit] [PATCH 01/11] arb_gpu_shader5: Use 'piglit_vertex' and 'vertex shader passthrough'

2016-06-29 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- .../built-in-functions/const-bitCount.shader_test | 16 ++- .../const-bitfieldExtract.shader_test | 16 ++- .../const-bitfieldInsert.shader_test |

[Piglit] [PATCH 07/11] mesa_shader_integer_functions: Clone arb_gpu_shader5 overload compiler tests

2016-06-29 Thread Ian Romanick
From: Ian Romanick cd tests/spec/mesa_shader_integer_functions/compiler cp ../../arb_gpu_shader5/compiler/overloads-*.vert for i in overloads-*.vert; do sed --in-place -e 's/1[.]50/1.30/g;s/150/130/g' \ -e

[Piglit] [PATCH 10/11] mesa_shader_integer_functions: Clone arb_gpu_shader5 textureGather compile tests

2016-06-29 Thread Ian Romanick
From: Ian Romanick GL_MESA_shader_integer_functions does not enhance textureGather support, so all of these tests should fail. cd tests/spec/mesa_shader_integer_functions/compiler cp -a ../../arb_gpu_shader5/compiler/builtin-functions . cd builtin-functions

[Piglit] [PATCH 06/11] mesa_shader_integer_functions: Clone arb_gpu_shader5 sample qualifier tests

2016-06-29 Thread Ian Romanick
From: Ian Romanick GL_MESA_shader_integer_functions does not add the sample qualifier, so all of these tests should fail. cd tests/spec/mesa_shader_integer_functions/compiler cp -a ../../arb_gpu_shader5/compiler/sample-qualifier . cd sample-qualifier rm

[Piglit] [PATCH 04/11] mesa_shader_integer_functions: Clone arb_gpu_shader5 interpolation tests

2016-06-29 Thread Ian Romanick
From: Ian Romanick GL_MESA_shader_integer_functions does not add the interpolateAt functions, so all of these tests should fail. cd tests/spec/mesa_shader_integer_functions/compiler cp -a ../../arb_gpu_shader5/compiler/interpolation cd interpolation for

[Piglit] [PATCH 05/11] mesa_shader_integer_functions: Clone arb_gpu_shader5 precise qualifier tests

2016-06-29 Thread Ian Romanick
From: Ian Romanick GL_MESA_shader_integer_functions does not add the precise qualifier, so all of these tests should fail. cd tests/spec/mesa_shader_integer_functions/compiler cp -a ../../arb_gpu_shader5/compiler/precise-qualifier . cd precise-qualifier

[Piglit] [PATCH 08/11] mesa_shader_integer_functions: Clone arb_gpu_shader5 implicit conversions compiler tests

2016-06-29 Thread Ian Romanick
From: Ian Romanick cd tests/spec/mesa_shader_integer_functions/compiler cp ../../arb_gpu_shader5/compiler/implicit-conversions-*.vert for i in implicit-conversions-*.vert; do sed --in-place -e 's/1[.]50/1.30/g;s/150/130/g' \ -e

[Piglit] [PATCH 03/11] mesa_shader_integer_functions: Clone arb_gpu_shader5 overloads test

2016-06-29 Thread Ian Romanick
From: Ian Romanick cd tests/spec/mesa_shader_integer_functions/execution cp ../../arb_gpu_shader5/execution/overloads.shader_test . sed --in-place -e 's/1[.]50/1.30/g;s/150/130/g' \ -e 's/ARB_gpu_shader5/MESA_shader_integer_functions/g' \

[Piglit] [PATCH 00/11] Add tests for MESA_shader_integer_functions

2016-06-29 Thread Ian Romanick
These are mostly clones of existing ARB_gpu_shader5 tests. A few of the positive tests are converted to negative tests for features in ARB_gpu_shader5 but not in MESA_shader_interger_functions. I suspect that patch 2 is too large to make it to the mailing list, but the rest should be fine. The

[Piglit] [PATCH 11/11] mesa_shader_integer_functions: Negative compile tests for fma()

2016-06-29 Thread Ian Romanick
From: Ian Romanick GL_MESA_shader_integer_functions does not add the fma functions, so all of these tests should fail. Signed-off-by: Ian Romanick --- .../compiler/builtin-functions/fs-fma.frag | 18 ++

Re: [Piglit] [PATCH] initialized-texmemory: report WARN on failure

2016-06-29 Thread Brian Paul
On 06/29/2016 03:16 AM, Marek Olšák wrote: From: Marek Olšák The tested behavior is not required by the spec and the test itself claims it reports WARN, so this is a typo. --- tests/security/initialized-texmemory.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[Piglit] [PATCH] cmake: add module dependencies for generators

2016-06-29 Thread Dylan Baker
Signed-off-by: Dylan Baker --- generated_tests/CMakeLists.txt | 2 ++ 1 file changed, 2 insertions(+) diff --git a/generated_tests/CMakeLists.txt b/generated_tests/CMakeLists.txt index 7240572..c01ee72 100644 --- a/generated_tests/CMakeLists.txt +++

Re: [Piglit] [PATCH] arb_gpu_shader_int64: add test generation to standard testing template

2016-06-29 Thread Dylan Baker
Quoting Dave Airlie (2016-06-09 20:39:28) > From: Dave Airlie > > Rather than forking this code further, I've just added support to the > standard generator for the 64-bit tests. > > This relies on the shader_runner int64 support. > > Signed-off-by: Dave Airlie

Re: [Piglit] [PATCH] arb_gpu_shader_int64: generate int64 conversion tests.

2016-06-29 Thread Dylan Baker
Quoting Dave Airlie (2016-06-19 22:05:39) > From: Dave Airlie > > This is "ported" from the fp64 conversion generator, it > probably has a lot of stuff left over it shouldn't, but it > does generate the tests I expected to test 64-bit integer > conversions both implicit and

Re: [Piglit] [PATCH] initialized-texmemory: report WARN on failure

2016-06-29 Thread Marek Olšák
On Wed, Jun 29, 2016 at 6:51 PM, Dylan Baker wrote: > Reviewed-by: Dylan Baker > > Maybe a good future change would be to pull these "good security > practices, but not required by the spec" tests into a separate profile > and report fail? In the

Re: [Piglit] [PATCH] initialized-texmemory: report WARN on failure

2016-06-29 Thread Dylan Baker
Reviewed-by: Dylan Baker Maybe a good future change would be to pull these "good security practices, but not required by the spec" tests into a separate profile and report fail? Quoting Marek Olšák (2016-06-29 02:16:16) > From: Marek Olšák > > The

Re: [Piglit] [PATCH 1/5] util: Wrapper to load exact hex patterns for ints

2016-06-29 Thread Emil Velikov
On 14 June 2016 at 22:36, Andres Gomez wrote: > For some cases we want to have shaders where we load an exact bit > pattern into a signed int. > > This fixes the errno-based range validation that was broken when the > integer vbo attribute parsing on 32-bit systems was recently

Re: [Piglit] [PATCH 5/5] generators/gen_vs_in_fp64: Adds u/byte and u/short tests

2016-06-29 Thread Alejandro Piñeiro
Ditto: Acked-by: Alejandro Piñeiro On 14/06/16 23:37, Andres Gomez wrote: > Added more test cases combinations by including u/byte and u/short GL > types in the VBO so the stride is smaller than a word. > > Signed-off-by: Andres Gomez > --- >

Re: [Piglit] [PATCH 4/5] generators/gen_vs_in_fp64: Adds support to specify the GL type in the VBO

2016-06-29 Thread Alejandro Piñeiro
I don't have too much experience with generators and python code in general. But the patch looks good: Acked-by: Alejandro Piñeiro On 14/06/16 23:37, Andres Gomez wrote: > Signed-off-by: Andres Gomez > --- > generated_tests/gen_vs_in_fp64.py

Re: [Piglit] [PATCH 3/5] util: Add u/byte, u/short and half support to [vertex data]

2016-06-29 Thread Alejandro Piñeiro
On 14/06/16 23:36, Andres Gomez wrote: > This allows to set data of u/byte, u/short and half types for > attributes with the shader runner. > > For example: > 0/byte/int attname1/ushort/uint attname3/half/float > > The syntax has been extended so the recommended way has replaced the > old COUNT

Re: [Piglit] [PATCH 1/5] util: Wrapper to load exact hex patterns for ints

2016-06-29 Thread Alejandro Piñeiro
On 14/06/16 23:36, Andres Gomez wrote: > For some cases we want to have shaders where we load an exact bit > pattern into a signed int. > > This fixes the errno-based range validation that was broken when the > integer vbo attribute parsing on 32-bit systems was recently fixed. > > Signed-off-by:

Re: [Piglit] [PATCH 2/5] util/gl: Wrapper to load hex values for half floats

2016-06-29 Thread Alejandro Piñeiro
Reviewed-by: Alejandro Piñeiro On 14/06/16 23:36, Andres Gomez wrote: > For some cases we want to have shaders where we load an exact bit > pattern into a half float. > > Signed-off-by: Andres Gomez > --- > tests/util/piglit-util-gl.h | 27

Re: [Piglit] [PATCH] arb_copy_buffer-intra-buffer-copy: new GL_ARB_copy_buffer test

2016-06-29 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Tue, Jun 21, 2016 at 7:18 PM, Brian Paul wrote: > To exercise copying within a single buffer. > --- > tests/all.py | 1 + > tests/spec/arb_copy_buffer/CMakeLists.gl.txt | 1 + >

[Piglit] [PATCH v3] arb_texture_barrier: add an arb_texture_barrier test

2016-06-29 Thread Alejandro Piñeiro
Allow to configure: * texture resolution * number of blend passes * number of draw passes (calls to DrawRangeElements) * granularity (triangle count) of the square being draw * number of textures This commits adds to all.py execute the test with a combination of all this parameters.

[Piglit] [PATCH] initialized-texmemory: report WARN on failure

2016-06-29 Thread Marek Olšák
From: Marek Olšák The tested behavior is not required by the spec and the test itself claims it reports WARN, so this is a typo. --- tests/security/initialized-texmemory.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git