Re: [Piglit] [PATCH] arb_gpu_shader5: add compiler tests for interpolateAt*

2013-11-11 Thread Anuj Phogat
On Sun, Nov 10, 2013 at 1:09 AM, Chris Forbes wrote: > > --- > .../fs-interpolateAtCentroid-array.frag| 25 ++ > .../fs-interpolateAtCentroid-bad-swizzle.frag | 27 +++ > .../fs-interpolateAtCentroid-centroid.frag | 30 > ++

[Piglit] [PATCH] multi-draw-elements-base-vertex: Require GLSL 1.30.

2013-11-11 Thread Vinson Lee
Signed-off-by: Vinson Lee --- .../arb_draw_elements_base_vertex/multi-draw-elements-base-vertex.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/spec/arb_draw_elements_base_vertex/multi-draw-elements-base-vertex.c b/tests/spec/arb_draw_elements_base_vertex/multi-draw-elements-b

Re: [Piglit] [PATCH] CL: Fix memory leak in parse_name

2013-11-11 Thread Tom Stellard
On Mon, Nov 11, 2013 at 08:43:56AM -0600, Aaron Watry wrote: > We need to free a regex after compiling/executing it. Reviewed-by: Tom Stellard > --- > tests/cl/program/program-tester.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/tests/cl/program/program-tester.c > b/tests/cl/pr

Re: [Piglit] [PATCH] Check that transform feedback size/type comes from gs, not vs.

2013-11-11 Thread Paul Berry
On 6 November 2013 14:21, Ian Romanick wrote: > On 10/23/2013 01:01 PM, Paul Berry wrote: > > When the current program has both a geometry shader and a vertex > > shader, transform feedback needs to link with the geometry shader. > > This test verifies that the type and size of the varyings captu

Re: [Piglit] Fix subtests

2013-11-11 Thread Dylan Baker
forgot to mention this is available from my github: https://github.com/crymson/piglit.git simplify_subtests On Tuesday, November 05, 2013 10:34:49 AM Dylan Baker wrote: > This series attempts to fix the way piglit handles subtests. In > particular there are two problems with subtests: > 1. A sub

Re: [Piglit] [PATCH] gs: Test redeclaring either gl_PerVertex in or out, and using both.

2013-11-11 Thread Paul Berry
On 6 November 2013 14:06, Ian Romanick wrote: > On 10/15/2013 03:53 PM, Paul Berry wrote: > > This test is > > Reviewed-by: Ian Romanick > > Do we already have tests that redeclare these blocks but try to use > fields that were not in the redeclaration? > Yes: - gs-redeclares-pervertex-in-befor

[Piglit] [PATCH] Fix improperly-named geometry shader tests.

2013-11-11 Thread Paul Berry
Several geometry shader tests were improperly added to all.tests with names like "spec/glsl-1.50/glsl-1.50-geometry-foo" instead of "spec/glsl-1.50/execution/geometry/foo". This patch renames the tests to follow our usual piglit conventions. --- tests/all.tests | 25 ++--- 1 f

Re: [Piglit] [PATCH 2/2] query_renderer: Add test that compares values from glXQueryRendererIntegerMESA et al

2013-11-11 Thread Paul Berry
On 25 October 2013 11:13, Ian Romanick wrote: > From: Ian Romanick > > Verifies that all values from glXQueryRendererIntegerMESA, > glXQueryCurrentRendererIntegerMESA, glXQueryRendererStringMESA, and > glXQueryCurrentRendererStringMESA can be queried. It also verifies that > each of the Current

Re: [Piglit] [PATCH 1/2] query_renderer: Add some common infrastructure for GLX_MESA_query_renderer tests

2013-11-11 Thread Paul Berry
On 25 October 2013 11:13, Ian Romanick wrote: > From: Ian Romanick > > All of the tests for this extension will require a bunch of function > pointers, etc. This patch adds a function to get these function > pointers and some macros to wrap calls using them. > > Signed-off-by: Ian Romanick > -

Re: [Piglit] [PATCH] gl-3.1/minmax: require at least 0 in GL_NUM_COMPRESSED_TEXTURE_FORMATS

2013-11-11 Thread Ian Romanick
On 11/01/2013 04:56 PM, Marek Olšák wrote: > The GL 4.4 spec counts the RGTC formats, so it looks like the ARB > reverted the change from 4.2. NUM_COMPRESSED_TEXTURE_FORMATS should > return at least 18 and the list of required formats is here: > > COMPRESSED_RED_RGTC1 > COMPRESSED_SIGNED_RED_RGTC1

[Piglit] [PATCH 2/2] igt: Enable dmesg checking by default

2013-11-11 Thread Daniel Vetter
An awful lot of tests rely on in-kernel checks to detect issues. So we better make them fail properly. Note that the dmesg check relies on dmesg timestamps to work correctly. If you don't have them it won't work. Cc: Paulo Zanoni --- tests/igt.tests | 3 +++ 1 file changed, 3 insertions(+) dif

[Piglit] [PATCH 1/2] framework: Ignore info/debug dmesg output

2013-11-11 Thread Daniel Vetter
i-g-t has a bunch of testcase that cause informational output. In addition I usually enable drm debugging on my machines, so the kms tests cause massive amounts of spew. I've thought whether an option would be useful, but then I didn't really see any reason why we should ever care about debug/info

[Piglit] [PATCH] GL: Verify that when GL_COORD_REPLACE is set, fragment shader texture coordinates do not get eliminated.

2013-11-11 Thread Nicholas Mack
--- tests/all.tests| 2 +- tests/spec/gl-3.2/CMakeLists.gl.txt| 3 +- ...oord-replace-doesnt-eliminate-frag-tex-coords.c | 135 + 3 files changed, 138 insertions(+), 2 deletions(-) create mode 100644 tests/spec/gl-3.2/gl

Re: [Piglit] [PATCH] ARB_texture_view: Add multisample texture target testing

2013-11-11 Thread Brian Paul
On 11/08/2013 01:40 PM, Jon Ashburn wrote: The "targets" test is extended to include testing of GL_TEXTURE_2D_MULTISAMPLE and GL_TEXTURE_2D_MULTISAMPLE_ARRAY. This will test making valid and invalid texture views of these two multisample targets and ensuring the correct GL error is set. --- te

[Piglit] [PATCH] CL: Fix memory leak in parse_name

2013-11-11 Thread Aaron Watry
We need to free a regex after compiling/executing it. --- tests/cl/program/program-tester.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/cl/program/program-tester.c b/tests/cl/program/program-tester.c index be6bc20..31aeec9 100644 --- a/tests/cl/program/program-tester.c +++ b/tests