Re: [Piglit] [PATCH] util: fix broken depth/stencil FBO setup

2012-06-15 Thread Jose Fonseca
- Original Message - The call to glTexImage2D(internalFormat=GL_DEPTH_STENCIL, format=GL_RGBA, type=GL_UNSIGNED_BYTE) generated in invalid operation error since the formats disagreed. The subsquent glFramebufferTexture2D() calls failed too. And Mesa hit a failed assertion (ex:

[Piglit] [PATCH 00/16] msaa: thoroughly test different formats.

2012-06-15 Thread Paul Berry
This patch series tests that MSAA works properly across a variety of framebuffer formats, in particular: integer formats, floating point formats, RED and RG formats, and sRGB formats. Patches 01-03 lay the necessary infrastructure groundwork, by refactoring piglit-util-gl.c and

[Piglit] [PATCH 01/16] util: Make a separate function to compare in-memory images.

2012-06-15 Thread Paul Berry
This patch splits off a new piglit-util function, piglit_compare_images_color(), from the existing piglit_probe_image_color() function. The new function compares two images in memory, so that the caller can take responsibility for calling glReadPixels(). In addition, this patch creates a

[Piglit] [PATCH 03/16] msaa: Make it possible to scale and offset ColorGradientSunburst.

2012-06-15 Thread Paul Berry
In order to test that MSAA works properly for integer framebuffers, we will need to be able to adjust the outputs of the ColorGradientSunburst program to cover the range of signed or unsigned integers, rather than the range [0, 1] that is appropriate for testing normalized color framebuffers.

[Piglit] [PATCH 04/16] msaa/formats: Adapt test pattern drawing to handle integer formats.

2012-06-15 Thread Paul Berry
This patch adapts the MSAA formats test so that it can draw to integer framebuffers, by creating 3 instantiations of the ColorGradientSunburst class, one for testing vec4's (for floating point and normalized formats), one for testing ivec4's (for signed integer formats), and one for testing

[Piglit] [PATCH 07/16] msaa/formats: Use a scale and offset for testing integer formats.

2012-06-15 Thread Paul Berry
This patch adapts the MSAA formats test so that when drawing into integer framebuffers, it applies the necessary scale and offset to cover the entire range of possible integer values. When reading pixels back from the framebuffer, the test applies the inverse scale and offset, so that the result

[Piglit] [PATCH 09/16] msaa/formats: Add EXT_texture_integer tests.

2012-06-15 Thread Paul Berry
The MSAA formats test now properly tests integer framebuffer formats. This patch updates all.tests to cause integer framebuffer formats to be tested. --- tests/all.tests |9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/tests/all.tests b/tests/all.tests index

[Piglit] [PATCH 08/16] msaa/formats: Make it possible to specify a test set on the command line.

2012-06-15 Thread Paul Berry
--- tests/spec/ext_framebuffer_multisample/formats.cpp | 13 ++--- 1 files changed, 10 insertions(+), 3 deletions(-) diff --git a/tests/spec/ext_framebuffer_multisample/formats.cpp b/tests/spec/ext_framebuffer_multisample/formats.cpp index d2386ad..a55f053 100644 ---

[Piglit] [PATCH 12/16] msaa/formats: Fix visualization of RED and RG formats

2012-06-15 Thread Paul Berry
The switch statement in visualize_image() wasn't properly handling formats whose base internal format is GL_RED or GL_RG. As a result, tests of these formats showed up as a black screen. This patch addes switch cases to handle the formats properly. ---

[Piglit] [PATCH 10/16] msaa/formats: Adjust tolerance appropriately for floating-point formats.

2012-06-15 Thread Paul Berry
When setting up a floating-point format, the GL implementation responds to queries GL_ATTACHMENT_{RED,GREEN,BLUE,ALPHA}_SIZE with the total number of bits in the floating-point value, regardless of whether those bits are used for sign, exponent, or mantissa. However, for the purpose of setting a

[Piglit] [PATCH 13/16] msaa/formats: Add ARB_texture_rg tests.

2012-06-15 Thread Paul Berry
The MSAA formats test now properly tests RED and RG format framebuffers. This patch updates all.tests to cause those formats to be tested. --- tests/all.tests |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/tests/all.tests b/tests/all.tests index a01ba4a..2a636f3

[Piglit] [PATCH 11/16] msaa/formats: Add floating point format tests.

2012-06-15 Thread Paul Berry
The MSAA formats test now properly tests floating point formats. This patch updates all.tests to cause those formats to be tested. --- tests/all.tests |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/tests/all.tests b/tests/all.tests index 3f4ab26..a01ba4a 100644 ---

[Piglit] [PATCH 14/16] msaa: Verify accuracy of sRGB MSAA resolves.

2012-06-15 Thread Paul Berry
From the GL spec, version 4.2, section 4.1.11 (Additional Multisample Fragment Operations): If a framebuffer object is not bound, after all operations have been completed on the multisample buffer, the sample values for each color in the multisample buffer are combined to produce a

[Piglit] [PATCH 16/16] msaa/formats: Add EXT_texture_sRGB tests.

2012-06-15 Thread Paul Berry
The MSAA formats test now properly tests sRGB format framebuffers. This patch updates all.tests to cause those formats to be tested. --- tests/all.tests |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/tests/all.tests b/tests/all.tests index 755eace..2102faa 100644 ---

[Piglit] [PATCH 15/16] msaa/formats: Fix testing of sRGB formats.

2012-06-15 Thread Paul Berry
sRGB buffers use a different blending operation to do multisample resolves from non-sRGB buffers. Accordingly, when testing sRGB formats, we need to use an sRGB reference image. This patch modifies the MSAA formats test so that when testing an sRGB format, it renders the reference image using

[Piglit] [PATCH 05/16] msaa/formats: Adapt glReadPixels() call to handle integer formats.

2012-06-15 Thread Paul Berry
This patch adapts the MSAA formats test to pass the proper parameters to glReadPixels() when checking that the proper image has been rendered. Since glReadPixels() requires an integral type when reading pixels from an integer framebuffer, we read the pixels into a temporary integer buffer and

[Piglit] [PATCH 02/16] msaa: Make it possible to change output type when drawing sunbursts.

2012-06-15 Thread Paul Berry
In order to test that MSAA works properly for integer framebuffers, we will need to be able to output ivec4 and uvec4 types when rendering the test image. This patch makes that possible by adapting the Sunburst GLSL program so that it can be compiled to output ivec4's, uvec4's, or vec4's. ---

[Piglit] [PATCH] Add a test to check whether the LODs are independant in parallel texture lookups.

2012-06-15 Thread Olivier Galibert
Theorically, we should also test that as the fragment shader level, but the code would be way more complex and the code paths are shared in mesa. That was designed to fix the issue in llvmpipe (which also exists in textureSize). NVidia passes. Surprisingly enough, softpipe fails.

Re: [Piglit] [PATCH 00/14] Move main() from libpiglitutil into each test

2012-06-15 Thread Chad Versace
On 06/15/2012 10:27 AM, Brian Paul wrote: On Fri, Jun 15, 2012 at 11:18 AM, Chad Versace chad.vers...@linux.intel.com wrote: On 06/13/2012 10:25 AM, Chad Versace wrote: On 06/13/2012 09:06 AM, Brian Paul wrote: On 06/12/2012 05:02 PM, Chad Versace wrote: Could you check this into a branch