freedreno: drop unused arg from fd_batch_flush()
authorRob Clark <robdclark@chromium.org>
Tue, 25 Jun 2019 18:21:28 +0000 (11:21 -0700)
committerRob Clark <robdclark@chromium.org>
Wed, 26 Jun 2019 15:43:02 +0000 (08:43 -0700)
The `force` arg has been unused for a while.. but apparently I forgot to
garbage collect it.

Signed-off-by: Rob Clark <robdclark@chromium.org>
12 files changed:
src/gallium/drivers/freedreno/a5xx/fd5_blitter.c
src/gallium/drivers/freedreno/a6xx/fd6_blitter.c
src/gallium/drivers/freedreno/freedreno_batch.c
src/gallium/drivers/freedreno/freedreno_batch.h
src/gallium/drivers/freedreno/freedreno_batch_cache.c
src/gallium/drivers/freedreno/freedreno_context.c
src/gallium/drivers/freedreno/freedreno_draw.c
src/gallium/drivers/freedreno/freedreno_fence.c
src/gallium/drivers/freedreno/freedreno_query_acc.c
src/gallium/drivers/freedreno/freedreno_query_hw.c
src/gallium/drivers/freedreno/freedreno_resource.c
src/gallium/drivers/freedreno/freedreno_state.c

index 8a829759cdcde387988692ab1a8993e6d5f50519..8fdf4c47c3188cc1569209b8a9f8c083be7b3bea 100644 (file)
@@ -481,7 +481,7 @@ fd5_blitter_blit(struct fd_context *ctx, const struct pipe_blit_info *info)
        fd_resource(info->dst.resource)->valid = true;
        batch->needs_flush = true;
 
-       fd_batch_flush(batch, false, false);
+       fd_batch_flush(batch, false);
        fd_batch_reference(&batch, NULL);
 
        return true;
index 3b7020510b06370382cf982967b1a84e5a2beb53..afa162e9ebe8f7f03ada53a8c859a90346433bb8 100644 (file)
@@ -698,7 +698,7 @@ handle_rgba_blit(struct fd_context *ctx, const struct pipe_blit_info *info)
        fd_resource(info->dst.resource)->valid = true;
        batch->needs_flush = true;
 
-       fd_batch_flush(batch, false, false);
+       fd_batch_flush(batch, false);
        fd_batch_reference(&batch, NULL);
 
        return true;
index 14b4e38568c1140b6081acd3611f460b3304544c..88d0a816c00d8259f45a4067f65cb52443ee68aa 100644 (file)
@@ -198,7 +198,7 @@ batch_flush_reset_dependencies(struct fd_batch *batch, bool flush)
 
        foreach_batch(dep, cache, batch->dependents_mask) {
                if (flush)
-                       fd_batch_flush(dep, false, false);
+                       fd_batch_flush(dep, false);
                fd_batch_reference(&dep, NULL);
        }
 
@@ -307,7 +307,7 @@ batch_cleanup_func(void *job, int id)
 }
 
 static void
-batch_flush(struct fd_batch *batch, bool force)
+batch_flush(struct fd_batch *batch)
 {
        DBG("%p: needs_flush=%d", batch, batch->needs_flush);
 
@@ -356,7 +356,7 @@ batch_flush(struct fd_batch *batch, bool force)
  *   a fence to sync on
  */
 void
-fd_batch_flush(struct fd_batch *batch, bool sync, bool force)
+fd_batch_flush(struct fd_batch *batch, bool sync)
 {
        struct fd_batch *tmp = NULL;
        bool newbatch = false;
@@ -372,7 +372,7 @@ fd_batch_flush(struct fd_batch *batch, bool sync, bool force)
                newbatch = true;
        }
 
-       batch_flush(tmp, force);
+       batch_flush(tmp);
 
        if (newbatch) {
                struct fd_context *ctx = batch->ctx;
@@ -438,7 +438,7 @@ flush_write_batch(struct fd_resource *rsc)
        fd_batch_reference_locked(&b, rsc->write_batch);
 
        mtx_unlock(&b->ctx->screen->lock);
-       fd_batch_flush(b, true, false);
+       fd_batch_flush(b, true);
        mtx_lock(&b->ctx->screen->lock);
 
        fd_bc_invalidate_batch(b, false);
@@ -512,7 +512,7 @@ fd_batch_check_size(struct fd_batch *batch)
        debug_assert(!batch->flushed);
 
        if (unlikely(fd_mesa_debug & FD_DBG_FLUSH)) {
-               fd_batch_flush(batch, true, false);
+               fd_batch_flush(batch, true);
                return;
        }
 
@@ -521,7 +521,7 @@ fd_batch_check_size(struct fd_batch *batch)
 
        struct fd_ringbuffer *ring = batch->draw;
        if ((ring->cur - ring->start) > (ring->size/4 - 0x1000))
-               fd_batch_flush(batch, true, false);
+               fd_batch_flush(batch, true);
 }
 
 /* emit a WAIT_FOR_IDLE only if needed, ie. if there has not already
index d38bd3f8b22d3fb84e2a85f535f1ab268fab604b..edf0840825be470855be6c77bc112d1512b9aad7 100644 (file)
@@ -229,7 +229,7 @@ struct fd_batch * fd_batch_create(struct fd_context *ctx, bool nondraw);
 
 void fd_batch_reset(struct fd_batch *batch);
 void fd_batch_sync(struct fd_batch *batch);
-void fd_batch_flush(struct fd_batch *batch, bool sync, bool force);
+void fd_batch_flush(struct fd_batch *batch, bool sync);
 void fd_batch_add_dep(struct fd_batch *batch, struct fd_batch *dep);
 void fd_batch_resource_used(struct fd_batch *batch, struct fd_resource *rsc, bool write);
 void fd_batch_check_size(struct fd_batch *batch);
index ec3aab128a87bd7f46a6f669d71d8c2fd29a97f2..82b285c9c6ba87d4d5b826b9dc9dcb5f0471d0d7 100644 (file)
@@ -159,7 +159,7 @@ bc_flush(struct fd_batch_cache *cache, struct fd_context *ctx, bool deferred)
                fd_context_unlock(ctx);
 
                for (unsigned i = 0; i < n; i++) {
-                       fd_batch_flush(batches[i], false, false);
+                       fd_batch_flush(batches[i], false);
                }
        }
 
@@ -307,7 +307,7 @@ fd_bc_alloc_batch(struct fd_batch_cache *cache, struct fd_context *ctx, bool non
                 */
                mtx_unlock(&ctx->screen->lock);
                DBG("%p: too many batches!  flush forced!", flush_batch);
-               fd_batch_flush(flush_batch, true, false);
+               fd_batch_flush(flush_batch, true);
                mtx_lock(&ctx->screen->lock);
 
                /* While the resources get cleaned up automatically, the flush_batch
index c30bfe44dd5ae123123bfa93eeadd0a6b64e7b8f..ad03ac30fb849ed0dfae5d550df3745a2b747ab3 100644 (file)
@@ -69,7 +69,7 @@ fd_context_flush(struct pipe_context *pctx, struct pipe_fence_handle **fencep,
        batch->needs_out_fence_fd = true;
 
        if (!ctx->screen->reorder) {
-               fd_batch_flush(batch, true, false);
+               fd_batch_flush(batch, true);
        } else if (flags & PIPE_FLUSH_DEFERRED) {
                fd_bc_flush_deferred(&ctx->screen->batch_cache, ctx);
        } else {
index 2908ac52c9c071fc64b12301f2382d0c563f324e..2bb19d37d4e21a8bcdc6e759618c6eacd64e0fbe 100644 (file)
@@ -457,7 +457,7 @@ fd_launch_grid(struct pipe_context *pctx, const struct pipe_grid_info *info)
        batch->needs_flush = true;
        ctx->launch_grid(ctx, info);
 
-       fd_batch_flush(batch, false, false);
+       fd_batch_flush(batch, false);
 
        fd_batch_reference(&ctx->batch, save_batch);
        fd_context_all_dirty(ctx);
index 68bbb25ee645dc373e41bdf5c64a0040e5f7feac..940497bd495729ae73ebf2ca8cdaa3a208429c46 100644 (file)
@@ -48,7 +48,7 @@ struct pipe_fence_handle {
 static void fence_flush(struct pipe_fence_handle *fence)
 {
        if (fence->batch)
-               fd_batch_flush(fence->batch, true, true);
+               fd_batch_flush(fence->batch, true);
        debug_assert(!fence->batch);
 }
 
index d7e1404fff80fe32220f475088d48bea50641123..7590fc1fbd7e3da28b7a4cea84b888eb1e99eaad 100644 (file)
@@ -139,7 +139,7 @@ fd_acc_get_query_result(struct fd_context *ctx, struct fd_query *q,
                         * spin forever:
                         */
                        if (aq->no_wait_cnt++ > 5)
-                               fd_batch_flush(rsc->write_batch, false, false);
+                               fd_batch_flush(rsc->write_batch, false);
                        return false;
                }
 
@@ -152,7 +152,7 @@ fd_acc_get_query_result(struct fd_context *ctx, struct fd_query *q,
        }
 
        if (rsc->write_batch)
-               fd_batch_flush(rsc->write_batch, true, false);
+               fd_batch_flush(rsc->write_batch, true);
 
        /* get the result: */
        fd_bo_cpu_prep(rsc->bo, ctx->pipe, DRM_FREEDRENO_PREP_READ);
index 7c3da59394b6675e432f1b94ba4c2c9222258dbc..212451b0de4d750971000f9c8c57435d7ec726c9 100644 (file)
@@ -209,7 +209,7 @@ fd_hw_get_query_result(struct fd_context *ctx, struct fd_query *q,
                         * spin forever:
                         */
                        if (hq->no_wait_cnt++ > 5)
-                               fd_batch_flush(rsc->write_batch, false, false);
+                               fd_batch_flush(rsc->write_batch, false);
                        return false;
                }
 
@@ -237,7 +237,7 @@ fd_hw_get_query_result(struct fd_context *ctx, struct fd_query *q,
                struct fd_resource *rsc = fd_resource(start->prsc);
 
                if (rsc->write_batch)
-                       fd_batch_flush(rsc->write_batch, true, false);
+                       fd_batch_flush(rsc->write_batch, true);
 
                /* some piglit tests at least do query with no draws, I guess: */
                if (!rsc->bo)
index 7b1b1bf43f86284c55c65375c26c22eb77da4817..e17a1e85d8759b4c9470f73f85eb62cd6bb7cfa1 100644 (file)
@@ -449,7 +449,7 @@ flush_resource(struct fd_context *ctx, struct fd_resource *rsc, unsigned usage)
                mtx_unlock(&ctx->screen->lock);
 
                foreach_batch(batch, &ctx->screen->batch_cache, batch_mask)
-                       fd_batch_flush(batch, false, false);
+                       fd_batch_flush(batch, false);
 
                foreach_batch(batch, &ctx->screen->batch_cache, batch_mask) {
                        fd_batch_sync(batch);
@@ -457,7 +457,7 @@ flush_resource(struct fd_context *ctx, struct fd_resource *rsc, unsigned usage)
                }
                assert(rsc->batch_mask == 0);
        } else if (write_batch) {
-               fd_batch_flush(write_batch, true, false);
+               fd_batch_flush(write_batch, true);
        }
 
        fd_batch_reference(&write_batch, NULL);
index 238efd448991dabe76ad4757db934f8ba1eb8771..e8ea6b0023da9c2c32cafc0fc0f5172db27d75c4 100644 (file)
@@ -248,14 +248,14 @@ fd_set_framebuffer_state(struct pipe_context *pctx,
                         * multiple times to the same surface), so we might as
                         * well go ahead and flush this one:
                         */
-                       fd_batch_flush(old_batch, false, false);
+                       fd_batch_flush(old_batch, false);
                }
 
                fd_batch_reference(&old_batch, NULL);
        } else {
                DBG("%d: cbufs[0]=%p, zsbuf=%p", ctx->batch->needs_flush,
                                framebuffer->cbufs[0], framebuffer->zsbuf);
-               fd_batch_flush(ctx->batch, false, false);
+               fd_batch_flush(ctx->batch, false);
                util_copy_framebuffer_state(&ctx->batch->framebuffer, cso);
        }