[Piglit] [PATCH] arb_direct_state_access: Simplified texturesubimage test.
Laura Ekstrand
laura at jlekstrand.net
Fri Dec 19 13:09:22 PST 2014
---
.../spec/arb_direct_state_access/texturesubimage.c | 390 ++++-----------------
1 file changed, 66 insertions(+), 324 deletions(-)
diff --git a/tests/spec/arb_direct_state_access/texturesubimage.c b/tests/spec/arb_direct_state_access/texturesubimage.c
index b3a09e6..9f3e0fb 100644
--- a/tests/spec/arb_direct_state_access/texturesubimage.c
+++ b/tests/spec/arb_direct_state_access/texturesubimage.c
@@ -25,363 +25,100 @@
/**
- * This should expose any errors in texel addressing within a texture image
- * when calling glTextureSubImage1D/2D/3D().
+ * Test glTextureSubImage*D. This is pretty trivial, since it only uses
+ * glTextureSubImage*D with offsets of 0 and the width, height, and depth of
+ * the full image. Moreover, it doesn't test varying depths for the 3D case.
+ * But since DSA functions share backends with the non-DSA ones, we really
+ * only need to test entry points here.
*
+ * Laura Ekstrand
+ * October 2014
+ *
+ * Based on texsubimage.c by
* Brian Paul
* October 2011
*
- * Adapted for testing ARB_direct_state_access by
- * Laura Ekstrand
- * October 2014
*/
#include "piglit-util-gl.h"
-#include "../fbo/fbo-formats.h"
+#include <stdlib.h>
+#include <string.h>
PIGLIT_GL_TEST_CONFIG_BEGIN
config.supports_gl_compat_version = 10;
- config.window_visual = PIGLIT_GL_VISUAL_RGBA | PIGLIT_GL_VISUAL_DOUBLE;
+ config.window_visual = PIGLIT_GL_VISUAL_RGBA |
+ PIGLIT_GL_VISUAL_DOUBLE;
PIGLIT_GL_TEST_CONFIG_END
-/**
- * This is a subset of the formats in fbo-formats.h
- * We don't test non-color, float, or int/uint textures at this time.
- */
-static const struct test_desc texsubimage_test_sets[] = {
- {
- core,
- ARRAY_SIZE(core),
- "Core formats",
- GL_UNSIGNED_NORMALIZED,
- },
- {
- tdfx_texture_compression_fxt1,
- ARRAY_SIZE(tdfx_texture_compression_fxt1),
- "GL_3DFX_texture_compression_FXT1",
- GL_UNSIGNED_NORMALIZED,
- {"GL_ARB_texture_compression",
- "GL_3DFX_texture_compression_FXT1"},
- },
- {
- ext_texture_compression_s3tc,
- ARRAY_SIZE(ext_texture_compression_s3tc),
- "GL_EXT_texture_compression_s3tc",
- GL_UNSIGNED_NORMALIZED,
- {"GL_ARB_texture_compression",
- "GL_EXT_texture_compression_s3tc"},
- },
- {
- ext_texture_compression_rgtc,
- ARRAY_SIZE(ext_texture_compression_rgtc),
- "GL_EXT_texture_compression_rgtc",
- GL_UNSIGNED_NORMALIZED,
- {"GL_EXT_texture_compression_rgtc"}
- },
- {
- ext_texture_compression_latc,
- ARRAY_SIZE(ext_texture_compression_latc),
- "GL_EXT_texture_compression_latc",
- GL_UNSIGNED_NORMALIZED,
- {"GL_EXT_texture_compression_latc"}
- }
-};
-
-
-/**
- * XXX add this to piglit-util if useful elsewhere.
- */
-static GLvoid
-piglit_draw_rect_tex3d(float x, float y, float w, float h,
- float tx, float ty, float tw, float th,
- float tz0, float tz1)
+GLubyte*
+random_image_data(int width, int height, int depth)
{
- float verts[4][4];
- float tex[4][3];
-
- verts[0][0] = x;
- verts[0][1] = y;
- verts[0][2] = 0.0;
- verts[0][3] = 1.0;
- tex[0][0] = tx;
- tex[0][1] = ty;
- tex[0][2] = tz0;
- verts[1][0] = x + w;
- verts[1][1] = y;
- verts[1][2] = 0.0;
- verts[1][3] = 1.0;
- tex[1][0] = tx + tw;
- tex[1][1] = ty;
- tex[1][2] = tz1;
- verts[2][0] = x + w;
- verts[2][1] = y + h;
- verts[2][2] = 0.0;
- verts[2][3] = 1.0;
- tex[2][0] = tx + tw;
- tex[2][1] = ty + th;
- tex[2][2] = tz1;
- verts[3][0] = x;
- verts[3][1] = y + h;
- verts[3][2] = 0.0;
- verts[3][3] = 1.0;
- tex[3][0] = tx;
- tex[3][1] = ty + th;
- tex[3][2] = tz0;
-
- glVertexPointer(4, GL_FLOAT, 0, verts);
- glTexCoordPointer(3, GL_FLOAT, 0, tex);
- glEnableClientState(GL_VERTEX_ARRAY);
- glEnableClientState(GL_TEXTURE_COORD_ARRAY);
-
- glDrawArrays(GL_QUADS, 0, 4);
-
- glDisableClientState(GL_VERTEX_ARRAY);
- glDisableClientState(GL_TEXTURE_COORD_ARRAY);
-}
-
-
-static GLboolean
-equal_images(const GLubyte *img1, const GLubyte *img2,
- GLuint w, GLuint h)
-{
- return memcmp(img1, img2, w*h*4) == 0;
-}
-
-
-/**
- * Get block size for compressed format.
- * \return GL_TRUE if format is compressed, GL_FALSE otherwise
- * XXX this could be a piglit util function if useful elsewhere.
- */
-static GLboolean
-get_format_block_size(GLenum format, GLuint *bw, GLuint *bh)
-{
- switch (format) {
- case GL_COMPRESSED_RGB_FXT1_3DFX:
- case GL_COMPRESSED_RGBA_FXT1_3DFX:
- *bw = 8;
- *bh = 4;
- return GL_TRUE;
- case GL_COMPRESSED_RGB_S3TC_DXT1_EXT:
- case GL_COMPRESSED_RGBA_S3TC_DXT1_EXT:
- case GL_COMPRESSED_RGBA_S3TC_DXT3_EXT:
- case GL_COMPRESSED_RGBA_S3TC_DXT5_EXT:
- case GL_COMPRESSED_SRGB_S3TC_DXT1_EXT:
- case GL_COMPRESSED_SRGB_ALPHA_S3TC_DXT1_EXT:
- case GL_COMPRESSED_SRGB_ALPHA_S3TC_DXT3_EXT:
- case GL_COMPRESSED_SRGB_ALPHA_S3TC_DXT5_EXT:
- *bw = 4;
- *bh = 4;
- return GL_TRUE;
- case GL_COMPRESSED_RED:
- case GL_COMPRESSED_RED_RGTC1_EXT:
- case GL_COMPRESSED_RG:
- case GL_COMPRESSED_RED_GREEN_RGTC2_EXT:
- *bw = 4;
- *bh = 4;
- return GL_TRUE;
- case GL_COMPRESSED_LUMINANCE_LATC1_EXT:
- case GL_COMPRESSED_LUMINANCE_ALPHA_LATC2_EXT:
- *bw = 4;
- *bh = 4;
- return GL_TRUE;
- default:
- *bw = *bh = 1;
- return GL_FALSE;
+ int i;
+ GLubyte *img = malloc(4 * width * height * depth * sizeof(GLubyte));
+ for (i = 0; i < 4 * width * height * depth; ++i) {
+ img[i] = rand() % 256;
}
-}
+ return img;
+} /* random_image_data */
+static int depth = 4;
+static GLubyte *refImg;
-/**
- * Create a texture image with reference values. Draw a textured quad.
- * Save reference image with glReadPixels().
- * Loop:
- * replace a sub-region of the texture image with same values
- * draw test textured quad
- * read test image with glReadPixels
- * compare reference image to test image
- * \param target GL_TEXTURE_1D/2D/3D
- * \param intFormat the internal texture format
- */
-static GLboolean
-test_format(GLenum target, GLenum intFormat)
+static enum piglit_result
+subtest(GLenum target)
{
- const GLenum srcFormat = GL_RGBA;
- GLuint w = 128, h = 64, d = 8;
- GLuint tex, i, j, k, n, t;
- GLubyte *img, *ref, *testImg;
- GLboolean pass = GL_TRUE;
- GLuint bw, bh, wMask, hMask, dMask;
- get_format_block_size(intFormat, &bw, &bh);
- wMask = ~(bw-1);
- hMask = ~(bh-1);
- dMask = ~0;
-
- if (target != GL_TEXTURE_3D)
- d = 1;
- if (target == GL_TEXTURE_1D)
- h = 1;
-
- img = (GLubyte *) malloc(w * h * d * 4);
- ref = (GLubyte *) malloc(w * h * d * 4);
- testImg = (GLubyte *) malloc(w * h * d * 4);
-
- /* fill source tex image */
- n = 0;
- for (i = 0; i < d; i++) {
- for (j = 0; j < h; j++) {
- for (k = 0; k < w; k++) {
- img[n++] = j * 4;
- img[n++] = k * 2;
- img[n++] = i * 16;
- img[n++] = 255;
- }
- }
- }
-
- glPixelStorei(GL_UNPACK_ROW_LENGTH, w);
- glPixelStorei(GL_UNPACK_IMAGE_HEIGHT, h);
+ bool pass = true;
+ GLuint tex;
+ /* Draw the reference image. */
glCreateTextures(target, 1, &tex);
- glBindTextureUnit(0, tex);
glTextureParameteri(tex, GL_TEXTURE_MIN_FILTER, GL_NEAREST);
glTextureParameteri(tex, GL_TEXTURE_MAG_FILTER, GL_NEAREST);
- glPixelStorei(GL_UNPACK_SKIP_PIXELS, 0);
- glPixelStorei(GL_UNPACK_SKIP_ROWS, 0);
- glPixelStorei(GL_UNPACK_SKIP_IMAGES, 0);
if (target == GL_TEXTURE_1D) {
- glTextureStorage1D(tex, 1, intFormat, w);
- glTextureSubImage1D(tex, 0, 0, w, srcFormat,
- GL_UNSIGNED_BYTE, img);
+ glTextureStorage1D(tex, 1, GL_RGBA8, piglit_width);
+ glTextureSubImage1D(tex, 0, 0, piglit_width, GL_RGBA,
+ GL_UNSIGNED_BYTE, refImg);
}
else if (target == GL_TEXTURE_2D) {
- glTextureStorage2D(tex, 1, intFormat, w, h);
- glTextureSubImage2D(tex, 0, 0, 0, w, h, srcFormat,
- GL_UNSIGNED_BYTE, img);
+ glTextureStorage2D(tex, 1, GL_RGBA8, piglit_width,
+ piglit_height);
+ glTextureSubImage2D(tex, 0, 0, 0, piglit_width, piglit_height,
+ GL_RGBA, GL_UNSIGNED_BYTE, refImg);
}
else if (target == GL_TEXTURE_3D) {
- glTextureStorage3D(tex, 1, intFormat, w, h, d);
- glTextureSubImage3D(tex, 0, 0, 0, 0, w, h, d, srcFormat,
- GL_UNSIGNED_BYTE, img);
+ glTextureStorage3D(tex, 1, GL_RGBA8, piglit_width,
+ piglit_height, depth);
+ glTextureSubImage3D(tex, 0, 0, 0, 0,
+ piglit_width, piglit_height, 1,
+ GL_RGBA, GL_UNSIGNED_BYTE,
+ refImg);
}
+ glBindTextureUnit(0, tex);
glEnable(target);
-
- glTexEnvi(GL_TEXTURE_ENV, GL_TEXTURE_ENV_MODE, GL_REPLACE);
-
- /* draw reference image */
glClear(GL_COLOR_BUFFER_BIT);
- piglit_draw_rect_tex3d(0, 0, w, h, 0.0, 0.0, 1.0, 1.0, 0.0, 1.0);
- glReadPixels(0, 0, w, h, GL_RGBA, GL_UNSIGNED_BYTE, ref);
-
- for (t = 0; t < 10; t++) {
- /* Choose random region of texture to update.
- * Use sizes and positions that are multiples of
- * the compressed block size.
- */
- GLint tw = (rand() % w) & wMask;
- GLint th = (rand() % h) & hMask;
- GLint td = (rand() % d) & dMask;
- GLint tx = (rand() % (w - tw)) & wMask;
- GLint ty = (rand() % (h - th)) & hMask;
- GLint tz = (rand() % (d - td)) & dMask;
-
- assert(tx + tw <= w);
- assert(ty + th <= h);
- assert(tz + td <= d);
-
- /* replace texture region (with same data) */
- glPixelStorei(GL_UNPACK_SKIP_PIXELS, tx);
- glPixelStorei(GL_UNPACK_SKIP_ROWS, ty);
- glPixelStorei(GL_UNPACK_SKIP_IMAGES, tz);
- if (target == GL_TEXTURE_1D) {
- glTextureSubImage1D(tex, 0, tx, tw,
- srcFormat, GL_UNSIGNED_BYTE, img);
- }
- else if (target == GL_TEXTURE_2D) {
- glTextureSubImage2D(tex, 0, tx, ty, tw, th,
- srcFormat, GL_UNSIGNED_BYTE, img);
- }
- else if (target == GL_TEXTURE_2D) {
- glTextureSubImage3D(tex, 0, tx, ty, tz, tw, th, td,
- srcFormat, GL_UNSIGNED_BYTE, img);
- }
-
- /* draw test image */
- glClear(GL_COLOR_BUFFER_BIT);
- piglit_draw_rect_tex3d(0, 0, w, h, 0.0, 0.0, 1.0, 1.0, 0.0, 1.0);
- glReadPixels(0, 0, w, h, GL_RGBA, GL_UNSIGNED_BYTE, testImg);
-
- piglit_present_results();
-
- if (!equal_images(ref, testImg, w, h)) {
- printf("texsubimage failed\n");
- printf(" target: %s\n", piglit_get_gl_enum_name(target));
- printf(" internal format: %s\n", piglit_get_gl_enum_name(intFormat));
- printf(" region: %d, %d %d x %d\n", tx, ty, tw, th);
- pass = GL_FALSE;
- break;
- }
+ piglit_draw_rect_tex(0, 0, piglit_width, piglit_height, 0, 0, 1, 1);
+ if (target == GL_TEXTURE_1D) {
+ pass &= piglit_probe_image_ubyte(0, 0, piglit_width, 1,
+ GL_RGBA, refImg);
}
-
- glDisable(target);
-
- free(img);
- free(ref);
- free(testImg);
-
- glDeleteTextures(1, &tex);
- return pass;
-}
-
-
-/**
- * Test all formats in texsubimage_test_sets[] for the given
- * texture target.
- */
-static GLboolean
-test_formats(GLenum target)
-{
- GLboolean pass = GL_TRUE;
- int i, j;
-
- /* loop over the format groups */
- for (i = 0; i < ARRAY_SIZE(texsubimage_test_sets); i++) {
- const struct test_desc *set = &texsubimage_test_sets[i];
- GLboolean skip = GL_FALSE;
-
- /* only test compressed formats with 2D textures */
- if (i > 0 && target != GL_TEXTURE_2D)
- continue;
-
- /* skip formats for unsupported extensions */
- for (j = 0; j < ARRAY_SIZE(set->ext); j++) {
- if (set->ext[j] &&
- !piglit_is_extension_supported(set->ext[j])) {
- /* req'd extension not supported */
- skip = GL_TRUE;
- break;
- }
- }
- if (skip)
- continue;
-
- /* loop over formats in the set */
- for (j = 0; j < set->num_formats; j++) {
- if (!test_format(target,
- set->format[j].internalformat)) {
- pass = GL_FALSE;
- }
- }
+ else {
+ pass &= piglit_probe_image_ubyte(0, 0, piglit_width,
+ piglit_height,
+ GL_RGBA, refImg);
}
- return pass;
-}
+ if (!piglit_automatic)
+ piglit_present_results();
+ piglit_report_subtest_result(pass ? PIGLIT_PASS : PIGLIT_FAIL,
+ "%s", piglit_get_gl_enum_name(target));
+ return pass ? PIGLIT_PASS : PIGLIT_FAIL;
+}
enum piglit_result
piglit_display(void)
@@ -391,15 +128,18 @@ piglit_display(void)
GL_TEXTURE_2D,
GL_TEXTURE_3D
};
- GLboolean pass = GL_TRUE;
int i;
+ enum piglit_result result = PIGLIT_PASS;
+ enum piglit_result subtest_result;
/* Loop over 1/2/3D texture targets */
for (i = 0; i < ARRAY_SIZE(targets); i++) {
- pass = test_formats(targets[i]) && pass;
+ subtest_result = subtest(targets[i]);
+ if (subtest_result != PIGLIT_PASS)
+ result = PIGLIT_FAIL;
}
- return pass ? PIGLIT_PASS : PIGLIT_FAIL;
+ return result;
}
@@ -408,8 +148,10 @@ piglit_init(int argc, char **argv)
{
piglit_require_extension("GL_ARB_direct_state_access");
- fbo_formats_init(argc, argv, 0);
- (void) fbo_formats_display;
+ srand(0);
piglit_ortho_projection(piglit_width, piglit_height, GL_FALSE);
+
+ /* Make the image data for testing. */
+ refImg = random_image_data(piglit_width, piglit_height, depth);
}
--
2.1.0
More information about the Piglit
mailing list