[Mesa-dev] [PATCH 1/5] mesa: Do (TCS && !TES) draw time validation in ES as well.
kenneth at whitecape.org
Sat Feb 11 09:24:49 UTC 2017
On Saturday, February 11, 2017 12:21:36 AM PST Alejandro Piñeiro wrote:
> On 11/02/17 08:52, Kenneth Graunke wrote:
> > Now that we have OES_tessellation_shader, the same situation can occur
> > in ES too, not just GL core profile.
> > Having a TCS but no TES may confuse drivers - i965 crashes, for example.
> > This prevents regressions in
> > ES31-CTS.core.tessellation_shader.single.xfb_captures_data_from_correct_stage
> > with some SSO pipeline validation changes I'm making.
> > Cc: "17.0" <mesa-stable at lists.freedesktop.org>
> > Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
> > ---
> > src/mesa/main/api_validate.c | 38 +++++++++++++++++++-------------------
> > 1 file changed, 19 insertions(+), 19 deletions(-)
> > diff --git a/src/mesa/main/api_validate.c b/src/mesa/main/api_validate.c
> > index 6c95701ea0e..d64264fe1e9 100644
> > --- a/src/mesa/main/api_validate.c
> > +++ b/src/mesa/main/api_validate.c
> > @@ -236,6 +236,25 @@ check_valid_to_render(struct gl_context *ctx, const char *function)
> > return false;
> > }
> > + /* The spec argues that this is allowed because a tess ctrl shader
> > + * without a tess eval shader can be used with transform feedback.
> > + * However, glBeginTransformFeedback doesn't allow GL_PATCHES and
> > + * therefore doesn't allow tessellation.
> > + *
> > + * Further investigation showed that this is indeed a spec bug and
> > + * a tess ctrl shader without a tess eval shader shouldn't have been
> > + * allowed, because there is no API in GL 4.0 that can make use this
> > + * to produce something useful.
> As the check is done for both OpenGL and OpenGL ES, how about update the
> comment to include both, as right now only mentions desktop?
> In any case, this is a nitpick, feel free to ignore if you think that
> increases the comment without good reason.
Good call, I've changed it to:
+ /* Section 11.2 (Tessellation) of the ES 3.2 spec says:
+ * "An INVALID_OPERATION error is generated by any command that
+ * transfers vertices to the GL if the current program state has
+ * one but not both of a tessellation control shader and tessellation
+ * evaluation shader."
+ * The OpenGL spec argues that this is allowed because a tess ctrl shader
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 833 bytes
Desc: This is a digitally signed message part.
More information about the mesa-dev