drm/amdgpu: always reset the asic in suspend (v2)
authorAlex Deucher <alexander.deucher@amd.com>
Fri, 12 Nov 2021 16:25:30 +0000 (11:25 -0500)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 27 Dec 2021 17:16:34 +0000 (12:16 -0500)
If the platform suspend happens to fail and the power rail
is not turned off, the GPU will be in an unknown state on
resume, so reset the asic so that it will be in a known
good state on resume even if the platform suspend failed.

v2: handle s0ix

Acked-by: Luben Tuikov <luben.tuikov@amd.com>
Acked-by: Evan Quan <evan.quan@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c

index 73ac023..1a97b8b 100644 (file)
@@ -2154,7 +2154,10 @@ static int amdgpu_pmops_suspend(struct device *dev)
        adev->in_s3 = true;
        r = amdgpu_device_suspend(drm_dev, true);
        adev->in_s3 = false;
-
+       if (r)
+               return r;
+       if (!adev->in_s0ix)
+               r = amdgpu_asic_reset(adev);
        return r;
 }