etnaviv: remove the "active" member of queries
authorChristian Gmeiner <christian.gmeiner@gmail.com>
Sun, 5 Apr 2020 19:46:55 +0000 (21:46 +0200)
committerMarge Bot <eric+marge@anholt.net>
Fri, 10 Apr 2020 12:42:24 +0000 (12:42 +0000)
The state tracker only gets to begin/query/destroy when !active and end
when active, so we have no need to try to track this ourselves.

Signed-off-by: Christian Gmeiner <christian.gmeiner@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4456>

src/gallium/drivers/etnaviv/etnaviv_query.c
src/gallium/drivers/etnaviv/etnaviv_query.h
src/gallium/drivers/etnaviv/etnaviv_query_acc.h

index ee936a6e6164f64e54c64b365acb0eb3280ee630..01ec3bebc8e5d1ac59058126eb1acb00c1e8b7d6 100644 (file)
@@ -61,11 +61,7 @@ etna_begin_query(struct pipe_context *pctx, struct pipe_query *pq)
 {
    struct etna_query *q = etna_query(pq);
 
-   if (q->active)
-      return false;
-
    q->funcs->begin_query(etna_context(pctx), q);
-   q->active = true;
 
    return true;
 }
@@ -75,11 +71,7 @@ etna_end_query(struct pipe_context *pctx, struct pipe_query *pq)
 {
    struct etna_query *q = etna_query(pq);
 
-   if (!q->active)
-      return false;
-
    q->funcs->end_query(etna_context(pctx), q);
-   q->active = false;
 
    return true;
 }
@@ -90,9 +82,6 @@ etna_get_query_result(struct pipe_context *pctx, struct pipe_query *pq,
 {
    struct etna_query *q = etna_query(pq);
 
-   if (q->active)
-      return false;
-
    util_query_clear_result(result, q->type);
 
    return q->funcs->get_query_result(etna_context(pctx), q, wait, result);
index afa1743bd61feb10b538f6192afaca8a8a5fabb3..c3c5911f7d0bb019f1f7413ce72d10531e08cdce 100644 (file)
@@ -43,7 +43,6 @@ struct etna_query_funcs {
 
 struct etna_query {
    const struct etna_query_funcs *funcs;
-   bool active;
    unsigned type;
 };
 
index 526513af12aff2b894d8dfd70932228e185d5bd3..9204befa03dd7fe645b031d9036092d0584e897f 100644 (file)
@@ -69,9 +69,6 @@ etna_acc_query_suspend(struct etna_acc_query *aq, struct etna_context *ctx)
 {
    const struct etna_acc_sample_provider *p = aq->provider;
 
-   if (!aq->base.active)
-      return;
-
    p->suspend(aq, ctx);
    aq->samples++;
 }
@@ -81,9 +78,6 @@ etna_acc_query_resume(struct etna_acc_query *aq, struct etna_context *ctx)
 {
    const struct etna_acc_sample_provider *p = aq->provider;
 
-   if (!aq->base.active)
-      return;
-
    p->resume(aq, ctx);
    aq->samples++;
 }