Re: [PATCH] drm/amdgpu: fix use-after-free issue

2024-04-22 Thread Christian König
Am 22.04.24 um 13:29 schrieb Lazar, Lijo: On 4/22/2024 4:52 PM, Christian König wrote: Am 22.04.24 um 11:37 schrieb Lazar, Lijo: On 4/22/2024 2:59 PM, Christian König wrote: Am 22.04.24 um 10:47 schrieb Jack Xiao: Delete fence fallback timer to fix the ramdom use-after-free issue. That's

Re: [PATCH] drm/amdgpu: fix use-after-free issue

2024-04-22 Thread Lazar, Lijo
On 4/22/2024 4:52 PM, Christian König wrote: > Am 22.04.24 um 11:37 schrieb Lazar, Lijo: >> >> On 4/22/2024 2:59 PM, Christian König wrote: >>> Am 22.04.24 um 10:47 schrieb Jack Xiao: Delete fence fallback timer to fix the ramdom use-after-free issue. >>> That's already done in

Re: [PATCH] drm/amdgpu: fix use-after-free issue

2024-04-22 Thread Christian König
Am 22.04.24 um 11:37 schrieb Lazar, Lijo: On 4/22/2024 2:59 PM, Christian König wrote: Am 22.04.24 um 10:47 schrieb Jack Xiao: Delete fence fallback timer to fix the ramdom use-after-free issue. That's already done in amdgpu_fence_driver_hw_fini() and absolutely shouldn't be in

Re: [PATCH] drm/amdgpu: fix use-after-free issue

2024-04-22 Thread Lazar, Lijo
On 4/22/2024 2:59 PM, Christian König wrote: > Am 22.04.24 um 10:47 schrieb Jack Xiao: >> Delete fence fallback timer to fix the ramdom >> use-after-free issue. > > That's already done in amdgpu_fence_driver_hw_fini() and absolutely > shouldn't be in amdgpu_ring_fini(). > > And the

Re: [PATCH] drm/amdgpu: fix use-after-free issue

2024-04-22 Thread Christian König
Am 22.04.24 um 10:47 schrieb Jack Xiao: Delete fence fallback timer to fix the ramdom use-after-free issue. That's already done in amdgpu_fence_driver_hw_fini() and absolutely shouldn't be in amdgpu_ring_fini(). And the kfree(ring->fence_drv.fences); shouldn't be there either since that is

[PATCH] drm/amdgpu: fix use-after-free issue

2024-04-22 Thread Jack Xiao
Delete fence fallback timer to fix the ramdom use-after-free issue. Signed-off-by: Jack Xiao --- drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c index