Fixed commit message and applied to drm-misc-fixes
On 4/25/2025 11:36 AM, Jacek Lawrynowicz wrote:
> From: Karol Wachowski <karol.wachow...@intel.com>
>
> The mutex unlock for vdev->submitted_jobs_lock was incorrectly placed
> after unlocking file_priv->lock. Change order of unlocks to avoid potential
> race conditions.
>
> Fixes: 5bbccadaf33e ("accel/ivpu: Abort all jobs after command queue
> unregister")
> Signed-off-by: Karol Wachowski <karol.wachow...@intel.com>
> Signed-off-by: Jacek Lawrynowicz <jacek.lawrynow...@linux.intel.com>
> ---
> drivers/accel/ivpu/ivpu_job.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/accel/ivpu/ivpu_job.c b/drivers/accel/ivpu/ivpu_job.c
> index 863e3cd6ace51..540923ffcbf8d 100644
> --- a/drivers/accel/ivpu/ivpu_job.c
> +++ b/drivers/accel/ivpu/ivpu_job.c
> @@ -681,8 +681,8 @@ static int ivpu_job_submit(struct ivpu_job *job, u8
> priority, u32 cmdq_id)
> err_erase_xa:
> xa_erase(&vdev->submitted_jobs_xa, job->job_id);
> err_unlock:
> - mutex_unlock(&vdev->submitted_jobs_lock);
> mutex_unlock(&file_priv->lock);
> + mutex_unlock(&vdev->submitted_jobs_lock);
> ivpu_rpm_put(vdev);
> return ret;
> }