* FAILED: patch "[PATCH] platform/x86: amd-pmc: only use callbacks for suspend" failed to apply to 5.16-stable tree
@ 2022-01-23 14:14 gregkh
2022-01-24 21:00 ` Limonciello, Mario
0 siblings, 1 reply; 2+ messages in thread
From: gregkh @ 2022-01-23 14:14 UTC (permalink / raw)
To: mario.limonciello, hdegoede; +Cc: stable
The patch below does not apply to the 5.16-stable tree.
If someone wants it applied there, or to any other stable or longterm
tree, then please email the backport, including the original git commit
id to <stable@vger.kernel.org>.
thanks,
greg k-h
------------------ original commit in Linus's tree ------------------
From d386f7ef9f410266bc1f364ad6a11cb28dae09a8 Mon Sep 17 00:00:00 2001
From: Mario Limonciello <mario.limonciello@amd.com>
Date: Fri, 10 Dec 2021 08:35:29 -0600
Subject: [PATCH] platform/x86: amd-pmc: only use callbacks for suspend
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>
Link: https://lore.kernel.org/r/20211210143529.10594-1-mario.limonciello@amd.com
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
diff --git a/drivers/platform/x86/amd-pmc.c b/drivers/platform/x86/amd-pmc.c
index c709ff993e8b..f794343d6aaa 100644
--- a/drivers/platform/x86/amd-pmc.c
+++ b/drivers/platform/x86/amd-pmc.c
@@ -585,7 +585,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[] = {
^ permalink raw reply related [flat|nested] 2+ messages in thread
* RE: FAILED: patch "[PATCH] platform/x86: amd-pmc: only use callbacks for suspend" failed to apply to 5.16-stable tree
2022-01-23 14:14 FAILED: patch "[PATCH] platform/x86: amd-pmc: only use callbacks for suspend" failed to apply to 5.16-stable tree gregkh
@ 2022-01-24 21:00 ` Limonciello, Mario
0 siblings, 0 replies; 2+ messages in thread
From: Limonciello, Mario @ 2022-01-24 21:00 UTC (permalink / raw)
To: gregkh, hdegoede; +Cc: stable
[Public]
It looks to me that this was already applied for both 5.15.y and 5.16.y, so that's why it won't be able to apply again, no concerns.
$ git log --oneline stable/linux-5.16.y drivers/platform/x86/amd-pmc.c | grep "only use callbacks for suspend"
09fc14061f3e platform/x86: amd-pmc: only use callbacks for suspend
$ git log --oneline stable/linux-5.15.y drivers/platform/x86/amd-pmc.c | grep "only use callbacks for suspend"
a42c41be8324 platform/x86: amd-pmc: only use callbacks for suspend
> -----Original Message-----
> From: gregkh@linuxfoundation.org <gregkh@linuxfoundation.org>
> Sent: Sunday, January 23, 2022 08:15
> To: Limonciello, Mario <Mario.Limonciello@amd.com>; hdegoede@redhat.com
> Cc: stable@vger.kernel.org
> Subject: FAILED: patch "[PATCH] platform/x86: amd-pmc: only use callbacks for
> suspend" failed to apply to 5.16-stable tree
>
>
> The patch below does not apply to the 5.16-stable tree.
> If someone wants it applied there, or to any other stable or longterm
> tree, then please email the backport, including the original git commit
> id to <stable@vger.kernel.org>.
>
> thanks,
>
> greg k-h
>
> ------------------ original commit in Linus's tree ------------------
>
> From d386f7ef9f410266bc1f364ad6a11cb28dae09a8 Mon Sep 17 00:00:00 2001
> From: Mario Limonciello <mario.limonciello@amd.com>
> Date: Fri, 10 Dec 2021 08:35:29 -0600
> Subject: [PATCH] platform/x86: amd-pmc: only use callbacks for suspend
>
> 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>
> Link:
> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flore.ker
> nel.org%2Fr%2F20211210143529.10594-1-
> mario.limonciello%40amd.com&data=04%7C01%7Cmario.limonciello%40a
> md.com%7Cec1064b7e16b4b48609208d9de7ab928%7C3dd8961fe4884e608e11
> a82d994e183d%7C0%7C0%7C637785440946508125%7CUnknown%7CTWFpbGZ
> sb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0
> %3D%7C3000&sdata=Wbi1ZGIBtIaQAmTiYhLNSb%2BtnHQG93PkLtG2OPN
> NR%2Bg%3D&reserved=0
> Reviewed-by: Hans de Goede <hdegoede@redhat.com>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
>
> diff --git a/drivers/platform/x86/amd-pmc.c b/drivers/platform/x86/amd-pmc.c
> index c709ff993e8b..f794343d6aaa 100644
> --- a/drivers/platform/x86/amd-pmc.c
> +++ b/drivers/platform/x86/amd-pmc.c
> @@ -585,7 +585,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[] = {
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-01-24 21:05 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-23 14:14 FAILED: patch "[PATCH] platform/x86: amd-pmc: only use callbacks for suspend" failed to apply to 5.16-stable tree gregkh
2022-01-24 21:00 ` Limonciello, Mario
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).