Re: [PATCH 0/5] Some drm scheduler internal renames

[Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>]

On 05/11/2023 01:51, Luben Tuikov wrote:
> On 2023-11-02 06:55, Tvrtko Ursulin wrote:
>> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>>
>> I found some of the naming a bit incosistent and unclear so just a small
>> attempt to clarify and tidy some of them. See what people think if my first
>> stab improves things or not.
>>
>> Cc: Luben Tuikov <luben.tuikov@amd.com>
>> Cc: Matthew Brost <matthew.brost@intel.com>
>>
>> Tvrtko Ursulin (5):
>>    drm/sched: Rename drm_sched_get_cleanup_job to be more descriptive
>>    drm/sched: Move free worker re-queuing out of the if block
>>    drm/sched: Rename drm_sched_free_job_queue to be more descriptive
>>    drm/sched: Rename drm_sched_run_job_queue_if_ready and clarify
>>      kerneldoc
>>    drm/sched: Drop suffix from drm_sched_wakeup_if_can_queue
>>
>>   drivers/gpu/drm/scheduler/sched_entity.c |  4 +-
>>   drivers/gpu/drm/scheduler/sched_main.c   | 53 ++++++++++++------------
>>   include/drm/gpu_scheduler.h              |  2 +-
>>   3 files changed, 29 insertions(+), 30 deletions(-)
>>
> 
> Series is,
> 
> Reviewed-by: Luben Tuikov <ltuikov89@gmail.com>
> 
> and pushed to drm-misc-next.

Oh thanks, I definitely did not expect that to happen so quickly, 
especially since it conflicts with your fix for RR and there are some 
other opens. But it is fine, all that can be worked on top.

Regards,

Tvrtko