From 659a41293b034631332bba232b0a076ea2b499ff Mon Sep 17 00:00:00 2001 From: Yogesh Mohan Marimuthu Date: Fri, 21 Feb 2025 09:18:19 +0530 Subject: [PATCH] winsys/amdgpu: same_queue variable should be set if there is only one queue MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fixes: 45fa34284fd ("winsys/amdgpu: don't add fence dependency of other queues for userq") Reviewed-by: Marek Olšák Reviewed-by: Pierre-Eric Pelloux-Prayer Part-of: --- src/gallium/winsys/amdgpu/drm/amdgpu_cs.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.cpp b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.cpp index 84d38c936a2..b701a5ee722 100644 --- a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.cpp +++ b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.cpp @@ -1653,8 +1653,8 @@ static void amdgpu_cs_submit_ib(void *job, void *gdata, int thread_index) if (prev_fence) { bool same_ctx = queue->last_ctx == acs->ctx; /* userqueue submission mode uses a single queue per process. */ - bool same_queue = aws->info.ip[acs->ip_type].num_queues > 1 && - queue_type != USERQ; + bool same_queue = queue_type == USERQ || aws->info.ip[acs->ip_type].num_queues == 1; + if (!same_ctx || !same_queue) add_seq_no_to_list(aws, &seq_no_dependencies, queue_index, prev_seq_no); }