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

[Luben Tuikov <ltuikov89@gmail.com>]

[-- Attachment #1.1.1: Type: text/plain, Size: 1675 bytes --]

On 2023-11-06 07:41, Tvrtko Ursulin wrote:
> 
> 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.

Yeah, it does conflict, and it does make some changes obsolete,
but your series was fine and an improvement, so might as well push it.

I'll rebase my patch on top of yours.
-- 
Regards,
Luben

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 677 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]