[Piglit] [PATCH] Add an array test for KHR_texture_compression_astc_ldr

2015-08-31 Thread Nanley Chery
From: Nanley Chery These tests check that 2D texture arrays work for the 5x5 and 12x12 block sizes. Signed-off-by: Nanley Chery --- .../khr_texture_compression_astc/CMakeLists.gl.txt | 1 + .../compressed/hdr/array/waffles-12x12.ktx

[Piglit] [PATCH] arb_shader_texture_image_samples: add tests for all variants

2015-08-31 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin --- .../compiler/fs-image-samples.frag | 30 ++ .../compiler/fs-texture-samples.frag | 29 + .../compiler/vs-texture-samples.vert | 29 + 3

Re: [Piglit] [PATCH v2 2/4] util: Add a piglit_probe_rects_equal_rgba() function

2015-08-31 Thread Chad Versace
On Fri 28 Aug 2015, Nanley Chery wrote: > From: Nanley Chery > > This function compares two rectangles for equality. This is useful > to compare the rendering of individual miplevels in a miptree while > avoiding too much resource consumption. > > Signed-off-by: Nanley

[Piglit] [PATCH] arb_copy_image: fix a few error check tests

2015-08-31 Thread Brian Paul
Some of the error checks were incorrect before. Per the spec: 1. GL_TEXTURE_BUFFER and GL_TEXTURE_CUBE_MAP_+/-_XYZ are not legal targets and should be flagged as invalid enums. 2. GL_INVALID_OPERATION should be generated when trying to copy between compressed/uncompressed formats whose

[Piglit] [PATCH] ext_texture_integer: add new tex formats test

2015-08-31 Thread Brian Paul
Test glTexImage2D() and glGetTexImage() with a variety of internalFormats and user formats/types. Note: currently fails with Mesa, passes with NVIDIA. --- tests/all.py | 1 + tests/spec/ext_texture_integer/CMakeLists.gl.txt | 1 +

[Piglit] [PATCH] arb_copy_image: add new format swizzle test

2015-08-31 Thread Brian Paul
Check that copies between textures whose formats may only differ by swizzling works correctly. --- tests/all.py| 1 + tests/spec/arb_copy_image/CMakeLists.gl.txt | 1 + tests/spec/arb_copy_image/format-swizzle.c | 229 3 files

Re: [Piglit] [PATCH] arb_copy_image: add new format swizzle test

2015-08-31 Thread Ilia Mirkin
Reviewed-by: Ilia Mirkin Thanks a lot for doing this! I assume that it fails with your gallium ARB_copy_image implementation as I predicted? On Mon, Aug 31, 2015 at 7:22 PM, Brian Paul wrote: > Check that copies between textures whose formats may only

Re: [Piglit] [PATCH] ext_texture_integer: add new tex formats test

2015-08-31 Thread Ilia Mirkin
On Mon, Aug 31, 2015 at 7:21 PM, Brian Paul wrote: > Test glTexImage2D() and glGetTexImage() with a variety of internalFormats > and user formats/types. > > Note: currently fails with Mesa, passes with NVIDIA. > --- > tests/all.py | 1 + >

Re: [Piglit] [PATCH] arb_copy_image: add new format swizzle test

2015-08-31 Thread Ilia Mirkin
On Mon, Aug 31, 2015 at 7:29 PM, Ilia Mirkin wrote: > Reviewed-by: Ilia Mirkin > > Thanks a lot for doing this! I assume that it fails with your gallium > ARB_copy_image implementation as I predicted? Oh, and same comment as on the other test -- this

Re: [Piglit] [PATCH] ext_texture_integer: add new tex formats test

2015-08-31 Thread Dave Airlie
On 1 September 2015 at 09:21, Brian Paul wrote: > Test glTexImage2D() and glGetTexImage() with a variety of internalFormats > and user formats/types. > > Note: currently fails with Mesa, passes with NVIDIA. As requested this passes on mesa with my format patches applied.

[Piglit] [PATCH] abr_gpu_shader_fp64: Verify that setting a double with glUniform*fv is an error

2015-08-31 Thread Ian Romanick
From: Ian Romanick Also verify that setting a float with glUniform*dv generates an error. NOTE: Mesa currently fails the matrix tests. Signed-off-by: Ian Romanick Cc: Dave Airlie --- I will have a patch out on the

Re: [Piglit] [PATCH] abr_gpu_shader_fp64: Verify that setting a double with glUniform*fv is an error

2015-08-31 Thread Ilia Mirkin
On Mon, Aug 31, 2015 at 11:55 PM, Ian Romanick wrote: > From: Ian Romanick > > Also verify that setting a float with glUniform*dv generates an error. > > NOTE: Mesa currently fails the matrix tests. > > Signed-off-by: Ian Romanick