drm/xe: Fix guc_exec_queue_set_priority
authorBrian Welty <brian.welty@intel.com>
Fri, 5 Jan 2024 19:04:39 +0000 (11:04 -0800)
committerThomas Hellström <thomas.hellstrom@linux.intel.com>
Mon, 15 Jan 2024 14:36:48 +0000 (15:36 +0100)
We need to set q->priority prior to calling guc_exec_queue_add_msg() as
that will call init_policies() and sets the scheduling properties to those
stored in the exec_queue.

Fixes: dd08ebf6c352 ("drm/xe: Introduce a new DRM driver for Intel GPUs")
Signed-off-by: Brian Welty <brian.welty@intel.com>
Signed-off-by: Matthew Brost <matthew.brost@intel.com>
Reviewed-by: Matthew Brost <matthew.brost@intel.com>
(cherry picked from commit b16483f9f8120b530327879fa3ea576e897946da)
Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
drivers/gpu/drm/xe/xe_guc_submit.c

index 21ac68e3246f86f1e16d05880da8aa9315769ecb..5de3ac47c4623d91b6a50804d0265babbe52d737 100644 (file)
@@ -1308,8 +1308,8 @@ static int guc_exec_queue_set_priority(struct xe_exec_queue *q,
        if (!msg)
                return -ENOMEM;
 
-       guc_exec_queue_add_msg(q, msg, SET_SCHED_PROPS);
        q->priority = priority;
+       guc_exec_queue_add_msg(q, msg, SET_SCHED_PROPS);
 
        return 0;
 }