ACK: [SRU][OEM-5.14/OEM-5.17/J][PATCH 0/2] Fix system hangs after s2idle on AMD A+A GPU

[Tim Gardner]

Acked-by: Tim Gardner <tim.gardner at canonical.com>

On 5/26/22 02:34, Aaron Ma wrote:
> BugLink: https://bugs.launchpad.net/bugs/1975804
> 
> [Impact]
> Sytesm may hang after s2idle on AMD A+A GPU config due to the following
> commits:
> 8b328c64c7082278c888927f00e526786eec2880 ("drm/amdgpu: don't use BACO
> for reset in S3") https://bugs.launchpad.net/bugs/1968475
> a26d80ba0d9e67ea11cbfc25618320163497d3fe ("drm/amd/pm: keep the BACO
> feature enabled for suspend") https://bugs.launchpad.net/bugs/1958371
> 
> [Fix]
> Revert one commit and don't reset dGPU when s2idle
> 
> [Test]
> Verified on AMD RMB, stress s2idle passed.
> 
> [Where problems could occur]
> Low risk, it may break s2idle on AMD platform.
> 
> Alex Deucher (1):
>    Revert "drm/amd/pm: keep the BACO feature enabled for suspend"
> 
> Mario Limonciello (1):
>    drm/amd: Don't reset dGPUs if the system is going to s2idle
> 
>   drivers/gpu/drm/amd/amdgpu/amdgpu.h       |  2 ++
>   drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c  | 14 ++++++++++++++
>   drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c   |  2 +-
>   drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c |  8 +-------
>   4 files changed, 18 insertions(+), 8 deletions(-)
> 

-- 
-----------
Tim Gardner
Canonical, Inc