[Piglit] [PATCH] Add a test for GLES2 using GL_EXT_unpack_subimage with GL_BGRA_EXT

Neil Roberts neil at linux.intel.com
Fri Oct 5 03:27:31 PDT 2012

Eric Anholt <eric at anholt.net> writes:

> If this is testing GL_EXT_unpack_subimage, it should go in
> tests/spec/gl_ext_unpack_subimage/

It's not really testing GL_EXT_unpack_subimage but is testing a specific
bug in the Intel driver when uploading a subregion of a BGR image. There
is already a test for GL_EXT_unpack_subimage and this test case borrows
some code from it. Maybe it would make more sense to convert this test
to use regular GL. Then it wouldn't need a shader and could be quite a
bit simpler. However, the bug was found via Wayland which is using GLES2
so it seemed like a good way to replicate the bug.

>> +static GLboolean pass;
> I'm assuming Brian's review covered most everything, but as a note:
> We're trying to move to reasonable C types like bool for everything that
> isn't specified by GL API.

I think Rob is adapting the test case for Brian's changes so hopefully
he can incorporate this suggestion too.

- Neil

More information about the Piglit mailing list