Re: [PATCH 8/9] x86/kexec: Cope with relocate_kernel() not being at the start of the page
From: David Woodhouse
Date: Mon Jan 06 2025 - 11:14:17 EST
On Mon, 2025-01-06 at 17:09 +0100, Ard Biesheuvel wrote:
>
> I suppose you're referring to
>
> #ifdef CONFIG_KEXEC_JUMP
> /* Saved in save_processor_state. */
> movq $saved_context, %rax
> lgdt saved_context_gdt_desc(%rax)
> #endif
>
> Any reason not to simply use
>
> lgdt saved_context+saved_context_gdt_desc(%rip)
>
> here?
Because the %rip isn't what you (and the linker) think it is.
This code is copied into a control page which is allocated as part of
the kexec kimage. It can only access things within that same page via
%rip. (Which is not as much of a restriction as it sounds, as for most
of its execution the rest of the kernel isn't even present in the page
tables).
Attachment:
smime.p7s
Description: S/MIME cryptographic signature