From f7102bed69dd46e51ca9bf6e5630ba46ad8b6716 Mon Sep 17 00:00:00 2001 From: Paulo Zanoni Date: Mon, 2 Aug 2021 13:13:13 -0700 Subject: [PATCH] iris: don't bump the seqno for the workaround_bo The last_seqnos list is used by iris_emit_buffer_barrier_for() and as far as I can understand we don't emit barriers for the workaround bo, so don't even bother doing the atomic operations required to bump the workaround_bo seqno list. Reviewed-by: Kenneth Graunke Signed-off-by: Paulo Zanoni Part-of: --- src/gallium/drivers/iris/iris_batch.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/iris/iris_batch.c b/src/gallium/drivers/iris/iris_batch.c index ff48cfa77b1..1d94cf86a41 100644 --- a/src/gallium/drivers/iris/iris_batch.c +++ b/src/gallium/drivers/iris/iris_batch.c @@ -281,10 +281,9 @@ iris_use_pinned_bo(struct iris_batch *batch, * would introduce data dependencies between multiple batches which share * the buffer. */ - if (bo == batch->screen->workaround_bo) + if (bo == batch->screen->workaround_bo) { writable = false; - - if (access < NUM_IRIS_DOMAINS) { + } else if (access < NUM_IRIS_DOMAINS) { assert(batch->sync_region_depth); iris_bo_bump_seqno(bo, batch->next_seqno, access); }