Re: [PATCH] drm/radeon: fix double free

2020-07-07 Thread Alex Deucher
Applied. Thanks! Alex On Mon, Jul 6, 2020 at 8:29 AM wrote: > > From: Tom Rix > > clang static analysis flags this error > > drivers/gpu/drm/radeon/ci_dpm.c:5652:9: warning: Use of memory after it is > freed [unix.Malloc] > kfree(rdev->pm.dpm.ps[i].ps_priv); >

[PATCH] drm/radeon: fix double free

2020-07-06 Thread trix
From: Tom Rix clang static analysis flags this error drivers/gpu/drm/radeon/ci_dpm.c:5652:9: warning: Use of memory after it is freed [unix.Malloc] kfree(rdev->pm.dpm.ps[i].ps_priv); ^~ drivers/gpu/drm/radeon/ci_dpm.c:5654:2:

[PATCH] drm/radeon: fix double free in radeon_gpu_reset

2012-08-29 Thread Christian König
radeon_ring_restore is freeing the memory for the saved ring data. We need to remember that, otherwise we try to restore the ring data again on the next try. Additional to that it shouldn't try the reset infinitely if we have saved ring data. Signed-off-by: Christian K?nig ---

[PATCH] drm/radeon: fix double free in radeon_gpu_reset

2012-08-29 Thread Alex Deucher
On Wed, Aug 29, 2012 at 7:24 AM, Christian K?nig wrote: > radeon_ring_restore is freeing the memory for the saved > ring data. We need to remember that, otherwise we try to > restore the ring data again on the next try. Additional > to that it shouldn't try the reset infinitely if we have > saved

[PATCH] drm/radeon: fix double free in radeon_gpu_reset

2012-08-29 Thread Christian König
radeon_ring_restore is freeing the memory for the saved ring data. We need to remember that, otherwise we try to restore the ring data again on the next try. Additional to that it shouldn't try the reset infinitely if we have saved ring data. Signed-off-by: Christian König deathsim...@vodafone.de

Re: [PATCH] drm/radeon: fix double free in radeon_gpu_reset

2012-08-29 Thread Alex Deucher
On Wed, Aug 29, 2012 at 7:24 AM, Christian König deathsim...@vodafone.de wrote: radeon_ring_restore is freeing the memory for the saved ring data. We need to remember that, otherwise we try to restore the ring data again on the next try. Additional to that it shouldn't try the reset infinitely