[Piglit] [PATCH 1/2] port ttexrect.cpp to piglit from glean
Brian Paul
brianp at vmware.com
Fri Jan 18 12:32:16 PST 2013
On 01/18/2013 01:04 PM, Tom Gall wrote:
> This ported test goes into tests/spec/arb_texture_rectangle. It
> tests the ARB_texture_rectangle extension.
>
> Create a 255x127 texture of varying colors and bind it as a
> GL_ARB_texture_recangle target. Draw that rectangle to the window, and
> check that the texture was drawn correctly. The common failure to be
> caught with this test is not adjusting the non-normalized coordinates on
> hardware that expects normalized coordinates.
>
> Signed-off-by: Tom Gall<tom.gall at linaro.org>
> ---
> tests/spec/CMakeLists.txt | 1 +
> tests/spec/arb_texture_rectangle/CMakeLists.gl.txt | 13 ++
> tests/spec/arb_texture_rectangle/CMakeLists.txt | 1 +
> tests/spec/arb_texture_rectangle/texrect-simple.c | 132 ++++++++++++++++++++
Update tests/all.tests too?
> 4 files changed, 147 insertions(+)
> create mode 100644 tests/spec/arb_texture_rectangle/CMakeLists.gl.txt
> create mode 100644 tests/spec/arb_texture_rectangle/CMakeLists.txt
> create mode 100644 tests/spec/arb_texture_rectangle/texrect-simple.c
>
> diff --git a/tests/spec/CMakeLists.txt b/tests/spec/CMakeLists.txt
> index a4342b7..8b06d5c 100644
> --- a/tests/spec/CMakeLists.txt
> +++ b/tests/spec/CMakeLists.txt
> @@ -24,6 +24,7 @@ add_subdirectory (arb_texture_buffer_object)
> add_subdirectory (arb_texture_compression)
> add_subdirectory (arb_texture_cube_map_array)
> add_subdirectory (arb_texture_float)
> +add_subdirectory (arb_texture_rectangle)
> add_subdirectory (arb_texture_storage)
> add_subdirectory (arb_timer_query)
> add_subdirectory (arb_transform_feedback2)
> diff --git a/tests/spec/arb_texture_rectangle/CMakeLists.gl.txt b/tests/spec/arb_texture_rectangle/CMakeLists.gl.txt
> new file mode 100644
> index 0000000..d84a850
> --- /dev/null
> +++ b/tests/spec/arb_texture_rectangle/CMakeLists.gl.txt
> @@ -0,0 +1,13 @@
> +include_directories(
> + ${GLEXT_INCLUDE_DIR}
> + ${OPENGL_INCLUDE_PATH}
> + ${piglit_SOURCE_DIR}/tests/mesa/util
> +)
> +
> +link_libraries (
> + piglitutil_${piglit_target_api}
> + ${OPENGL_gl_LIBRARY}
> + ${OPENGL_glu_LIBRARY}
> +)
> +
> +piglit_add_executable (texrect_simple_arb_texrect texrect-simple.c)
> diff --git a/tests/spec/arb_texture_rectangle/CMakeLists.txt b/tests/spec/arb_texture_rectangle/CMakeLists.txt
> new file mode 100644
> index 0000000..144a306
> --- /dev/null
> +++ b/tests/spec/arb_texture_rectangle/CMakeLists.txt
> @@ -0,0 +1 @@
> +piglit_include_target_api()
> diff --git a/tests/spec/arb_texture_rectangle/texrect-simple.c b/tests/spec/arb_texture_rectangle/texrect-simple.c
> new file mode 100644
> index 0000000..1a8f33f
> --- /dev/null
> +++ b/tests/spec/arb_texture_rectangle/texrect-simple.c
> @@ -0,0 +1,132 @@
> +/* Copyright © 2013 Linaro Inc
> + *
> + * 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.
> + */
> +
> +/**
> + * Ported from glean ttexrect.cpp into piglit.
> + * Test the ARB_texture_rectangle extension
> + * Create a 255x127 texture of varying colors and bind it as a
> + * GL_ARB_texture_recangle target. Draw that rectangle to the window, and
> + * check that the texture was drawn correctly. The common failure to be
> + * caught with this test is not adjusting the non-normalized coordinates on
> + * hardware that expects normalized coordinates.
> + * \author: Eric Anholt<eric at anholt.net> (original)
> + * \author: Tom Gall<tom.gall at linaro.org> (port)
> + */
> +#include "piglit-util-gl-common.h"
> +
> +#define TEXTURE_WIDTH 255
> +#define TEXTURE_HEIGHT 127
> +#define WINDOW_SIZE 256
> +
> +PIGLIT_GL_TEST_CONFIG_BEGIN
> +
> + config.supports_gl_compat_version = 10;
> + config.window_width = WINDOW_SIZE;
> + config.window_height = WINDOW_SIZE;
> + config.window_visual = PIGLIT_GL_VISUAL_RGB | PIGLIT_GL_VISUAL_DEPTH;
Add PIGLIT_GL_VISUAL_DOUBLE since you call glDrawBuffer(GL_BACK) below.
> +
> +PIGLIT_GL_TEST_CONFIG_END
> +
> +
> +enum piglit_result
> +piglit_display(void)
> +{
> + int x,y,i;
> + enum piglit_result final_result = PIGLIT_PASS;
> + float image[TEXTURE_WIDTH * TEXTURE_HEIGHT * 3];
> +
> + /* Set up a texture that is color ramps with red to black top to
> + * bottom and green to black left to right.
> + */
> + for (y = 0; y< TEXTURE_HEIGHT; y++) {
> + for (x = 0; x< TEXTURE_WIDTH; x++) {
> + i = (y * TEXTURE_WIDTH + x) * 3;
> +
> + image[i + 0] = (float)x / (TEXTURE_WIDTH - 1);
> + image[i + 1] = 1.0 - ((float) y / (TEXTURE_HEIGHT - 1));
> + image[i + 2] = 0.0;
> + }
> + }
> +
> + glClearColor(0.0, 0.0, 0.0, 0.0);
> + glClear(GL_COLOR_BUFFER_BIT);
> +
> + glShadeModel(GL_FLAT);
> +
> + glMatrixMode(GL_PROJECTION);
> + glLoadIdentity();
> + glOrtho(0, WINDOW_SIZE, 0, WINDOW_SIZE, -1, 1);
> + glMatrixMode(GL_MODELVIEW);
> + glLoadIdentity();
> + glViewport(0, 0, WINDOW_SIZE, WINDOW_SIZE);
> +
> + glTexImage2D(GL_TEXTURE_RECTANGLE_ARB, 0, GL_RGB,
> + TEXTURE_WIDTH, TEXTURE_HEIGHT, 0, GL_RGB, GL_FLOAT, image);
> + glTexParameteri(GL_TEXTURE_RECTANGLE_ARB, GL_TEXTURE_MIN_FILTER,
> + GL_NEAREST);
> + glTexParameteri(GL_TEXTURE_RECTANGLE_ARB, GL_TEXTURE_MAG_FILTER,
> + GL_NEAREST);
> + glTexEnvi(GL_TEXTURE_ENV, GL_TEXTURE_ENV_MODE, GL_REPLACE);
> + glEnable(GL_TEXTURE_RECTANGLE_ARB);
> +
> + glDrawBuffer(GL_BACK);
> + glReadBuffer(GL_BACK);
We'd typically put most of this setup code in piglit_init() but it's
not a big deal to me.
> +
> + /* Draw our texture to the window such that each texel should map
> + * to the corresponding pixel of the window.
> + */
> + glBegin(GL_POLYGON);
> + glTexCoord2f(0, 0);
> + glVertex2f(0, 0);
> +
> + glTexCoord2f(TEXTURE_WIDTH, 0);
> + glVertex2f(TEXTURE_WIDTH, 0);
> +
> + glTexCoord2f(TEXTURE_WIDTH, TEXTURE_HEIGHT);
> + glVertex2f(TEXTURE_WIDTH, TEXTURE_HEIGHT);
> +
> + glTexCoord2f(0, TEXTURE_HEIGHT);
> + glVertex2f(0, TEXTURE_HEIGHT);
> + glEnd();
I think there's a piglit_draw_xxx() function for a simple textured
quad like that. Not a big deal though.
> +
> + piglit_swap_buffers();
> +
> + /* Verify the output */
> + for (y = 0; y< TEXTURE_HEIGHT; y++) {
> + for (x = 0; x< TEXTURE_WIDTH; x++) {
> + i = (y * TEXTURE_WIDTH + x) * 3;
> +
> + if (!piglit_probe_pixel_rgb(x,y,&image[i])) {
> + final_result = PIGLIT_FAIL;
> + }
> + }
> + }
> +
> + piglit_report_result(final_result);
> + return final_result;
> +}
> +
> +
> +void
> +piglit_init(int argc, char *argv[])
> +{
> +}
Looks good otherwise.
Reviewed-by: Brian Paul <brianp at vmware.com>
Thanks for doing this!
-Brian
More information about the Piglit
mailing list