]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
PM: hibernate: Restrict GFP mask in hibernation_snapshot()
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 10 Sep 2025 09:41:59 +0000 (11:41 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 19 Sep 2025 14:37:30 +0000 (16:37 +0200)
commit 449c9c02537a146ac97ef962327a221e21c9cab3 upstream.

Commit 12ffc3b1513e ("PM: Restrict swap use to later in the suspend
sequence") incorrectly removed a pm_restrict_gfp_mask() call from
hibernation_snapshot(), so memory allocations involving swap are not
prevented from being carried out in this code path any more which may
lead to serious breakage.

The symptoms of such breakage have become visible after adding a
shrink_shmem_memory() call to hibernation_snapshot() in commit
2640e819474f ("PM: hibernate: shrink shmem pages after dev_pm_ops.prepare()")
which caused this problem to be much more likely to manifest itself.

However, since commit 2640e819474f was initially present in the DRM
tree that did not include commit 12ffc3b1513e, the symptoms of this
issue were not visible until merge commit 260f6f4fda93 ("Merge tag
'drm-next-2025-07-30' of https://gitlab.freedesktop.org/drm/kernel")
that exposed it through an entirely reasonable merge conflict
resolution.

Fixes: 12ffc3b1513e ("PM: Restrict swap use to later in the suspend sequence")
Closes: https://bugzilla.kernel.org/show_bug.cgi?id=220555
Reported-by: Todd Brandt <todd.e.brandt@linux.intel.com>
Tested-by: Todd Brandt <todd.e.brandt@linux.intel.com>
Cc: 6.16+ <stable@vger.kernel.org> # 6.16+
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Reviewed-by: Mario Limonciello (AMD) <superm1@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
kernel/power/hibernate.c

index 9216e3b91d3b3bfdabdbf161da10ee7d6ec74c46..c8022a477d3a1cd2a974a5f51abb948b73ce8e59 100644 (file)
@@ -423,6 +423,7 @@ int hibernation_snapshot(int platform_mode)
        }
 
        console_suspend_all();
+       pm_restrict_gfp_mask();
 
        error = dpm_suspend(PMSG_FREEZE);