[PATCH 1/2] drm/amdgpu: drop non-necessary call trace dump
Christian König
ckoenig.leichtzumerken at gmail.com
Wed Jul 20 12:50:18 UTC 2022
Am 20.07.22 um 11:06 schrieb Evan Quan:
> This extra call trace dump comes out in every gpu reset.
> And it gives people a wrong impression that something
> went wrong. Although actually there was not.
>
> Signed-off-by: Evan Quan <evan.quan at amd.com>
> Change-Id: I884af405b6b3cd52b9024408a21fd39811a01f4d
Acked-by: Christian König <christian.koenig at amd.com>
And please guys don't add calls to dump_stack() in the future. That's
really not supposed to be here.
Thanks,
Christian.
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> index b79ee4ffb879..1b1a70a6da18 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> @@ -4667,7 +4667,6 @@ static int amdgpu_reset_reg_dumps(struct amdgpu_device *adev)
> int i;
>
> lockdep_assert_held(&adev->reset_domain->sem);
> - dump_stack();
>
> for (i = 0; i < adev->num_regs; i++) {
> adev->reset_dump_reg_value[i] = RREG32(adev->reset_dump_reg_list[i]);
More information about the amd-gfx
mailing list