[Piglit] [PATCH 1/8] GL3.2 GL_ARB_sync: Test to check the correct error messages are returned for invalid inputs for ClientWaitSync

2013-10-07 Thread Nicholas Mack
v2: Fix comments, initialize variables, rewrite loop through all bit masks v3: Remove redundant code, minor fixes, and add to all.tests --- tests/all.tests | 1 + tests/spec/arb_sync/CMakeLists.gl.txt | 1 + tests/spec/arb_sync/ClientWaitSync-errors.c | 96

[Piglit] [PATCH 2/8] GL3.2 GL_ARB_sync: Test for valid return values from ClientWaitSync

2013-10-07 Thread Nicholas Mack
v2: Fix comments, initialize variables. Still need to figure out if GPU commands needed before testing these. v3: Rewrite test cases, fix comment and config block and add to all.tests --- tests/all.tests | 1 + tests/spec/arb_sync/CMakeLists.gl.txt|

[Piglit] [PATCH 6/8] GL3.2 GL_ARB_sync: Test that IsSync returns true/false if it is given a valid/invalid sync object name

2013-10-07 Thread Nicholas Mack
v2: Fix comments, initialize variables v3: Minor fixes and added to all.tests --- tests/all.tests | 1 + tests/spec/arb_sync/CMakeLists.gl.txt | 1 + tests/spec/arb_sync/IsSync.c | 83 +++ 3 files changed, 85 insertions(+) create

[Piglit] [PATCH 4/8] GL3.2 GL_ARB_sync: Test for the correct error messages caused by invalid input to FenceSync

2013-10-07 Thread Nicholas Mack
v2: Fix comments, remove unnecessary tests. v3: Added to all.tests Reviewed-by: Chad Versace chad.vers...@linux.intel.com --- tests/all.tests| 1 + tests/spec/arb_sync/CMakeLists.gl.txt | 1 + tests/spec/arb_sync/FenceSync-errors.c | 72

[Piglit] [PATCH 5/8] GL3.2 GL_ARB_sync: Test that GetSynciv sets correct error codes.

2013-10-07 Thread Nicholas Mack
v2: Fix comments, initialize variables v3: Minor fixes and add to all.tests --- tests/all.tests| 1 + tests/spec/arb_sync/CMakeLists.gl.txt | 1 + tests/spec/arb_sync/GetSynciv-errors.c | 91 ++ 3 files changed, 93 insertions(+) create

[Piglit] [PATCH 3/8] GL3.2 GL_ARB_sync: Basic test for DeleteSync

2013-10-07 Thread Nicholas Mack
v2: Fix comments, add test for passing invalid sync to IsSync(), change variable types. v3: Minor fixes and add to all.tests --- tests/all.tests | 1 + tests/spec/arb_sync/CMakeLists.gl.txt | 1 + tests/spec/arb_sync/DeleteSync.c | 78

[Piglit] [PATCH 7/8] GL3.2 GL_ARB_sync: Test that a sync object is initialized with the correct properties

2013-10-07 Thread Nicholas Mack
v2: Fix comments, add extra checks for length variable being modified v3: Start some rendering to insure initialized values and add to all.tests --- tests/all.tests | 1 + tests/spec/arb_sync/CMakeLists.gl.txt | 1 + tests/spec/arb_sync/sync-initialize.c | 134

Re: [Piglit] [PATCH 4/5] arb_texture_query_levels: add compiler tests for textureQueryLevels()

2013-10-07 Thread Ian Romanick
On 10/04/2013 09:08 PM, Chris Forbes wrote: diff --git a/tests/spec/arb_texture_query_levels/compiler/builtin-functions-float-samplers.frag b/tests/spec/arb_texture_query_levels/compiler/builtin-functions-float-samplers.frag new file mode 100644 index 000..d6a19ba --- /dev/null +++

[Piglit] [PATCH 8/8] GL3.2 GL_ARB_sync: Test that the correct error messages are returned from invalid input for WaitSync

2013-10-07 Thread Nicholas Mack
v2: Fix comments, remove redundant code v3: Add to all.tests --- tests/all.tests | 1 + tests/spec/arb_sync/CMakeLists.gl.txt | 1 + tests/spec/arb_sync/WaitSync-errors.c | 82 +++ 3 files changed, 84 insertions(+) create mode 100644

Re: [Piglit] [PATCH V9] KHR_debug: test ObjectLabel(), GetObjectLabel(), ObjectPtrLabel() and GetObjectPtrLabel()

2013-10-07 Thread Eric Anholt
I've pushed the test now. I made some trivial fixes on the way: whitespace consistency, but also fixing duplication of the extension name in the summary results. Thanks for all your work on this! diff --git a/tests1 /all.tests b/tests/all.tests index 3e5e1a3..cec532a 100644 ---

Re: [Piglit] [PATCH 5/5] arb_texture_query_levels: add execution tests

2013-10-07 Thread Ian Romanick
On 10/04/2013 10:04 PM, Matt Turner wrote: On Fri, Oct 4, 2013 at 9:08 PM, Chris Forbes chr...@ijw.co.nz wrote: Ensure that the returned level count is correct in various situations. Signed-off-by: Chris Forbes chr...@ijw.co.nz --- .../execution/fs-baselevel.shader_test | 40

Re: [Piglit] [PATCH 1/4] util: Add utility function to accumulate results of subtests

2013-10-07 Thread Eric Anholt
Ian Romanick i...@freedesktop.org writes: From: Ian Romanick ian.d.roman...@intel.com This looks a lot like piglit_merge_result. pgpcE5xiQ7TU_.pgp Description: PGP signature ___ Piglit mailing list Piglit@lists.freedesktop.org

Re: [Piglit] [PATCH 1/4] util: Add utility function to accumulate results of subtests

2013-10-07 Thread Chad Versace
On 10/07/2013 09:02 AM, Eric Anholt wrote: Ian Romanick i...@freedesktop.org writes: From: Ian Romanick ian.d.roman...@intel.com This looks a lot like piglit_merge_result. Agreed. Let's re-use piglit_merge_result(). ___ Piglit mailing list

Re: [Piglit] [PATCH 2/4] util: Add common framework for command-line invoked subtests

2013-10-07 Thread Chad Versace
On 10/04/2013 06:10 PM, Ian Romanick wrote: From: Ian Romanick ian.d.roman...@intel.com There are two parts to this. First, command line options of the form -subtest foo have their argument (the foo part) stored in a list in the piglit_gl_test_config. Tests can use this functionality without

Re: [Piglit] [PATCH 3/4] util: Add a -list-subtests option that will list all the subtests

2013-10-07 Thread Chad Versace
On 10/04/2013 06:11 PM, Ian Romanick wrote: From: Ian Romanick ian.d.roman...@intel.com This required some ugly hacking about to get the list of subtests to process_args. Suggestions? Let's avoid these ugly hacks. Here's my suggestion, which covers patches 2 and 3. Patch 2/4 moves `struct

Re: [Piglit] [PATCH] GLSL: Test that input layout qualifiers cannot be used on variable declarations

2013-10-07 Thread Paul Berry
On 20 September 2013 12:31, Nicholas Mack nichm...@gmail.com wrote: --- ...-layout-qualifiers-with-variable-declarations.geom | 19 +++ ...-layout-qualifiers-with-variable-declarations.geom | 19 +++ 2 files changed, 38 insertions(+) create mode 100644

Re: [Piglit] [PATCH 3/4] util: Add a -list-subtests option that will list all the subtests

2013-10-07 Thread Ian Romanick
On 10/07/2013 11:02 AM, Chad Versace wrote: On 10/04/2013 06:11 PM, Ian Romanick wrote: From: Ian Romanick ian.d.roman...@intel.com This required some ugly hacking about to get the list of subtests to process_args. Suggestions? Let's avoid these ugly hacks. Here's my suggestion, which

[Piglit] [PATCH] shaders: add test for bool casting of large integers.

2013-10-07 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com I noticed the TGSI convertor just does INEG then AND 0x3f80, which works for small values, but fails for larger ints. This seems to fail on nvidia (4.3.0 NVIDIA 325.15) and on ironlake with mesa 9.2. Signed-off-by: Dave Airlie airl...@redhat.com ---

Re: [Piglit] [PATCH 3/4] util: Add a -list-subtests option that will list all the subtests

2013-10-07 Thread Chad Versace
On 10/07/2013 04:19 PM, Ian Romanick wrote: On 10/07/2013 11:02 AM, Chad Versace wrote: On 10/04/2013 06:11 PM, Ian Romanick wrote: From: Ian Romanick ian.d.roman...@intel.com This required some ugly hacking about to get the list of subtests to process_args. Suggestions? Let's avoid these

Re: [Piglit] [PATCH] textureSize: Initialize sample_count to 1.

2013-10-07 Thread Paul Berry
On 23 September 2013 20:46, Kenneth Graunke kenn...@whitecape.org wrote: The multisampling tests initialize the number of miplevels to the number of samples. Since this is zero, we allocate an array of zero miplevels, and then try to write to the first element. Fixes the following Valgrind

Re: [Piglit] [PATCH 1/2] GLSL 1.50: Test that version 140 and version 150 shaders can be linked together

2013-10-07 Thread Paul Berry
On 27 September 2013 11:55, Nicholas Mack nichm...@gmail.com wrote: --- .../glsl-1.50/linker/versions-mingled.shader_test | 29 ++ 1 file changed, 29 insertions(+) create mode 100644 tests/spec/glsl-1.50/linker/versions-mingled.shader_test diff --git

Re: [Piglit] [PATCH] GL 1.50: Test that UNIFORM_BLOCK_REFERENCED_BY_GEOMETRY_SHADER is accepted by GetActiveUniformBlockiv()

2013-10-07 Thread Eric Anholt
Nicholas Mack nichm...@gmail.com writes: v2: Incorporate this test into referenced-by-shader.c @@ -40,10 +41,6 @@ PIGLIT_GL_TEST_CONFIG_BEGIN config.supports_gl_compat_version = 10; - config.window_width = 10; - config.window_height = 10; - config.window_visual =

[Piglit] [PATCH 01/13] arb_shader_atomic_counters: Import common helper functions.

2013-10-07 Thread Francisco Jerez
--- tests/all.tests| 3 + tests/spec/CMakeLists.txt | 1 + .../arb_shader_atomic_counters/CMakeLists.gl.txt | 13 + .../spec/arb_shader_atomic_counters/CMakeLists.txt | 1 + tests/spec/arb_shader_atomic_counters/common.c |

[Piglit] [PATCH 00/13] Tests for the ARB_shader_atomic_counters_extension.

2013-10-07 Thread Francisco Jerez
This patch series implements a bunch of tests for the ARB_shader_atomic_counters extension -- Patches adding support for it to the i965 driver have been around for a while on the mesa-dev mailing list, and they can also be found in my personal branch [1]. 54 of these tests pass using the i965

[Piglit] [PATCH 03/13] arb_shader_atomic_counters: Test the atomic counter query functions.

2013-10-07 Thread Francisco Jerez
--- tests/all.tests| 1 + .../arb_shader_atomic_counters/CMakeLists.gl.txt | 2 + .../arb_shader_atomic_counters/active-counters.c | 411 + 3 files changed, 414 insertions(+) create mode 100644

[Piglit] [PATCH 02/13] arb_shader_atomic_counters: Import GLSL parser tests.

2013-10-07 Thread Francisco Jerez
--- tests/all.tests| 3 +++ .../compiler/builtins.frag | 24 ++ .../compiler/declaration-allowed.frag | 21 +++ .../compiler/declaration-arg-inout.frag| 21 +++

[Piglit] [PATCH 04/13] arb_shader_atomic_counters: Test dynamic indexing of atomic counter arrays.

2013-10-07 Thread Francisco Jerez
--- tests/all.tests| 1 + .../arb_shader_atomic_counters/CMakeLists.gl.txt | 1 + .../arb_shader_atomic_counters/array-indexing.c| 175 + 3 files changed, 177 insertions(+) create mode 100644

[Piglit] [PATCH 06/13] arb_shader_atomic_counters: Test that atomics aren't allocated from the default uniform block.

2013-10-07 Thread Francisco Jerez
--- tests/all.tests| 1 + .../arb_shader_atomic_counters/CMakeLists.gl.txt | 1 + .../arb_shader_atomic_counters/default-partition.c | 122 + 3 files changed, 124 insertions(+) create mode 100644

[Piglit] [PATCH 09/13] arb_shader_atomic_counters: Test the maximum number of supported counters, buffers and bindings.

2013-10-07 Thread Francisco Jerez
--- tests/all.tests| 1 + .../arb_shader_atomic_counters/CMakeLists.gl.txt | 1 + .../spec/arb_shader_atomic_counters/max-counters.c | 434 + 3 files changed, 436 insertions(+) create mode 100644

[Piglit] [PATCH 08/13] arb_shader_atomic_counters: Test that atomic counters can be passed as function arguments.

2013-10-07 Thread Francisco Jerez
--- tests/all.tests| 1 + .../arb_shader_atomic_counters/CMakeLists.gl.txt | 1 + .../arb_shader_atomic_counters/function-argument.c | 114 + 3 files changed, 116 insertions(+) create mode 100644

[Piglit] [PATCH 10/13] arb_shader_atomic_counters: Test the minimum maximum limits defined by the spec.

2013-10-07 Thread Francisco Jerez
--- tests/all.tests| 1 + .../arb_shader_atomic_counters/CMakeLists.gl.txt | 1 + tests/spec/arb_shader_atomic_counters/minmax.c | 71 ++ 3 files changed, 73 insertions(+) create mode 100644

[Piglit] [PATCH 05/13] arb_shader_atomic_counters: Test the buffer binding API functions.

2013-10-07 Thread Francisco Jerez
--- tests/all.tests| 1 + .../arb_shader_atomic_counters/CMakeLists.gl.txt | 1 + .../arb_shader_atomic_counters/buffer-binding.c| 159 + 3 files changed, 161 insertions(+) create mode 100644

[Piglit] [PATCH 07/13] arb_shader_atomic_counters: Test that atomic ops aren't executed for discarded fragments.

2013-10-07 Thread Francisco Jerez
--- tests/all.tests| 1 + .../arb_shader_atomic_counters/CMakeLists.gl.txt | 1 + .../arb_shader_atomic_counters/fragment-discard.c | 113 + 3 files changed, 115 insertions(+) create mode 100644

[Piglit] [PATCH 11/13] arb_shader_atomic_counters: Test linkage of atomic counters with conflicting locations.

2013-10-07 Thread Francisco Jerez
--- tests/all.tests| 1 + .../arb_shader_atomic_counters/CMakeLists.gl.txt | 1 + .../arb_shader_atomic_counters/multiple-defs.c | 213 + 3 files changed, 215 insertions(+) create mode 100644

[Piglit] [PATCH 12/13] arb_shader_atomic_counters: Test that atomic built-ins execute the expected operations on memory.

2013-10-07 Thread Francisco Jerez
--- tests/all.tests| 1 + .../arb_shader_atomic_counters/CMakeLists.gl.txt | 1 + tests/spec/arb_shader_atomic_counters/semantics.c | 383 + 3 files changed, 385 insertions(+) create mode 100644