Yeah... since the split of Gen4/5 and Gen6+, those checks seem vestigial anyway. This patch is
Reviewed-by: Ian Romanick <[email protected]> On 11/23/2017 02:24 AM, Chris Wilson wrote: > Since we have HW contexts on gen4/5, we could take advantage of them, as > done for gen6+ in commit e32cd5ffbb72 ("i965: Rely on hardware contexts > for query objects on Gen6+."), to only emit a pair of counters at > begin/end queryobj, rather than around every primitive. However, to keep > queryobj working in the meantime as we bringup support for HW ctx on > gen4/5, we can keep using the existing code. > > References: e32cd5ffbb72 ("i965: Rely on hardware contexts for query objects > on Gen6+.") > Cc: Kenneth Graunke <[email protected]> > --- > src/mesa/drivers/dri/i965/brw_queryobj.c | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_queryobj.c > b/src/mesa/drivers/dri/i965/brw_queryobj.c > index bd3f5738eb..f667f55b6b 100644 > --- a/src/mesa/drivers/dri/i965/brw_queryobj.c > +++ b/src/mesa/drivers/dri/i965/brw_queryobj.c > @@ -480,9 +480,6 @@ brw_emit_query_begin(struct brw_context *brw) > struct gl_context *ctx = &brw->ctx; > struct brw_query_object *query = brw->query.obj; > > - if (brw->hw_ctx) > - return; > - > /* Skip if we're not doing any queries, or we've already recorded the > * initial query value for this batchbuffer. > */ > @@ -507,9 +504,6 @@ brw_emit_query_end(struct brw_context *brw) > { > struct brw_query_object *query = brw->query.obj; > > - if (brw->hw_ctx) > - return; > - > if (!brw->query.begin_emitted) > return; > > _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
