[Public] No, it mode2 reset that it uses for failure case.
From: Lazar, Lijo <lijo.la...@amd.com> Sent: Tuesday, May 17, 2022 11:51 To: Limonciello, Mario <mario.limoncie...@amd.com>; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amd: Don't reset dGPUs if the system is going to s2idle [Public] Ya, second is too lengthy. Better to leave it as it is. BTW, is this specific to reset by BACO? BACO entry/exit may take longer (better chance of suspend entry abort by some wake-up source). Thanks, Lijo ________________________________ From: Limonciello, Mario <mario.limoncie...@amd.com<mailto:mario.limoncie...@amd.com>> Sent: Tuesday, May 17, 2022 10:00:16 PM To: Lazar, Lijo <lijo.la...@amd.com<mailto:lijo.la...@amd.com>>; amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org> <amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org>> Subject: RE: [PATCH] drm/amd: Don't reset dGPUs if the system is going to s2idle [Public] > PM_SUSPEND_TO_IDLE should be under a compile guard It is actually. All of the amdgpu_acpi_* are. It's not obvious though looking at the patch, you need to apply it to notice it. > It makes sense to rename to something like amdgpu_need_reset_on_suspend() as > it decides on reset only for a suspend situation. Remember it's in amdgpu_acpi.c. So it would be amdgpu_acpi_need_reset_on_suspend()? It's a bit lengthy. From: Lazar, Lijo <lijo.la...@amd.com<mailto:lijo.la...@amd.com>> Sent: Tuesday, May 17, 2022 11:25 To: Limonciello, Mario <mario.limoncie...@amd.com<mailto:mario.limoncie...@amd.com>>; amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org> Cc: Limonciello, Mario <mario.limoncie...@amd.com<mailto:mario.limoncie...@amd.com>> Subject: Re: [PATCH] drm/amd: Don't reset dGPUs if the system is going to s2idle [Public] A couple of things - PM_SUSPEND_TO_IDLE should be under a compile guard It makes sense to rename to something like amdgpu_need_reset_on_suspend() as it decides on reset only for a suspend situation. Thanks, Lijo