winsys/amdgpu: same_queue variable should be set if there is only one queue

Fixes: 45fa34284f ("winsys/amdgpu: don't add fence dependency of other queues for userq")

Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/33661>
(cherry picked from commit 659a41293b)
This commit is contained in:
Yogesh Mohan Marimuthu 2025-02-21 09:18:19 +05:30 committed by Eric Engestrom
parent f8e7fecd7e
commit 13b2f1e72d
2 changed files with 3 additions and 3 deletions

View file

@ -804,7 +804,7 @@
"description": "winsys/amdgpu: same_queue variable should be set if there is only one queue",
"nominated": true,
"nomination_type": 2,
"resolution": 0,
"resolution": 1,
"main_sha": null,
"because_sha": "45fa34284fd46a48d99f1f62bbbef0b5c1326fcb",
"notes": null

View file

@ -1658,8 +1658,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);
}