[Mesa-dev] [PATCH] mesa: relax draw api validation on ES2
Ian Romanick
idr at freedesktop.org
Tue Sep 30 08:13:18 PDT 2014
On 09/30/2014 12:28 AM, Tapani Pälli wrote:
> Patch fixes failing test in WebGL conformance test
> 'point-no-attributes' when running Chrome on OpenGL ES.
> (Shader program may draw points using constant data in shader.)
>
> No Piglit regressions.
This sounds believable. Did you also try the ES2 or ES3 conformance
suite? I could have sworn that we had a bug related to this a long time
ago, and we discovered it using the conformance suite.
Either way, we should get a piglit test too... I think we have a test
for desktop OpenGL (maybe 3.1?), so it shouldn't be too hard to adapt that.
> Signed-off-by: Tapani Pälli <tapani.palli at intel.com>
> ---
> src/mesa/main/api_validate.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/src/mesa/main/api_validate.c b/src/mesa/main/api_validate.c
> index 51a3d1f..9b80600 100644
> --- a/src/mesa/main/api_validate.c
> +++ b/src/mesa/main/api_validate.c
> @@ -112,9 +112,8 @@ check_valid_to_render(struct gl_context *ctx, const char *function)
>
> switch (ctx->API) {
> case API_OPENGLES2:
> - /* For ES2, we can draw if any vertex array is enabled (and we
> - * should always have a vertex program/shader). */
> - if (ctx->Array.VAO->_Enabled == 0x0 || !ctx->VertexProgram._Current)
> + /* For ES2, we can draw if we have a vertex program/shader). */
> + if (!ctx->VertexProgram._Current)
> return GL_FALSE;
> break;
>
>
More information about the mesa-dev
mailing list