[Piglit] [PATCH] Change INVALID_VALUE to INVALID_OPERATION in ARB_vertex_array_bgra test.

2013-10-11 Thread Matt Turner
From: Kenneth Graunke In the OpenGL 3.3 specification, this changed from INVALID_VALUE to INVALID_OPERATION. According to Ian Romanick, the GL specification provides the guidance that INVALID_VALUE should be generated for "Numeric argument out of range". Since FALSE is in range for a boolean, I

Re: [Piglit] [PATCH] framework: stop copying dma buffer source padding

2013-10-11 Thread Chad Versace
On 10/10/2013 01:15 AM, Topi Pohjolainen wrote: Fixes a bug spotted by Chad. The initial support allowed the stride of the source content in the dma buffer creation to have some padding but it had to be less than the padding in the target dma buffer being created. Moreover the implementation al

Re: [Piglit] [PATCH] gs: Check GL errors in primitive-id-restart test.

2013-10-11 Thread Jordan Justen
Reviewed-by: Jordan Justen On Fri, Oct 11, 2013 at 11:36 AM, Paul Berry wrote: > Previously, if drawing failed due to a GL error, no primitives would > be generated, so the test would have nothing to check, and it would > erroneously pass. Avoid that by checking for GL errors at the end of > th

[Piglit] [PATCH] gs: Check GL errors in primitive-id-restart test.

2013-10-11 Thread Paul Berry
Previously, if drawing failed due to a GL error, no primitives would be generated, so the test would have nothing to check, and it would erroneously pass. Avoid that by checking for GL errors at the end of the test. --- tests/spec/glsl-1.50/execution/geometry/primitive-id-restart.c | 2 ++ 1 file

[Piglit] [PATCH 2/2] glsl-1.50: add size requirements to 3 shader_test tests

2013-10-11 Thread Jordan Justen
Signed-off-by: Jordan Justen --- .../spec/glsl-1.50/execution/geometry/point-size-out.shader_test |5 +++-- .../execution/redeclare-pervertex-out-subset-gs.shader_test |5 +++-- .../glsl-1.50/execution/redeclare-pervertex-subset-vs.shader_test |5 +++-- 3 files changed, 9 inser

[Piglit] [PATCH 1/2] shader_runner: support SIZE in requirement section of a shader_test

2013-10-11 Thread Jordan Justen
If a test requires a certain size, it adds 'SIZE Width Height' to the requirements section. Width and Height must be unsigned integers. Signed-off-by: Jordan Justen --- tests/shaders/shader_runner.c | 40 +++- 1 file changed, 27 insertions(+), 13 deletions(-

Re: [Piglit] [PATCH 1/2] GL 3.2: Test gl_Layer render to layered attachements

2013-10-11 Thread Jordan Justen
Jacob, This and your other 3 patch gl_Layer series are: Reviewed-by: Jordan Justen and applied. Let me know if I missed any of your remaining patches. (I added 8 total patches of yours to the tree.) Thanks, -Jordan On Thu, Sep 26, 2013 at 1:46 PM, Jacob Penner wrote: > --- > tests/all.tests

[Piglit] [PATCH] cl: Add test opencv-merge-hist

2013-10-11 Thread Tom Stellard
From: Tom Stellard --- tests/cl/program/execute/opencv-merge-hist.cl | 97 +++ 1 file changed, 97 insertions(+) create mode 100644 tests/cl/program/execute/opencv-merge-hist.cl diff --git a/tests/cl/program/execute/opencv-merge-hist.cl b/tests/cl/program/execute/opencv