Mesa (master): freedreno/query/acc: Set needs_flush

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Mon May 3 23:50:04 UTC 2021


Module: Mesa
Branch: master
Commit: 71cff8171c22d0b363073625cfb4aab46c15c2ba
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=71cff8171c22d0b363073625cfb4aab46c15c2ba

Author: Rob Clark <robdclark at chromium.org>
Date:   Mon May  3 13:28:21 2021 -0700

freedreno/query/acc: Set needs_flush

Somehow this was missed, but when we emit a query start/stop we need
have something that will need to be flushed in the batch.

Detected due to TC assert, but this had the potential to cause problems
in the non-TC case as well.

Signed-off-by: Rob Clark <robdclark at chromium.org>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/10599>

---

 src/gallium/drivers/freedreno/freedreno_query_acc.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/gallium/drivers/freedreno/freedreno_query_acc.c b/src/gallium/drivers/freedreno/freedreno_query_acc.c
index 77268e6ab7c..c8370f7733a 100644
--- a/src/gallium/drivers/freedreno/freedreno_query_acc.c
+++ b/src/gallium/drivers/freedreno/freedreno_query_acc.c
@@ -75,6 +75,7 @@ fd_acc_query_pause(struct fd_acc_query *aq) assert_dt
    if (!aq->batch)
       return;
 
+   fd_batch_needs_flush(aq->batch);
    p->pause(aq, aq->batch);
    aq->batch = NULL;
 }
@@ -85,6 +86,7 @@ fd_acc_query_resume(struct fd_acc_query *aq, struct fd_batch *batch) assert_dt
    const struct fd_acc_sample_provider *p = aq->provider;
 
    aq->batch = batch;
+   fd_batch_needs_flush(aq->batch);
    p->resume(aq, aq->batch);
 
    fd_screen_lock(batch->ctx->screen);



More information about the mesa-commit mailing list