[Piglit] [PATCH 02/24] arb_direct_state_access: glTextureSubImage*D test.
Laura Ekstrand
laura at jlekstrand.net
Fri Dec 19 09:47:38 PST 2014
I agree with you that DSA really needs minimal testing. I used this test
because it was a quick way to test most of the entry points in a trivial
manner (but not so trivial as to not use any image data). In my DSA
patches, the DSA entry points share the same implementation as the
traditional ones, so as long as the traditional functions are thoroughly
tested, the DSA functions really just need to be called.
Therefore, I'm inclined not to merge your changes into my texturesubimage.c
test. I will probably just fix that TEXTURE_3D error you found (thanks for
pointing that out), and maybe scale the test back by removing the formats
and other unneeded details.
Thanks for pointing this out, and let me know if you have a better idea.
Laura
On Thu, Dec 18, 2014 at 1:29 PM, Neil Roberts <neil at linux.intel.com> wrote:
> Today I pushed some patches to the original version of this test which
> change it quite a lot. Previously it updated a sub-region of the texture
> with the exact same contents as were already in the texture so an
> implementation that did nothing on glTexSubImage[123]D would actually
> pass the test. It might be worth updating this test to the new version
> otherwise it's really not testing very much and it doesn't test
> glTextureSubImage3D at all.
>
> On the other hand I think testing textures with all possible formats
> might be a bit overkill just for testing DSA and it's a shame to have so
> much duplicated code. I wonder if it would be better to make a simpler
> test that only tests one format and one target per function?
>
> http://cgit.freedesktop.org/piglit/commit/?id=5cfdc9d1d46fce911284576
> http://cgit.freedesktop.org/piglit/commit/?id=8433c118dfa6f03f615724d
> http://cgit.freedesktop.org/piglit/commit/?id=e4e88dfdb22be7b75150eae
>
> I've also just posted some patches today to test it with array textures
> but I don't think it would make sense to apply those for DSA.
>
> Regards,
> - Neil
>
> Laura Ekstrand <laura at jlekstrand.net> writes:
>
> > ---
> > tests/all.py | 1 +
> > .../spec/arb_direct_state_access/CMakeLists.gl.txt | 1 +
> > .../spec/arb_direct_state_access/texturesubimage.c | 415
> +++++++++++++++++++++
> > 3 files changed, 417 insertions(+)
> > create mode 100644 tests/spec/arb_direct_state_access/texturesubimage.c
> >
> > diff --git a/tests/all.py b/tests/all.py
> > index 84639c2..5ed0444 100644
> > --- a/tests/all.py
> > +++ b/tests/all.py
> > @@ -4308,6 +4308,7 @@ import_glsl_parser_tests(arb_derivative_control,
> >
> > spec['ARB_direct_state_access'] = {}
> > spec['ARB_direct_state_access']['dsa-textures'] =
> PiglitGLTest('arb_direct_state_access-dsa-textures', run_concurrent=True)
> > +spec['ARB_direct_state_access']['texturesubimage'] =
> PiglitGLTest('arb_direct_state_access-texturesubimage', run_concurrent=True)
> >
> > profile.tests['hiz'] = hiz
> > profile.tests['fast_color_clear'] = fast_color_clear
> > diff --git a/tests/spec/arb_direct_state_access/CMakeLists.gl.txt
> b/tests/spec/arb_direct_state_access/CMakeLists.gl.txt
> > index cb0f7da..102579c 100644
> > --- a/tests/spec/arb_direct_state_access/CMakeLists.gl.txt
> > +++ b/tests/spec/arb_direct_state_access/CMakeLists.gl.txt
> > @@ -10,4 +10,5 @@ link_libraries (
> > )
> >
> > piglit_add_executable (arb_direct_state_access-dsa-textures
> dsa-textures.c dsa-utils.c)
> > +piglit_add_executable (arb_direct_state_access-dsa-texturesubimage
> texturesubimage.c)
> > # vim: ft=cmake:
> > diff --git a/tests/spec/arb_direct_state_access/texturesubimage.c
> b/tests/spec/arb_direct_state_access/texturesubimage.c
> > new file mode 100644
> > index 0000000..b3a09e6
> > --- /dev/null
> > +++ b/tests/spec/arb_direct_state_access/texturesubimage.c
> > @@ -0,0 +1,415 @@
> > +/*
> > + * Copyright © 2011 VMware, Inc.
> > + * Copyright © 2014 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.
> > + *
> > + */
> > +
> > +
> > +/**
> > + * This should expose any errors in texel addressing within a texture
> image
> > + * when calling glTextureSubImage1D/2D/3D().
> > + *
> > + * 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"
> > +
> > +PIGLIT_GL_TEST_CONFIG_BEGIN
> > +
> > + config.supports_gl_compat_version = 10;
> > +
> > + 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)
> > +{
> > + 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;
> > + }
> > +}
> > +
> > +
> > +/**
> > + * 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)
> > +{
> > + 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);
> > +
> > + 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);
> > + }
> > + else if (target == GL_TEXTURE_2D) {
> > + glTextureStorage2D(tex, 1, intFormat, w, h);
> > + glTextureSubImage2D(tex, 0, 0, 0, w, h, srcFormat,
> > + GL_UNSIGNED_BYTE, img);
> > + }
> > + 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);
> > + }
> > +
> > + 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;
> > + }
> > + }
> > +
> > + 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;
> > + }
> > + }
> > + }
> > +
> > + return pass;
> > +}
> > +
> > +
> > +enum piglit_result
> > +piglit_display(void)
> > +{
> > + static const GLenum targets[] = {
> > + GL_TEXTURE_1D,
> > + GL_TEXTURE_2D,
> > + GL_TEXTURE_3D
> > + };
> > + GLboolean pass = GL_TRUE;
> > + int i;
> > +
> > + /* Loop over 1/2/3D texture targets */
> > + for (i = 0; i < ARRAY_SIZE(targets); i++) {
> > + pass = test_formats(targets[i]) && pass;
> > + }
> > +
> > + return pass ? PIGLIT_PASS : PIGLIT_FAIL;
> > +}
> > +
> > +
> > +void
> > +piglit_init(int argc, char **argv)
> > +{
> > + piglit_require_extension("GL_ARB_direct_state_access");
> > +
> > + fbo_formats_init(argc, argv, 0);
> > + (void) fbo_formats_display;
> > +
> > + piglit_ortho_projection(piglit_width, piglit_height, GL_FALSE);
> > +}
> > --
> > 2.1.0
> >
> > _______________________________________________
> > Piglit mailing list
> > Piglit at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/piglit
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20141219/d07446df/attachment-0001.html>
More information about the Piglit
mailing list