[Piglit] [PATCH 3/4] arb_internalformat_query2: fix TBO testing

[Marek Olšák]

On Sun, Oct 22, 2017 at 7:55 PM, Alejandro Piñeiro <apinheiro at igalia.com> wrote:
> On 22/10/17 17:21, Ilia Mirkin wrote:
>> On Sun, Oct 22, 2017 at 6:33 AM, Alejandro Piñeiro <apinheiro at igalia.com> wrote:
>>> On 21/10/17 14:55, Marek Olšák wrote:
>>>> From: Marek Olšák <marek.olsak at amd.com>
>>>>
>>>> This was never tested and it uses functions that are illegal with TBOs,
>>>> like glGetTexParameteriv and glTexImage2DMultisample.
>>> Hi, I didn't check all the patch in detail, but as Illia is saying, it
>>> is correct to pass TEXTURE_BUFFER to GetInternalFormat*. I also disagree
>>> that this case it is not tested, or at least for the case I checked. See
>>> below, I have two extra comments.
>> It's untested because these are all run in compat contexts, which in
>> turn don't have GL_TEXTURE_BUFFER support. [At least untested against
>> current mesa.]
>
> Ah ok. In any case, somehow is still testing it, as in compat mode it
> should return NONE, as you say, because on that case it doesn't support
> TEXTURE_BUFFER (dependencies are checked).

I think GL_INVALID_ENUM is returned by Mesa, because GL_TEXTURE_BUFFER
is not a valid enum in OpenGL 3.0 + ARB_internalformat_query2.

>
> In any case, as I said (you mentioned it before) the solution is not
> test TBO, but as you said, fix the piglit tests to run in non compat
> modes. Probably improve more than fix, in order to test both compat and
> non compat modes.

My primary goal is adjust the tests such they don't fail with OpenGL
3.1. Of course, people can improve/fix the tests if they want.

Marek