[PATCH] drm/scheduler: fix drm_sched_get_cleanup_job

Christian König christian.koenig at amd.com
Wed Apr 15 10:35:59 UTC 2020


Sorry for the holiday/vacation/COVID-19 delay. I've just pushed this 
patch into drm-misc-fixes.

I assume it already landed in our internal branches?

Thanks,
Christian.

Am 14.04.20 um 16:33 schrieb Andrey Grodzovsky:
> Reviewed-by: Andrey Grodzovsky <andrey.grodzovsky at amd.com>
>
> Andrey
>
> On 4/14/20 10:22 AM, Kent Russell wrote:
>> From: Christian König <christian.koenig at amd.com>
>>
>> We are racing to initialize sched->thread here, just always check the
>> current thread.
>>
>> Signed-off-by: Christian Koenig <christian.koenig at amd.com>
>> Reviewed-by: Kent Russell <kent.russell at amd.com>
>> ---
>>   drivers/gpu/drm/scheduler/sched_main.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/scheduler/sched_main.c 
>> b/drivers/gpu/drm/scheduler/sched_main.c
>> index 8e731ed0d9d9..2f319102ae9f 100644
>> --- a/drivers/gpu/drm/scheduler/sched_main.c
>> +++ b/drivers/gpu/drm/scheduler/sched_main.c
>> @@ -676,7 +676,7 @@ drm_sched_get_cleanup_job(struct 
>> drm_gpu_scheduler *sched)
>>        */
>>       if ((sched->timeout != MAX_SCHEDULE_TIMEOUT &&
>>           !cancel_delayed_work(&sched->work_tdr)) ||
>> -        __kthread_should_park(sched->thread))
>> +        kthread_should_park())
>>           return NULL;
>>         spin_lock(&sched->job_list_lock);



More information about the amd-gfx mailing list