Re: [Piglit] [PATCH] cl: From API clRetain* tests, removed attempt to release already released and destroyed objects

2013-08-22 Thread Ville Korhonen
Problem with releasing already released objects originates from ICD Loader, which requires that object is alive when calling OpenCL host functions. Khronos ICD Loader specs (http://www.khronos.org/registry/cl/extensions/khr/cl_khr_icd.txt) says: At every OpenCL function call, the ICD

Re: [Piglit] [PATCH] glsl-1.10 / glsl-1.20: Relax precision requirements of variable indexing tests

2013-08-22 Thread Brian Paul
On 08/21/2013 09:44 PM, Ian Romanick wrote: From: Ian Romanick ian.d.roman...@intel.com Many of these tests fail on older GPU such as Intel i915 due to the precision requirement. This is because many GPUs in this class only use 24-bit floats. Many also implement many instructions cheaply

[Piglit] [PATCH V2] ARB_seamless_cube_map: test that sample from three faces returns value without border color

2013-08-22 Thread Steve Miller
add new test to existing arb_seamless_cube_map group: sample three adjoining faces and use average. if three faces are the same color, that color must be used. add test to makelists and all.tests V2 : test now uses contemporary pipeline --- tests/all.tests

Re: [Piglit] [PATCH 1/5] GL 3.2: Test functionality of ProvokingVertex()

2013-08-22 Thread Paul Berry
On 20 August 2013 10:56, Nicholas Mack nichm...@gmail.com wrote: --- tests/spec/CMakeLists.txt | 1 + tests/spec/arb_provoking_vertex/CMakeLists.gl.txt | 14 tests/spec/arb_provoking_vertex/CMakeLists.txt | 1 + .../provoking-vertex-control.c

Re: [Piglit] [PATCH 3/5] GL 3.2: Test the validity of QUADS_FOLLOW_PROVOKING_VERTEX_CONVENTION

2013-08-22 Thread Paul Berry
On 20 August 2013 10:56, Nicholas Mack nichm...@gmail.com wrote: Note: The spec refers to QUADS_FOLLOW_PROVOKING_VERTEX but this is a spec bug. --- tests/spec/arb_provoking_vertex/CMakeLists.gl.txt | 1 + .../quads-follow-provoking-vertex.c| 64 ++ 2

Re: [Piglit] [PATCH 4/5] GL 3.2: Test that the Transform Feedback Buffer collects data before flatshading occurs

2013-08-22 Thread Paul Berry
On 20 August 2013 10:56, Nicholas Mack nichm...@gmail.com wrote: --- tests/spec/arb_provoking_vertex/CMakeLists.gl.txt | 1 + .../arb_provoking_vertex/xfb-before-flatshading.c | 203 + 2 files changed, 204 insertions(+) create mode 100644

Re: [Piglit] [PATCH 2/5] GL 3.2: Test that the initial value of provoking vertex mode is LAST_VERTEX_CONVENTION

2013-08-22 Thread Paul Berry
On 20 August 2013 10:56, Nicholas Mack nichm...@gmail.com wrote: --- tests/spec/arb_provoking_vertex/CMakeLists.gl.txt | 1 + .../provoking-vertex-initial.c | 64 ++ 2 files changed, 65 insertions(+) create mode 100644

Re: [Piglit] [PATCH 2/2] Use utility function piglit_srgb_to_linear()

2013-08-22 Thread Paul Berry
On 20 August 2013 16:04, Anuj Phogat anuj.pho...@gmail.com wrote: Signed-off-by: Anuj Phogat anuj.pho...@gmail.com --- tests/spec/ext_framebuffer_multisample/common.cpp | 18 ++ 1 file changed, 2 insertions(+), 16 deletions(-) Series is: Reviewed-by: Paul Berry

[Piglit] [PATCH 1/8] arb_gpu_shader5: Add frexp fragment shader execution test.

2013-08-22 Thread Matt Turner
--- .../built-in-functions/fs-frexp.shader_test| 64 ++ 1 file changed, 64 insertions(+) create mode 100644 tests/spec/arb_gpu_shader5/execution/built-in-functions/fs-frexp.shader_test diff --git

[Piglit] [PATCH 2/8] arb_gpu_shader5: Add frexp vertex shader execution test.

2013-08-22 Thread Matt Turner
--- .../built-in-functions/vs-frexp.shader_test| 68 ++ 1 file changed, 68 insertions(+) create mode 100644 tests/spec/arb_gpu_shader5/execution/built-in-functions/vs-frexp.shader_test diff --git

[Piglit] [PATCH 3/8] arb_gpu_shader5: Add ldexp fragment shader execution test.

2013-08-22 Thread Matt Turner
--- .../built-in-functions/fs-ldexp.shader_test| 88 ++ 1 file changed, 88 insertions(+) create mode 100644 tests/spec/arb_gpu_shader5/execution/built-in-functions/fs-ldexp.shader_test diff --git

[Piglit] [PATCH 4/8] arb_gpu_shader5: Add ldexp vertex shader execution test.

2013-08-22 Thread Matt Turner
--- .../built-in-functions/vs-ldexp.shader_test| 93 ++ 1 file changed, 93 insertions(+) create mode 100644 tests/spec/arb_gpu_shader5/execution/built-in-functions/vs-ldexp.shader_test diff --git

[Piglit] [PATCH 7/8] arb_gpu_shader5: Add fma vertex shader execution test.

2013-08-22 Thread Matt Turner
--- .../built-in-functions/vs-fma.shader_test | 62 ++ 1 file changed, 62 insertions(+) create mode 100644 tests/spec/arb_gpu_shader5/execution/built-in-functions/vs-fma.shader_test diff --git

[Piglit] [PATCH 6/8] arb_gpu_shader5: Add fma fragment shader execution test.

2013-08-22 Thread Matt Turner
--- .../built-in-functions/fs-fma.shader_test | 58 ++ 1 file changed, 58 insertions(+) create mode 100644 tests/spec/arb_gpu_shader5/execution/built-in-functions/fs-fma.shader_test diff --git

[Piglit] [PATCH 8/8] arb_gpu_shader5: Add fma constant eval execution test.

2013-08-22 Thread Matt Turner
--- .../built-in-functions/const-fma.shader_test | 38 ++ 1 file changed, 38 insertions(+) create mode 100644 tests/spec/arb_gpu_shader5/execution/built-in-functions/const-fma.shader_test diff --git

Re: [Piglit] [PATCH 0/4] Minor Improvements

2013-08-22 Thread Ken Phillis Jr
I probably should mention that the first two patches of the four patches is meant to fix some problems with compatibility fall back... The problem is caused by the test suite note resetting the variable that stores the list of supported extensions after the old context is destroyed. There is a

[Piglit] [PATCH] glsl-1.50: Remove gl_Position from fragment shader.

2013-08-22 Thread Kenneth Graunke
This must've been cut and pasted from a vertex shader by accident. Signed-off-by: Kenneth Graunke kenn...@whitecape.org Cc: Nicholas Mack nichm...@gmail.com --- tests/spec/glsl-1.50/compiler/layout-global-only.frag | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git