The patch titled Subject: reboot: retire hw_protection_reboot and hw_protection_shutdown helpers has been added to the -mm mm-nonmm-unstable branch. Its filename is reboot-retire-hw_protection_reboot-and-hw_protection_shutdown-helpers.patch This patch will shortly appear at https://git.kernel.org/pub/scm/linux/kernel/git/akpm/25-new.git/tree/patches/reboot-retire-hw_protection_reboot-and-hw_protection_shutdown-helpers.patch This patch will later appear in the mm-nonmm-unstable branch at git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/process/submit-checklist.rst when testing your code *** The -mm tree is included into linux-next via the mm-everything branch at git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm and is updated there every 2-3 working days ------------------------------------------------------ From: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx> Subject: reboot: retire hw_protection_reboot and hw_protection_shutdown helpers Date: Mon, 17 Feb 2025 21:39:52 +0100 The hw_protection_reboot and hw_protection_shutdown functions mix mechanism with policy: They let the driver requesting an emergency action for hardware protection also decide how to deal with it. This is inadequate in the general case as a driver reporting e.g. an imminent power failure can't know whether a shutdown or a reboot would be more appropriate for a given hardware platform. With the addition of the hw_protection parameter, it's now possible to configure at runtime the default emergency action and drivers are expected to use hw_protection_trigger to have this parameter dictate policy. As no current users of either hw_protection_shutdown or hw_protection_shutdown helpers remain, remove them, as not to tempt driver authors to call them. Existing users now either defer to hw_protection_trigger or call __hw_protection_trigger with a suitable argument directly when they have inside knowledge on whether a reboot or shutdown would be more appropriate. Link: https://lkml.kernel.org/r/20250217-hw_protection-reboot-v3-12-e1c09b090c0c@xxxxxxxxxxxxxx Signed-off-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx> Reviewed-by: Tzung-Bi Shih <tzungbi@xxxxxxxxxx> Cc: Benson Leung <bleung@xxxxxxxxxxxx> Cc: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx> Cc: Fabio Estevam <festevam@xxxxxxx> Cc: Guenter Roeck <groeck@xxxxxxxxxxxx> Cc: Jonathan Corbet <corbet@xxxxxxx> Cc: Liam Girdwood <lgirdwood@xxxxxxxxx> Cc: Lukasz Luba <lukasz.luba@xxxxxxx> Cc: Mark Brown <broonie@xxxxxxxxxx> Cc: Matteo Croce <teknoraver@xxxxxxxx> Cc: Matti Vaittinen <mazziesaccount@xxxxxxxxx> Cc: "Rafael J. Wysocki" <rafael@xxxxxxxxxx> Cc: Rob Herring (Arm) <robh@xxxxxxxxxx> Cc: Rui Zhang <rui.zhang@xxxxxxxxx> Cc: Sascha Hauer <kernel@xxxxxxxxxxxxxx> Cc: "Serge E. Hallyn" <serge@xxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- include/linux/reboot.h | 10 ---------- 1 file changed, 10 deletions(-) --- a/include/linux/reboot.h~reboot-retire-hw_protection_reboot-and-hw_protection_shutdown-helpers +++ a/include/linux/reboot.h @@ -211,16 +211,6 @@ static inline void hw_protection_trigger __hw_protection_trigger(reason, ms_until_forced, HWPROT_ACT_DEFAULT); } -static inline void hw_protection_reboot(const char *reason, int ms_until_forced) -{ - __hw_protection_trigger(reason, ms_until_forced, HWPROT_ACT_REBOOT); -} - -static inline void hw_protection_shutdown(const char *reason, int ms_until_forced) -{ - __hw_protection_trigger(reason, ms_until_forced, HWPROT_ACT_SHUTDOWN); -} - /* * Emergency restart, callable from an interrupt handler. */ _ Patches currently in -mm which might be from a.fatoum@xxxxxxxxxxxxxx are reboot-replace-__hw_protection_shutdown-bool-action-parameter-with-an-enum.patch reboot-reboot-not-shutdown-on-hw_protection_reboot-timeout.patch docs-thermal-sync-hardware-protection-doc-with-code.patch reboot-describe-do_kernel_restarts-cmd-argument-in-kernel-doc.patch reboot-rename-now-misleading-__hw_protection_shutdown-symbols.patch reboot-indicate-whether-it-is-a-hardware-protection-reboot-or-shutdown.patch reboot-add-support-for-configuring-emergency-hardware-protection-action.patch regulator-allow-user-configuration-of-hardware-protection-action.patch platform-chrome-cros_ec_lpc-prepare-for-hw_protection_shutdown-removal.patch dt-bindings-thermal-give-os-some-leeway-in-absence-of-critical-action.patch thermal-core-allow-user-configuration-of-hardware-protection-action.patch reboot-retire-hw_protection_reboot-and-hw_protection_shutdown-helpers.patch