diff options
| author | Alex Deucher <[email protected]> | 2025-09-04 16:35:05 +0000 |
|---|---|---|
| committer | Alex Deucher <[email protected]> | 2025-09-09 20:38:26 +0000 |
| commit | 7838fb5f119191403560eca2e23613380c0e425e (patch) | |
| tree | 9621f5869499db0a90533017a347b608803d6ee6 /drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c | |
| parent | amd/amdkfd: correct mem limit calculation for small APUs (diff) | |
| download | kernel-7838fb5f119191403560eca2e23613380c0e425e.tar.gz kernel-7838fb5f119191403560eca2e23613380c0e425e.zip | |
drm/amdgpu: fix a memory leak in fence cleanup when unloading
Commit b61badd20b44 ("drm/amdgpu: fix usage slab after free")
reordered when amdgpu_fence_driver_sw_fini() was called after
that patch, amdgpu_fence_driver_sw_fini() effectively became
a no-op as the sched entities we never freed because the
ring pointers were already set to NULL. Remove the NULL
setting.
Reported-by: Lin.Cao <[email protected]>
Cc: Vitaly Prosyak <[email protected]>
Cc: Christian König <[email protected]>
Fixes: b61badd20b44 ("drm/amdgpu: fix usage slab after free")
Reviewed-by: Christian König <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
(cherry picked from commit a525fa37aac36c4591cc8b07ae8957862415fbd5)
Cc: [email protected]
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c')
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c index 6379bb25bf5c..486c3646710c 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c @@ -421,8 +421,6 @@ void amdgpu_ring_fini(struct amdgpu_ring *ring) dma_fence_put(ring->vmid_wait); ring->vmid_wait = NULL; ring->me = 0; - - ring->adev->rings[ring->idx] = NULL; } /** |