Commit cd8e9956 authored by Nirmoy Das's avatar Nirmoy Das
Browse files

drm/xe: Take job list lock in xe_sched_first_pending_job



Access to the pending_list should always happens under job_list_lock.

Cc: Matthew Brost <matthew.brost@intel.com>
Reviewed-by: default avatarMatthew Brost <matthew.brost@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20241105160327.2970277-1-nirmoy.das@intel.com


Signed-off-by: default avatarNirmoy Das <nirmoy.das@intel.com>
parent f7278da7
Loading
Loading
Loading
Loading
+8 −2
Original line number Diff line number Diff line
@@ -71,8 +71,14 @@ static inline void xe_sched_add_pending_job(struct xe_gpu_scheduler *sched,
static inline
struct xe_sched_job *xe_sched_first_pending_job(struct xe_gpu_scheduler *sched)
{
	return list_first_entry_or_null(&sched->base.pending_list,
	struct xe_sched_job *job;

	spin_lock(&sched->base.job_list_lock);
	job = list_first_entry_or_null(&sched->base.pending_list,
				       struct xe_sched_job, drm.list);
	spin_unlock(&sched->base.job_list_lock);

	return job;
}

static inline int