On 2016-12-12 07:06:50, Rafael Antognolli wrote: > On Fri, Dec 09, 2016 at 10:35:58PM -0800, Jordan Justen wrote: > > On 2016-12-09 13:39:52, Rafael Antognolli wrote: > > > This extension adds new query types which can be used to detect overflow > > > of transform feedback buffers. The new query types are also accepted by > > > conditional rendering commands. > > > > > > Signed-off-by: Rafael Antognolli <rafael.antogno...@intel.com> > > > --- > > > docs/features.txt | 2 +- > > > docs/relnotes/13.1.0.html | 1 + > > > src/mesa/drivers/dri/i965/intel_extensions.c | 1 + > > > 3 files changed, 3 insertions(+), 1 deletion(-) > > > > > > diff --git a/docs/features.txt b/docs/features.txt > > > index c27d521..bb7925e 100644 > > > --- a/docs/features.txt > > > +++ b/docs/features.txt > > > @@ -303,7 +303,7 @@ Khronos, ARB, and OES extensions that are not part of > > > any OpenGL or OpenGL ES ve > > > GL_ARB_sparse_texture2 not started > > > GL_ARB_sparse_texture_clamp not started > > > GL_ARB_texture_filter_minmax not started > > > - GL_ARB_transform_feedback_overflow_query not started > > > + GL_ARB_transform_feedback_overflow_query DONE (i965/gen7+) > > > GL_KHR_blend_equation_advanced_coherent DONE (i965/gen9+) > > > GL_KHR_no_error not started > > > GL_KHR_texture_compression_astc_hdr DONE (core only) > > > diff --git a/docs/relnotes/13.1.0.html b/docs/relnotes/13.1.0.html > > > index 5b8b016..4f52cd1 100644 > > > --- a/docs/relnotes/13.1.0.html > > > +++ b/docs/relnotes/13.1.0.html > > > @@ -45,6 +45,7 @@ Note: some of the new features are only available with > > > certain drivers. > > > > > > <ul> > > > <li>GL_ARB_post_depth_coverage on i965/gen9+</li> > > > +<li>GL_ARB_transform_feedback_overflow_query on i965/gen7+</li> > > > <li>GL_NV_image_formats on any driver supporting > > > GL_ARB_shader_image_load_store (i965, nvc0, radeonsi, softpipe)</li> > > > </ul> > > > > > > diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c > > > b/src/mesa/drivers/dri/i965/intel_extensions.c > > > index c1f42aa..d5e4164 100644 > > > --- a/src/mesa/drivers/dri/i965/intel_extensions.c > > > +++ b/src/mesa/drivers/dri/i965/intel_extensions.c > > > @@ -320,6 +320,7 @@ intelInitExtensions(struct gl_context *ctx) > > > ctx->Extensions.EXT_framebuffer_multisample = true; > > > ctx->Extensions.EXT_framebuffer_multisample_blit_scaled = true; > > > ctx->Extensions.EXT_transform_feedback = true; > > > + ctx->Extensions.ARB_transform_feedback_overflow_query = true; > > > > Is this enabling the extension on gen6? > > Yes. > > > Should it depend on brw->predicate.supported (and thus the next > > patch)? > > It is possible to query for overflow on gen6, so in theory there's no > reason to not enable it.
The code (gen6+) doesn't match with the doc changes (gen7+). I think the patch subject should also indicate the gen. For example: i965/gen6+: Enable arb_transform_feedback_overflow_query > However, predication only becomes available on gen7, and having > transform feedback overflow query without conditional render support > might not be that useful. > > PS: The next patch enables predication on gen >= 8, but it was > already enabled on haswell (gen7). I guess I was concerned that after patch 6, the driver might be in a state where gen8 was advertising something that wouldn't work until after patch 7. So, this isn't the case? I expected to see the last patch in a series enables the extension, which is why I asked about it. -Jordan _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
