Re: [PATCH] accel/ivpu: Correct mutex unlock order in job submission

2025-04-30 Thread Jacek Lawrynowicz
Fixed commit message and applied to drm-misc-fixes On 4/25/2025 11:36 AM, Jacek Lawrynowicz wrote: > From: Karol Wachowski > > 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. >

Re: [PATCH] accel/ivpu: Correct mutex unlock order in job submission

2025-04-29 Thread Jeff Hugo
On 4/29/2025 2:41 AM, Jacek Lawrynowicz wrote: Hi, On 4/28/2025 4:21 PM, Jeff Hugo wrote: On 4/28/2025 12:47 AM, Jacek Lawrynowicz wrote: Hi, On 4/25/2025 7:22 PM, Jeff Hugo wrote: On 4/25/2025 3:36 AM, Jacek Lawrynowicz wrote: From: Karol Wachowski The mutex unlock for vdev->submitted_jo

Re: [PATCH] accel/ivpu: Correct mutex unlock order in job submission

2025-04-29 Thread Jacek Lawrynowicz
Hi, On 4/28/2025 4:21 PM, Jeff Hugo wrote: > On 4/28/2025 12:47 AM, Jacek Lawrynowicz wrote: >> Hi, >> >> On 4/25/2025 7:22 PM, Jeff Hugo wrote: >>> On 4/25/2025 3:36 AM, Jacek Lawrynowicz wrote: From: Karol Wachowski The mutex unlock for vdev->submitted_jobs_lock was incorrectly p

Re: [PATCH] accel/ivpu: Correct mutex unlock order in job submission

2025-04-28 Thread Jeff Hugo
On 4/28/2025 12:47 AM, Jacek Lawrynowicz wrote: Hi, On 4/25/2025 7:22 PM, Jeff Hugo wrote: On 4/25/2025 3:36 AM, Jacek Lawrynowicz wrote: From: Karol Wachowski The mutex unlock for vdev->submitted_jobs_lock was incorrectly placed after unlocking file_priv->lock. Change order of unlocks to av

Re: [PATCH] accel/ivpu: Correct mutex unlock order in job submission

2025-04-27 Thread Jacek Lawrynowicz
Hi, On 4/25/2025 7:22 PM, Jeff Hugo wrote: > On 4/25/2025 3:36 AM, Jacek Lawrynowicz wrote: >> From: Karol Wachowski >> >> The mutex unlock for vdev->submitted_jobs_lock was incorrectly placed >> after unlocking file_priv->lock. Change order of unlocks to avoid potential > > This should read "be

Re: [PATCH] accel/ivpu: Correct mutex unlock order in job submission

2025-04-25 Thread Jeff Hugo
On 4/25/2025 3:36 AM, Jacek Lawrynowicz wrote: From: Karol Wachowski The mutex unlock for vdev->submitted_jobs_lock was incorrectly placed after unlocking file_priv->lock. Change order of unlocks to avoid potential This should read "before unlocking", right? race conditions. Fixes: 5bbccad

[PATCH] accel/ivpu: Correct mutex unlock order in job submission

2025-04-25 Thread Jacek Lawrynowicz
From: Karol Wachowski 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 Wachows