mirror of
https://github.com/torvalds/linux.git
synced 2026-06-08 14:42:37 +02:00
drm/amdgpu: Fix some unload driver issues
[ Upstream commit bb0cd09be4 ]
When unloading driver after killing some applications, it will hit sdma
flush tlb job timeout which is called by ttm_bo_delay_delete. So
to avoid the job submit after fence driver fini, call ttm_bo_lock_delayed_workqueue
before fence driver fini. And also put drm_sched_fini before waiting fence.
Signed-off-by: Emily Deng <Emily.Deng@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
5b62053e76
commit
b168fffa38
|
|
@ -3551,6 +3551,7 @@ void amdgpu_device_fini(struct amdgpu_device *adev)
|
||||||
{
|
{
|
||||||
dev_info(adev->dev, "amdgpu: finishing device.\n");
|
dev_info(adev->dev, "amdgpu: finishing device.\n");
|
||||||
flush_delayed_work(&adev->delayed_init_work);
|
flush_delayed_work(&adev->delayed_init_work);
|
||||||
|
ttm_bo_lock_delayed_workqueue(&adev->mman.bdev);
|
||||||
adev->shutdown = true;
|
adev->shutdown = true;
|
||||||
|
|
||||||
kfree(adev->pci_state);
|
kfree(adev->pci_state);
|
||||||
|
|
|
||||||
|
|
@ -532,6 +532,8 @@ void amdgpu_fence_driver_fini(struct amdgpu_device *adev)
|
||||||
|
|
||||||
if (!ring || !ring->fence_drv.initialized)
|
if (!ring || !ring->fence_drv.initialized)
|
||||||
continue;
|
continue;
|
||||||
|
if (!ring->no_scheduler)
|
||||||
|
drm_sched_fini(&ring->sched);
|
||||||
r = amdgpu_fence_wait_empty(ring);
|
r = amdgpu_fence_wait_empty(ring);
|
||||||
if (r) {
|
if (r) {
|
||||||
/* no need to trigger GPU reset as we are unloading */
|
/* no need to trigger GPU reset as we are unloading */
|
||||||
|
|
@ -540,8 +542,7 @@ void amdgpu_fence_driver_fini(struct amdgpu_device *adev)
|
||||||
if (ring->fence_drv.irq_src)
|
if (ring->fence_drv.irq_src)
|
||||||
amdgpu_irq_put(adev, ring->fence_drv.irq_src,
|
amdgpu_irq_put(adev, ring->fence_drv.irq_src,
|
||||||
ring->fence_drv.irq_type);
|
ring->fence_drv.irq_type);
|
||||||
if (!ring->no_scheduler)
|
|
||||||
drm_sched_fini(&ring->sched);
|
|
||||||
del_timer_sync(&ring->fence_drv.fallback_timer);
|
del_timer_sync(&ring->fence_drv.fallback_timer);
|
||||||
for (j = 0; j <= ring->fence_drv.num_fences_mask; ++j)
|
for (j = 0; j <= ring->fence_drv.num_fences_mask; ++j)
|
||||||
dma_fence_put(ring->fence_drv.fences[j]);
|
dma_fence_put(ring->fence_drv.fences[j]);
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue
Block a user