<html> <head> <base href="https://bugs.freedesktop.org/"> </head> <body> <div> <a class="bz_bug_link bz_status_ASSIGNED " title="ASSIGNED - SSO: wrong interface validation between GS and VS (regresion due to latest gles 3.1)" href="https://bugs.freedesktop.org/show_bug.cgi?id=96358#c4">Comment # 4</a> on <a class="bz_bug_link bz_status_ASSIGNED " title="ASSIGNED - SSO: wrong interface validation between GS and VS (regresion due to latest gles 3.1)" href="https://bugs.freedesktop.org/show_bug.cgi?id=96358">bug 96358</a> from <a class="email" href="mailto:idr@freedesktop.org" title="Ian Romanick <idr@freedesktop.org>"> Ian Romanick</a> <pre>Created <a href="attachment.cgi?id=124453" name="attach_124453" title="Strip arrayness from interface block names in some IO validation">attachment 124453</a> <a href="attachment.cgi?id=124453&action=edit" title="Strip arrayness from interface block names in some IO validation">[details]</a> <a href='page.cgi?id=splinter.html&bug=96358&attachment=124453'>[review]</a> Strip arrayness from interface block names in some IO validation This patch is a bit of a hack, but I think it should fix the problem. If it works, I'll improve it & send to the list for review. If this doesn't fix the problem, I think we should just remove IO validation from desktop GL altogether. diff --git a/src/mesa/main/pipelineobj.c b/src/mesa/main/pipelineobj.c index 5a46cfe..10a9544 100644 --- a/src/mesa/main/pipelineobj.c +++ b/src/mesa/main/pipelineobj.c @@ -928,8 +928,7 @@ _mesa_validate_program_pipeline(struct gl_context* ctx, * Based on this, only perform the most-strict checking on ES or when the * application has created a debug context. */ - if ((_mesa_is_gles(ctx) || (ctx->Const.ContextFlags & GL_CONTEXT_FLAG_DEBUG_BIT)) && - !_mesa_validate_pipeline_io(pipe)) + if (_mesa_is_gles(ctx) && !_mesa_validate_pipeline_io(pipe)) return GL_FALSE; pipe->Validated = GL_TRUE;</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the QA Contact for the bug.</li> </ul> </body> </html>