Re: [Piglit] [PATCH] arb_texture_cube_map_array: scale texcoords

2013-09-13 Thread Chris Forbes
I'll have patches up to fix it shortly. Sorry for leaving it broken so long -- I hadn't observed hangs. -- Chris On Sat, Sep 14, 2013 at 2:54 PM, Paul Berry wrote: > On 9 September 2013 11:29, Paul Berry wrote: >> >> On 8 September 2013 02:10, Chris Forbes wrote: >>> >>> The magnitude of the

Re: [Piglit] [PATCH] arb_texture_cube_map_array: scale texcoords

2013-09-13 Thread Paul Berry
On 9 September 2013 11:29, Paul Berry wrote: > On 8 September 2013 02:10, Chris Forbes wrote: > >> The magnitude of the vector formed by the first three texcoords >> should not affect the result. >> >> This turns out to be a significant edge case in the i965 driver, >> which normalizes the coord

[Piglit] [PATCH 1/2] status.py: Adds a new status class and subclasses

2013-09-13 Thread Dylan Baker
These status classes are intended to replace the string statuses currently used by piglit. They give a couple of very nice advantages, first, they can be directly compared to each other to determine whether a status is worse than another status. These statuses can also be compared to any object im

[Piglit] [PATCH 2/2] satus: Make use of the status objects

2013-09-13 Thread Dylan Baker
This adds code in framework/summary.py and framework/core.py to make use of the status classes in status.py. This makes comparisons between statuses much simpler and cleaner Signed-off-by: Dylan Baker --- framework/core.py| 18 ++- framework/summary.py | 85 +-

Re: [Piglit] [PATCH 3/3 V2] GL 3.2: Test layered framebuffers clear the depth attachment properly.

2013-09-13 Thread Paul Berry
On 10 September 2013 17:47, Chad Versace wrote: > On 09/10/2013 01:47 PM, Jacob Penner wrote: > >> --- >> tests/all.tests| 1 + >> .../gl-3.2/layered-rendering/**CMakeLists.gl.txt | 1 + >> tests/spec/gl-3.2/layered-**rendering/clear-depth.c | 224 >

Re: [Piglit] [PATCH] Test that GetProgramiv() now accepts GEOMETRY_INPUT_TYPE, GEOMETRY_OUTPUT_TYPE and GEOMETRY_VERTICES_OUT

2013-09-13 Thread Paul Berry
On 9 September 2013 16:00, Nicholas Mack wrote: > **NOTE**: At the time this test was written, the new enums, > GEOMETRY_INPUT_TYPE, > GEOMETRY_OUTPUT_TYPE and GEOMETRY_VERTICES_OUT, do not seem to be > implemented, > thus causing the tests to fail. > --- > tests/all.tests

[Piglit] [PATCH] builtin_functions.py: tweak integer constants

2013-09-13 Thread Brian Paul
This fixes some misleading fs-op-div-* failures when the driver does integer arithmetic with floating point. When we compute the reciprocal of some integers we wind up with inexact floating point results. So expressions like X/X result in 0. or such. This patch tweaks the generator code to u

[Piglit] [PATCH] ext_texture_array/compressed: check for GL_EXT_texture_array

2013-09-13 Thread Brian Paul
We obviously should require this extension. --- tests/spec/ext_texture_array/compressed.c |1 + 1 file changed, 1 insertion(+) diff --git a/tests/spec/ext_texture_array/compressed.c b/tests/spec/ext_texture_array/compressed.c index c964fd7..7f87b62 100644 --- a/tests/spec/ext_texture_array/c

[Piglit] [PATCH] arb_internalformat_query-minmax: add more version/ext checking

2013-09-13 Thread Brian Paul
The valid_formats[] list includes formats supported either by GL 3.0 or GL_EXT_texture_float and GL_EXT_texture_rg. --- tests/spec/arb_internalformat_query/minmax.c |6 ++ 1 file changed, 6 insertions(+) diff --git a/tests/spec/arb_internalformat_query/minmax.c b/tests/spec/arb_internalf

[Piglit] genmipmap-errors test

2013-09-13 Thread Brian Paul
Anuj, Your genmipmap-errors test checks if glGenerateMipmaps raises GL_INVALID_OPERATION for integer and depth/stencil texture formats. Can you tell me which version of the GL spec or extension describes this? genmipmap-errors fails with NVIDIA's OpenGL 4.2 since it does not raise any error