Re: [PATCH v2] drm/sched: Avoid double re-lock on the job free path

2025-07-14 Thread Tvrtko Ursulin
On 12/07/2025 14:12, Maíra Canal wrote: Hi Danilo, On 7/11/25 16:22, Danilo Krummrich wrote: On 7/11/25 9:08 PM, Maíra Canal wrote: Hi Tvrtko, On 11/07/25 12:09, Tvrtko Ursulin wrote: Currently the job free work item will lock sched->job_list_lock first time to see if there are any jobs,

Re: [PATCH v2] drm/sched: Avoid double re-lock on the job free path

2025-07-12 Thread Maíra Canal
Hi Danilo, On 7/11/25 16:22, Danilo Krummrich wrote: On 7/11/25 9:08 PM, Maíra Canal wrote: Hi Tvrtko, On 11/07/25 12:09, Tvrtko Ursulin wrote: Currently the job free work item will lock sched->job_list_lock first time to see if there are any jobs, free a single job, and then lock again to d

Re: [PATCH v2] drm/sched: Avoid double re-lock on the job free path

2025-07-11 Thread Danilo Krummrich
On 7/11/25 9:08 PM, Maíra Canal wrote: Hi Tvrtko, On 11/07/25 12:09, Tvrtko Ursulin wrote: Currently the job free work item will lock sched->job_list_lock first time to see if there are any jobs, free a single job, and then lock again to decide whether to re-queue itself if there are more finis

Re: [PATCH v2] drm/sched: Avoid double re-lock on the job free path

2025-07-11 Thread Maíra Canal
Hi Tvrtko, On 11/07/25 12:09, Tvrtko Ursulin wrote: Currently the job free work item will lock sched->job_list_lock first time to see if there are any jobs, free a single job, and then lock again to decide whether to re-queue itself if there are more finished jobs. Since drm_sched_get_finished_

[PATCH v2] drm/sched: Avoid double re-lock on the job free path

2025-07-11 Thread Tvrtko Ursulin
Currently the job free work item will lock sched->job_list_lock first time to see if there are any jobs, free a single job, and then lock again to decide whether to re-queue itself if there are more finished jobs. Since drm_sched_get_finished_job() already looks at the second job in the queue we c

[PATCH v2] drm/sched: Avoid double re-lock on the job free path

2025-01-16 Thread Tvrtko Ursulin
Currently the job free work item will lock sched->job_list_lock first time to see if there are any jobs, free a single job, and then lock again to decide whether to re-queue itself if there are more finished jobs. Since drm_sched_get_finished_job() already looks at the second job in the queue we c