Hi Mario,
On 12/10/21 15:35, Mario Limonciello wrote:
This driver is intended to be used exclusively for suspend to idle so callbacks to send OS_HINT during hibernate and S5 will set OS_HINT at the wrong time leading to an undefined behavior.
Cc: stable@vger.kernel.org Signed-off-by: Mario Limonciello mario.limonciello@amd.com
I notice that there are no [Bug]Link tags here ? It would be helpful to have some links to tickets / forum-posts from people who are actually hitting issues because of this. Both so that people with similar issues can then compare the symptoms as described in the links, as well as for me to get an idea of how urgent of a fix this is.
Regards,
Hans
drivers/platform/x86/amd-pmc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/platform/x86/amd-pmc.c b/drivers/platform/x86/amd-pmc.c index 841c44cd64c2..230593ae5d6d 100644 --- a/drivers/platform/x86/amd-pmc.c +++ b/drivers/platform/x86/amd-pmc.c @@ -508,7 +508,8 @@ static int __maybe_unused amd_pmc_resume(struct device *dev) } static const struct dev_pm_ops amd_pmc_pm_ops = {
- SET_NOIRQ_SYSTEM_SLEEP_PM_OPS(amd_pmc_suspend, amd_pmc_resume)
- .suspend_noirq = amd_pmc_suspend,
- .resume_noirq = amd_pmc_resume,
}; static const struct pci_device_id pmc_pci_ids[] = {