iris: Fix overzealous query object batch flushing.
authorKenneth Graunke <kenneth@whitecape.org>
Wed, 26 Jun 2019 04:14:11 +0000 (21:14 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Wed, 26 Jun 2019 16:49:01 +0000 (09:49 -0700)
commitab009b7d6e25909dacf2fa15fdbd15fd80b16327
tree33e25f88006d6f4517cd08c4f0ac19045f1bfdb2
parentdb878a728c9f3cc932ec9af77dbabcb28ff5a5ba
iris: Fix overzealous query object batch flushing.

In the past, each query object had their own BO.  Checking if the batch
referenced that BO was an easy way to check if commands were still
queued to compute the query value.  If so, we needed to flush.

More recently (c24a574e6c), we started using an u_upload_mgr for query
objects, placing multiple queries in the same BO.  One side-effect is
that iris_batch_references is a no longer a reasonable way to check if
commands are still queued for our query.  Ours might be done, but a
later query that happens to be in the same BO might be queued.  We don't
want to flush in that case.

Instead, check if the current batch's signalling syncpt is the one we
referenced when ending the query.  We know the syncpt can't have been
reused because our query is holding a reference, so a simple pointer
comparison should suffice.

Removes all batch flushing caused by query objects in Shadow of Mordor.

Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
src/gallium/drivers/iris/iris_query.c