Re: [PATCH v2 2/2] drm/amd: Stop evicting resources on APUs in suspend

2024-02-07 Thread Alex Deucher
On Wed, Feb 7, 2024 at 5:36 PM Mario Limonciello wrote: > > On 2/7/2024 16:34, Alex Deucher wrote: > > On Wed, Feb 7, 2024 at 3:48 PM Mario Limonciello > > wrote: > >> > >> commit 5095d5418193 ("drm/amd: Evict resources during PM ops prepare() > >> callback") > >> intentionally moved the

Re: [PATCH v2 2/2] drm/amd: Stop evicting resources on APUs in suspend

2024-02-07 Thread Mario Limonciello
On 2/7/2024 16:34, Alex Deucher wrote: On Wed, Feb 7, 2024 at 3:48 PM Mario Limonciello wrote: commit 5095d5418193 ("drm/amd: Evict resources during PM ops prepare() callback") intentionally moved the eviction of resources to earlier in the suspend process, but this introduced a subtle

Re: [PATCH v2 2/2] drm/amd: Stop evicting resources on APUs in suspend

2024-02-07 Thread Alex Deucher
On Wed, Feb 7, 2024 at 3:48 PM Mario Limonciello wrote: > > commit 5095d5418193 ("drm/amd: Evict resources during PM ops prepare() > callback") > intentionally moved the eviction of resources to earlier in the suspend > process, but this introduced a subtle change that it occurs before >

[PATCH v2 2/2] drm/amd: Stop evicting resources on APUs in suspend

2024-02-07 Thread Mario Limonciello
commit 5095d5418193 ("drm/amd: Evict resources during PM ops prepare() callback") intentionally moved the eviction of resources to earlier in the suspend process, but this introduced a subtle change that it occurs before adev->in_s0ix or adev->in_s3 are set. This meant that APUs actually started