Kenneth Graunke <kenn...@whitecape.org> writes: > brw->query.index is initialized to 0 just a few lines before it's > copied to first_index. > > Presumably the idea here was to reuse the query BO for subsequent > queries of the same type, but since that doesn't happen, there's no need > to have the extra code complexity.
Yeah, that was the idea. Only, you also want to avoid keeping the old query buffer in use for any longer after the old query had ended, or you'll introduce extra latency on the query results. So I'm happy to see this garbage collected now. (For any other reviewers trying to confirm the assertion that we don't reuse, it's the drm_intel_bo_unreference(brw->query.bo) in brw_end_query()) Reviewed-by: Eric Anholt <e...@anholt.net>
pgppqY1Pi9BNE.pgp
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
