On Mon, Apr 15, 2019 at 1:04 PM Pavel Machek pavel@denx.de wrote:
On Mon 2019-04-15 20:58:23, Greg Kroah-Hartman wrote:
[ Upstream commit 7ee18d677989e99635027cee04c878950e0752b9 ]
My previous attempt to fix a couple of bugs in __restore_processor_context():
5b06bbcfc2c6 ("x86/power: Fix some ordering bugs in __restore_processor_context()")
... introduced yet another bug, breaking suspend-resume.
Rather than trying to come up with a minimal fix, let's try to clean it up for real. This patch fixes quite a few things:
5b06bbcfc2c6 fixed theoretical bug; rather than porting it to stable than fixing it up, it would be better not to port it to stable in the first place or simply revert it there.
Are you sure about that? The bug was reported by real users who had their systems really crash:
https://lore.kernel.org/lkml/?q=0fede9f9-88b0-a6e7-1027-dfb2019b8ef2%40linux...
https://lore.kernel.org/lkml/CA+55aFwsMuHUBQz5kDNwRf17JnasXMWjvmLq5qXGH-694y...
And we had a report that the bug got backported:
https://lore.kernel.org/stable/20190407160005.djiw4reapwvbxmgo@debian/
And if we're going to backport some of the fix, we should definitely backport the whole set to avoid having the -stable kernels be in a state that was never in any released kernel.
--Andy https://lore.kernel.org/stable/20190407160005.djiw4reapwvbxmgo@debian/