[RFC 3/6] drm/amdgpu: Fix crash on modprobe
Christian König
ckoenig.leichtzumerken at gmail.com
Tue Dec 21 07:02:56 UTC 2021
Am 20.12.21 um 20:22 schrieb Andrey Grodzovsky:
>
> On 2021-12-20 2:17 a.m., Christian König wrote:
>> Am 17.12.21 um 23:27 schrieb Andrey Grodzovsky:
>>> Restrict jobs resubmission to suspend case
>>> only since schedulers not initialised yet on
>>> probe.
>>>
>>> Signed-off-by: Andrey Grodzovsky <andrey.grodzovsky at amd.com>
>>> ---
>>> drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c
>>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c
>>> index 5527c68c51de..8ebd954e06c6 100644
>>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c
>>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c
>>> @@ -582,7 +582,7 @@ void amdgpu_fence_driver_hw_init(struct
>>> amdgpu_device *adev)
>>> if (!ring || !ring->fence_drv.initialized)
>>> continue;
>>> - if (!ring->no_scheduler) {
>>> + if (adev->in_suspend && !ring->no_scheduler) {
>>
>> Uff, why is that suddenly necessary? Because of the changed order?
>>
>> Christian.
>
>
> Yes.
Mhm, that's quite bad design then.
How about we keep the order as is and allow specifying the reset work
queue with drm_sched_start() ?
Christian.
>
> Andrey
>
>
>>
>>> drm_sched_resubmit_jobs(&ring->sched);
>>> drm_sched_start(&ring->sched, true);
>>> }
>>
More information about the amd-gfx
mailing list