[Mesa-dev] [PATCH] gl-3.0/api/drawpixels-integer: New test for GL 3.0 integer glDrawPixels.

Eric Anholt eric at anholt.net
Sun Nov 27 18:08:04 PST 2011


When the extension was folded in, glDrawPixels of integer formats was
disallowed.  Disable the GL_EXT_texture_integer if 3.0 is exposed, and
replace it with a simple "make sure it throws the error" API test.

I think actually the intent was that this is a correction to
EXT_texture_integer, because I don't see how it was actually supposed
to work back then (see comment in fbo-integer-precision-drawpixels).
---
 tests/all.tests                                    |    1 +
 tests/spec/ext_texture_integer/api-drawpixels.c    |    5 +
 .../fbo-integer-precision-drawpixels.c             |   30 +++++++
 tests/spec/gl-3.0/api/CMakeLists.gl.txt            |    1 +
 tests/spec/gl-3.0/api/drawpixels-integer.c         |   86 ++++++++++++++++++++
 5 files changed, 123 insertions(+), 0 deletions(-)
 create mode 100644 tests/spec/gl-3.0/api/drawpixels-integer.c

diff --git a/tests/all.tests b/tests/all.tests
index bbd31f1..96d2f51 100644
--- a/tests/all.tests
+++ b/tests/all.tests
@@ -800,6 +800,7 @@ add_concurrent_test(gl30, 'clearbuffer-invalid-drawbuffer')
 add_concurrent_test(gl30, 'clearbuffer-invalid-buffer')
 add_concurrent_test(gl30, 'clearbuffer-stencil')
 add_concurrent_test(gl30, 'getfragdatalocation')
+add_concurrent_test(gl30, 'gl-3.0-drawpixels-integer')
 add_concurrent_test(gl30, 'gl-3.0-required-sized-texture-formats')
 add_concurrent_test(gl30, 'gl-3.0-required-renderbuffer-attachment-formats')
 add_concurrent_test(gl30, 'gl-3.0-required-texture-attachment-formats')
diff --git a/tests/spec/ext_texture_integer/api-drawpixels.c b/tests/spec/ext_texture_integer/api-drawpixels.c
index adf311e..eb91718 100644
--- a/tests/spec/ext_texture_integer/api-drawpixels.c
+++ b/tests/spec/ext_texture_integer/api-drawpixels.c
@@ -54,5 +54,10 @@ piglit_display(void)
 void
 piglit_init(int argc, char **argv)
 {
+	if (GLEW_VERSION_3_0) {
+		printf("See tests/spec/gl-3.0/api/drawpixels-integer.c\n");
+		piglit_report_result(PIGLIT_SKIP);
+	}
+
 	piglit_require_extension("GL_EXT_texture_integer");
 }
diff --git a/tests/spec/ext_texture_integer/fbo-integer-precision-drawpixels.c b/tests/spec/ext_texture_integer/fbo-integer-precision-drawpixels.c
index 5a341f7..fe1a0b2 100644
--- a/tests/spec/ext_texture_integer/fbo-integer-precision-drawpixels.c
+++ b/tests/spec/ext_texture_integer/fbo-integer-precision-drawpixels.c
@@ -283,6 +283,36 @@ piglit_init(int argc, char **argv)
 	piglit_require_extension("GL_EXT_texture_integer");
 	piglit_require_GLSL_version(130);
 
+	if (GLEW_VERSION_3_0) {
+		/* The EXT_texture_integer spec doesn't explicitly
+		 * disallow glDrawPixels with integer components, and
+		 * specifies a couple of error cases, e.g. in
+		 * "Rasterization of Pixel Rectangles":
+		 *
+		 *     "If format is one of the integer component
+		 *      formats as defined in table 3.6 and type is
+		 *      FLOAT, the error INVALID ENUM occurs."
+		 *
+		 * while in Section 3.7.4 of the GL 3.0 specification
+		 * (20080923, page 149), a new restriction has been
+		 * added:
+		 *
+		 *     "If format contains integer components, as
+		 *      shown in table 3.6, an INVALID OPERATION error
+		 *      is generated."
+		 *
+		 * The GL 3.0 behavior is tested in
+		 * gl-3.0/api/drawpixels-integer.c.  This restriction
+		 * persists through GL 3.3 compatibility specs and
+		 * presumably into the future.  It makes sense,
+		 * because how is the non-normalized integer value
+		 * supposed to be assigned to the floating-point
+		 * gl_Color input of the fragment shader?
+		 */
+		printf("GL 3.0 obsoletes this test\n");
+		piglit_report_result(PIGLIT_SKIP);
+	}
+
 	PassthroughFragShader = piglit_compile_shader_text(GL_FRAGMENT_SHADER,
 							   PassthroughFragShaderText);
 	assert(PassthroughFragShader);
diff --git a/tests/spec/gl-3.0/api/CMakeLists.gl.txt b/tests/spec/gl-3.0/api/CMakeLists.gl.txt
index 466ffeb..2cb72aa 100644
--- a/tests/spec/gl-3.0/api/CMakeLists.gl.txt
+++ b/tests/spec/gl-3.0/api/CMakeLists.gl.txt
@@ -19,5 +19,6 @@ add_executable (clearbuffer-invalid-drawbuffer clearbuffer-invalid-drawbuffer.c)
 add_executable (clearbuffer-invalid-buffer clearbuffer-invalid-buffer.c)
 add_executable (clearbuffer-stencil clearbuffer-common.c clearbuffer-stencil.c)
 add_executable (getfragdatalocation getfragdatalocation.c)
+add_executable (gl-3.0-drawpixels-integer drawpixels-integer.c)
 
 # vim: ft=cmake:
diff --git a/tests/spec/gl-3.0/api/drawpixels-integer.c b/tests/spec/gl-3.0/api/drawpixels-integer.c
new file mode 100644
index 0000000..bc6aa68
--- /dev/null
+++ b/tests/spec/gl-3.0/api/drawpixels-integer.c
@@ -0,0 +1,86 @@
+/*
+ * 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 drawpixels-integer.c
+ *
+ * Tests the GL 3.0 error behavior of glDrawPixels(), which is more
+ * restrictive than the original GL_EXT_texture_integer spec.
+ */
+
+#include "piglit-util.h"
+
+int piglit_width = 10, piglit_height = 10;
+int piglit_window_mode = GLUT_RGB | GLUT_ALPHA | GLUT_DOUBLE;
+
+enum piglit_result
+piglit_display(void)
+{
+	static const int black[4] = {0, 0, 0, 0};
+	static const float green[4] = {0, 1, 0, 0};
+	bool pass;
+
+	/* We don't have to do an integer FBO for this test, because
+	 * no error is specified in the non-integer case:
+	 *
+	 *     "Results of rasterization are undefined if any of the
+	 *      selected draw buffers of the draw framebuffer have an
+	 *      integer format and no fragment shader is active.  "
+	 */
+	glClearColor(0.0, 1.0, 0.0, 0.0);
+	glClear(GL_COLOR_BUFFER_BIT);
+
+	/* GL 3.0 introduced a new restriction on glDrawPixels() over
+	 * what was in GL_EXT_texture_integer.  From section 3.7.4
+	 * ("Rasterization of Pixel Rectangles) on page 151 of the GL
+	 * 3.0 specification:
+	 *
+	 *     "If format contains integer components, as shown in
+	 *      table 3.6, an INVALID OPERATION error is generated."
+	 */
+	glDrawPixels(1, 1, GL_RGBA_INTEGER_EXT, GL_UNSIGNED_INT, black);
+	piglit_check_gl_error(GL_INVALID_OPERATION, PIGLIT_FAIL);
+
+	/* This precedes the restriction from GL_EXT_texture_integer
+	 * which is still included in that section:
+	 *
+	 *     "If format is one of the integer component formats as
+	 *      defined in table 3.6 and type is FLOAT, the error
+	 *      INVALID ENUM occurs."
+	 *
+	 * Based on this, we test for GL_INVALID_OPERATION even for FLOAT.
+	 */
+	glDrawPixels(1, 1, GL_RGBA_INTEGER_EXT, GL_FLOAT, black);
+	piglit_check_gl_error(GL_INVALID_OPERATION, PIGLIT_FAIL);
+
+	/* Make sure that we really didn't render anything. */
+	pass = piglit_probe_rect_rgba(0, 0, piglit_width, piglit_height, green);
+
+	return pass ? PIGLIT_PASS : PIGLIT_FAIL;
+}
+
+void
+piglit_init(int argc, char **argv)
+{
+	piglit_require_gl_version(30);
+}
-- 
1.7.7.3



More information about the mesa-dev mailing list