[Piglit] [PATCH 5/7] pack_depth_stencil: Create textures with GL_DEPTH_STENCIL base format
Ian Romanick
idr at freedesktop.org
Mon Dec 23 12:30:46 PST 2013
On 12/21/2013 03:24 AM, Pohjolainen, Topi wrote:
> On Thu, Dec 19, 2013 at 11:37:33AM -0800, Ian Romanick wrote:
>> From: Ian Romanick <ian.d.romanick at intel.com>
>>
>> Depending on the GL version and extensions supported, this should either
>> work (not generate a GL error) or generate a variety of GL errors.
>>
>> This test passes on NVIDIA (304.64 on GTX 260). This driver exposes
>> OpenGL 3.3 and GL_ARB_texture_storage.
>>
>> This test fails on current Mesa on drivers that support depth textures.
>> the glTexImage3D(GL_TEXTURE_3D) tests generate GL_INVALID_ENUM instead
>> of GL_INVALID_OPERATION. The glTexStorage3D(GL_TEXTURE_3D) test
>> doesn't generate an error, but it should generate GL_INVALID_OPERATION.
>>
>> Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
>> ---
>> tests/all.tests | 1 +
>> .../ext_packed_depth_stencil/CMakeLists.gl.txt | 1 +
>> .../depth-stencil-texture.c | 245 +++++++++++++++++++++
>> 3 files changed, 247 insertions(+)
>> create mode 100644 tests/spec/ext_packed_depth_stencil/depth-stencil-texture.c
>>
>> diff --git a/tests/all.tests b/tests/all.tests
>> index 091a2e5..fa25c6c 100644
>> --- a/tests/all.tests
>> +++ b/tests/all.tests
>> @@ -2136,6 +2136,7 @@ add_depthstencil_render_miplevels_tests(
>> 's=z24_s8_d=z24_s8', 's=z24_s8_d=z24', 'd=s=z24_s8', 's=d=z24_s8',
>> 'ds=z24_s8'))
>> ext_packed_depth_stencil['fbo-clear-formats stencil'] = concurrent_test('fbo-clear-formats GL_EXT_packed_depth_stencil stencil')
>> +ext_packed_depth_stencil['DEPTH_STENCIL texture'] = concurrent_test('ext_packed_depth_stencil-depth-stencil-texture')
>>
>> ext_texture_array = Group()
>> spec['EXT_texture_array'] = ext_texture_array
>> diff --git a/tests/spec/ext_packed_depth_stencil/CMakeLists.gl.txt b/tests/spec/ext_packed_depth_stencil/CMakeLists.gl.txt
>> index 22d6f34..2f47328 100644
>> --- a/tests/spec/ext_packed_depth_stencil/CMakeLists.gl.txt
>> +++ b/tests/spec/ext_packed_depth_stencil/CMakeLists.gl.txt
>> @@ -9,6 +9,7 @@ link_libraries (
>> ${OPENGL_glu_LIBRARY}
>> )
>>
>> +piglit_add_executable (ext_packed_depth_stencil-depth-stencil-texture depth-stencil-texture.c)
>> piglit_add_executable (ext_packed_depth_stencil-readpixels-24_8 readpixels-24_8.c)
>>
>> # vim: ft=cmake:
>> diff --git a/tests/spec/ext_packed_depth_stencil/depth-stencil-texture.c b/tests/spec/ext_packed_depth_stencil/depth-stencil-texture.c
>> new file mode 100644
>> index 0000000..c641772
>> --- /dev/null
>> +++ b/tests/spec/ext_packed_depth_stencil/depth-stencil-texture.c
>> @@ -0,0 +1,245 @@
>> +/*
>> + * Copyright © 2011 Intel Corporation
>> + *
>> + * Permission is hereby granted, free of charge, to any person obtaining a
>> + * copy of this software and associated documentation files (the "Software"),
>> + * to deal in the Software without restriction, including without limitation
>> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
>> + * and/or sell copies of the Software, and to permit persons to whom the
>> + * Software is furnished to do so, subject to the following conditions:
>> + *
>> + * The above copyright notice and this permission notice (including the next
>> + * paragraph) shall be included in all copies or substantial portions of the
>> + * Software.
>> + *
>> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
>> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
>> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
>> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
>> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
>> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
>> + * IN THE SOFTWARE.
>> + */
>> +
>> +/**
>> + * \file negative-texture
>
> Would this need to be 'depth-stencil-texture.c' instead?
Yes... The test started with a different name and goals for the test,
but those changed when I got a little deeper. I'll update the comment.
>> + * Create a depth-stencil texture when ARB_depth_texture is not supported.
>> + *
>> + * The EXT_packed_depth_stencil spec neglects to mention an interaction
>> + * (though the header of the spec says "ARB_depth_texture affects the
>> + * definition of this extension."), but ARB_framebuffer_object, which includes
>> + * EXT_packed_depth_stencil functionality says:
>> + *
>> + * "If ARB_depth_texture or SGIX_depth_texture is supported,
>> + * GL_DEPTH_STENCIL/GL_UNSIGNED_INT_24_8 data can also be used for
>> + * textures;"
>> + *
>> + * In cases where neither ARB_depth_texture nor SGIX_depth_texture is
>> + * supported, trying to create a texture with a depth-stencil format should
>> + * generate an error.
>> + */
>> +
>> +#include "piglit-util-gl-common.h"
>> +
>> +PIGLIT_GL_TEST_CONFIG_BEGIN
>> +
>> + config.supports_gl_compat_version = 12;
>> + config.supports_gl_core_version = 31;
>> +
>> + config.window_visual = PIGLIT_GL_VISUAL_RGBA;
>> +
>> +PIGLIT_GL_TEST_CONFIG_END
>> +
>> +static bool has_texture_cube_map = false;
>> +static bool has_depth_texture_cube_map = false;
>> +static bool has_depth_texture = false;
>> +
>> +static bool
>> +try_TexImage(GLenum internalFormat)
>> +{
>> + bool pass = true;
>> + GLuint tex[4];
>> +
>> + const GLenum expected_error = has_depth_texture
>> + ? GL_NO_ERROR : GL_INVALID_VALUE;
>> +
>> + const GLenum expected_3D_error = has_depth_texture
>> + ? GL_INVALID_OPERATION : GL_INVALID_VALUE;
>> +
>> + const GLenum expected_cube_error = has_depth_texture_cube_map
>> + ? GL_NO_ERROR : expected_3D_error;
>> +
>> + printf("Testing glTexImage with %s...\n",
>> + piglit_get_gl_enum_name(internalFormat));
>> +
>> + glGenTextures(ARRAY_SIZE(tex), tex);
>> +
>> + glBindTexture(GL_TEXTURE_1D, tex[0]);
>> + glTexImage1D(GL_TEXTURE_1D, 0, internalFormat,
>> + 16, 0,
>> + GL_DEPTH_STENCIL, GL_UNSIGNED_INT_24_8, NULL);
>> + pass = piglit_check_gl_error(expected_error) && pass;
>> +
>> +
>> + glBindTexture(GL_TEXTURE_2D, tex[1]);
>> + glTexImage2D(GL_TEXTURE_2D, 0, internalFormat,
>> + 16, 16, 0,
>> + GL_DEPTH_STENCIL, GL_UNSIGNED_INT_24_8, NULL);
>> + pass = piglit_check_gl_error(expected_error) && pass;
>> +
>> +
>> + /* Section 3.8.1 (Texture Image Specification) of the OpenGL 2.1 spec
>> + * says:
>> + *
>> + * "Textures with a base internal format of DEPTH_COMPONENT are
>> + * supported by texture image specification commands only if
>> + * target is TEXTURE_1D, TEXTURE_2D, PROXY_TEXTURE_1D or
>> + * PROXY_TEXTURE_2D. Using this format in conjunction with any
>> + * other target will result in an INVALID_OPERATION error."
>> + *
>> + * The OpenGL 4.4 spec lists the same error, but it greatly expands
>> + * the list of valid texture targets.
>> + */
>> + glBindTexture(GL_TEXTURE_3D, tex[2]);
>> + glTexImage3D(GL_TEXTURE_3D, 0, internalFormat,
>> + 8, 8, 8, 0,
>> + GL_DEPTH_STENCIL, GL_UNSIGNED_INT_24_8, NULL);
>> + pass = piglit_check_gl_error(expected_3D_error) && pass;
>> +
>> + if (has_texture_cube_map) {
>> + unsigned i;
>> +
>> + glBindTexture(GL_TEXTURE_CUBE_MAP, tex[3]);
>> +
>> + for (i = 0; i < 6; i++) {
>> + glTexImage2D(GL_TEXTURE_CUBE_MAP_POSITIVE_X + i,
>> + 0, internalFormat,
>> + 16, 16, 0,
>> + GL_DEPTH_STENCIL, GL_UNSIGNED_INT_24_8,
>> + NULL);
>> + pass = piglit_check_gl_error(expected_cube_error)
>> + && pass;
>> + }
>> + }
>> +
>> + glBindTexture(GL_TEXTURE_1D, 0);
>> + glBindTexture(GL_TEXTURE_2D, 0);
>> + glBindTexture(GL_TEXTURE_3D, 0);
>> + if (has_texture_cube_map)
>> + glBindTexture(GL_TEXTURE_CUBE_MAP, 0);
>> +
>> + glDeleteTextures(ARRAY_SIZE(tex), tex);
>> +
>> + printf("Done.\n\n");
>> +
>> + return pass;
>> +}
>> +
>> +static bool
>> +try_TexStorage(GLenum internalFormat)
>> +{
>> + bool pass = true;
>> + GLuint tex[4];
>> +
>> + GLenum expected_error;
>> + GLenum expected_3D_error;
>> + GLenum expected_cube_error;
>> +
>> + /* The GL_ARB_texture_storage spec says:
>> + *
>> + * "- If <internalformat> is one of the internal formats listed in
>> + * table 3.11, an INVALID_ENUM error is generated."
>> + *
>> + * Table 3.11 lists the unsized formats, including GL_DEPTH_STENCIL.
>> + */
>> + if (internalFormat == GL_DEPTH_STENCIL) {
>> + expected_error = has_depth_texture
>> + ? GL_INVALID_ENUM : GL_INVALID_VALUE;
>> + expected_3D_error = has_depth_texture
>> + ? GL_INVALID_ENUM : GL_INVALID_VALUE;
>> + expected_cube_error = has_depth_texture_cube_map
>> + ? GL_INVALID_ENUM : GL_INVALID_VALUE;
>> + } else {
>> + expected_error = has_depth_texture
>> + ? GL_NO_ERROR : GL_INVALID_VALUE;
>> + expected_3D_error = has_depth_texture
>> + ? GL_INVALID_OPERATION : GL_INVALID_VALUE;
>> + expected_cube_error = has_depth_texture_cube_map
>> + ? GL_NO_ERROR : expected_3D_error;
>> + }
>> +
>> + printf("Testing glTexStorage with %s...\n",
>> + piglit_get_gl_enum_name(internalFormat));
>> +
>> + glGenTextures(ARRAY_SIZE(tex), tex);
>> +
>> + glBindTexture(GL_TEXTURE_1D, tex[0]);
>> + glTexStorage1D(GL_TEXTURE_1D, 1, internalFormat, 16);
>> + pass = piglit_check_gl_error(expected_error) && pass;
>> +
>> + glBindTexture(GL_TEXTURE_2D, tex[1]);
>> + glTexStorage2D(GL_TEXTURE_2D, 1, internalFormat, 16, 16);
>> + pass = piglit_check_gl_error(expected_error) && pass;
>> +
>> + glBindTexture(GL_TEXTURE_3D, tex[2]);
>> + glTexStorage3D(GL_TEXTURE_3D, 1, internalFormat, 8, 8, 8);
>> + pass = piglit_check_gl_error(expected_3D_error) && pass;
>> +
>> + if (has_texture_cube_map) {
>> + glBindTexture(GL_TEXTURE_CUBE_MAP, tex[3]);
>> + glTexStorage2D(GL_TEXTURE_CUBE_MAP, 1, internalFormat, 16, 16);
>> + pass = piglit_check_gl_error(expected_cube_error) && pass;
>> + }
>> +
>> + glBindTexture(GL_TEXTURE_1D, 0);
>> + glBindTexture(GL_TEXTURE_2D, 0);
>> + glBindTexture(GL_TEXTURE_3D, 0);
>> + if (has_texture_cube_map)
>> + glBindTexture(GL_TEXTURE_CUBE_MAP, 0);
>> +
>> + glDeleteTextures(ARRAY_SIZE(tex), tex);
>> +
>> + printf("Done.\n\n");
>> +
>> + return pass;
>> +}
>> +
>> +void piglit_init(int argc, char **argv)
>> +{
>> + bool pass = true;
>> +
>> + if (piglit_get_gl_version() < 30
>> + && !piglit_is_extension_supported("GL_EXT_packed_depth_stencil")
>> + && !piglit_is_extension_supported("GL_ARB_framebuffer_object")) {
>> + printf("OpenGL 3.0, GL_EXT_packed_depth_stencil, or "
>> + "GL_ARB_framebuffer_object is required.\n");
>> + piglit_report_result(PIGLIT_SKIP);
>> + }
>> +
>> + has_depth_texture = piglit_get_gl_version() >= 14
>> + || piglit_is_extension_supported("GL_ARB_depth_texture")
>> + || piglit_is_extension_supported("GL_SGIX_depth_texture");
>> +
>> + has_texture_cube_map = piglit_get_gl_version() >= 13
>> + || piglit_is_extension_supported("GL_ARB_texture_cube_map");
>> +
>> + has_depth_texture_cube_map = piglit_get_gl_version() >= 30
>> + || piglit_is_extension_supported("GL_EXT_gpu_shader4");
>> +
>> + pass = try_TexImage(GL_DEPTH_STENCIL) && pass;
>> + pass = try_TexImage(GL_DEPTH24_STENCIL8) && pass;
>> +
>> + if (piglit_is_extension_supported("GL_ARB_texture_storage")) {
>> + pass = try_TexStorage(GL_DEPTH_STENCIL) && pass;
>> + pass = try_TexStorage(GL_DEPTH24_STENCIL8) && pass;
>> + }
>> +
>> + piglit_report_result(pass ? PIGLIT_PASS : PIGLIT_FAIL);
>> +}
>> +
>> +enum piglit_result
>> +piglit_display(void)
>> +{
>> + /* UNREACHED */
>> + return PIGLIT_FAIL;
>> +}
>> --
>> 1.8.1.4
>>
>> _______________________________________________
>> Piglit mailing list
>> Piglit at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/piglit
>
More information about the Piglit
mailing list