[PATCH v2] drm/amdgpu: fix a call trace when unload amdgpu driver

[Lazar, Lijo]

On 9/4/2024 8:08 PM, Philip Yang wrote:
> 
> On 2024-09-04 04:04, Asher Song wrote:
>> In some APUs, the bo type of GART page table is ttm_bo_type_sg.
>> Those type BOs is released by bo->delayed_delete which is added in ttm_device->wq, not released immediately.
>>
>> To make sure all the ttm_resource is released before ttm_resource_manager is finilized, drain the workqueue in ttm_device.
>>
>> v2: move drain_workqueue to amdgpu_ttm.c
>>
>> Fixes:d99fbd9aab62 ("drm/ttm: Always take the bo delayed cleanup path for imported bos")
>> Suggested-by: Christian König <christian.koenig at amd.com>
>> Signed-off-by: Asher Song <Asher.Song at amd.com>
> 
> Acked-by: Philip Yang <Philip.Yang at amd.com>
> 
> Most likely this will fix another bug caused by race condition b/w GPU mode 1 reset and delayed bo cleanup worker.
> 

Unfortunately this won't - sw_fini doesn't get called during a reset.

Thanks,
Lijo

> Thank you.
> Philip
> 
>> ---
>>  drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
>> index 5c938ff0bf48..cbac21df5c47 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
>> @@ -2461,6 +2461,7 @@ void amdgpu_ttm_fini(struct amdgpu_device *adev)
>>  		drm_dev_exit(idx);
>>  	}
>>  
>> +	drain_workqueue(adev->mman.bdev.wq);
>>  	amdgpu_direct_gma_fini(adev);
>>  	amdgpu_vram_mgr_fini(adev);
>>  	amdgpu_gtt_mgr_fini(adev);