[Piglit] [PATCH 5/7] pack_depth_stencil: Create textures with GL_DEPTH_STENCIL base format
Pohjolainen, Topi
topi.pohjolainen at intel.com
Sat Dec 21 03:24:28 PST 2013
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?
> + * 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