Commit dc6ffa6c authored by Rafael J. Wysocki's avatar Rafael J. Wysocki Committed by Borislav Petkov (AMD)
Browse files

kexec_core: Add and update comments regarding the KEXEC_JUMP flow 



The KEXEC_JUMP flow is analogous to hibernation flows occurring before
and after creating an image and before and after jumping from the
restore kernel to the image one, which is why it uses the same device
callbacks as those hibernation flows.

Add comments explaining that to the code in question and update an
existing comment in it which appears a bit out of context.

No functional changes.

Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: default avatarDavid Woodhouse <dwmw@amazon.co.uk>
Signed-off-by: default avatarBorislav Petkov (AMD) <bp@alien8.de>
Link: https://lore.kernel.org/r/20250109140757.2841269-8-dwmw2@infradead.org
parent 2114796c

kernel/kexec_core.c

+17 −6
Original line number Diff line number Diff line
@@ -1001,6 +1001,12 @@ int kernel_kexec(void) 


#ifdef CONFIG_KEXEC_JUMP

	if (kexec_image->preserve_context) {

		/*

		 * This flow is analogous to hibernation flows that occur

		 * before creating an image and before jumping from the

		 * restore kernel to the image one, so it uses the same

		 * device callbacks as those two flows.

		 */

		pm_prepare_console();

		error = freeze_processes();

		if (error) {
@@ -1011,12 +1017,10 @@ int kernel_kexec(void) 
		error = dpm_suspend_start(PMSG_FREEZE);

		if (error)

			goto Resume_console;

		/* At this point, dpm_suspend_start() has been called,

		 * but *not* dpm_suspend_end(). We *must* call

		 * dpm_suspend_end() now.  Otherwise, drivers for

		 * some devices (e.g. interrupt controllers) become

		 * desynchronized with the actual state of the

		 * hardware at resume time, and evil weirdness ensues.

		/*

		 * dpm_suspend_end() must be called after dpm_suspend_start()

		 * to complete the transition, like in the hibernation flows

		 * mentioned above.

		 */

		error = dpm_suspend_end(PMSG_FREEZE);

		if (error)
@@ -1052,6 +1056,13 @@ int kernel_kexec(void) 


#ifdef CONFIG_KEXEC_JUMP

	if (kexec_image->preserve_context) {

		/*

		 * This flow is analogous to hibernation flows that occur after

		 * creating an image and after the image kernel has got control

		 * back, and in case the devices have been reset or otherwise

		 * manipulated in the meantime, it uses the device callbacks

		 * used by the latter.

		 */

		syscore_resume();

 Enable_irqs:

		local_irq_enable();