x86/efistub: Remap kernel text read-only before dropping NX attribute
authorArd Biesheuvel <ardb@kernel.org>
Thu, 25 Jan 2024 13:32:07 +0000 (14:32 +0100)
committerArd Biesheuvel <ardb@kernel.org>
Sat, 9 Mar 2024 10:37:18 +0000 (11:37 +0100)
commit9c55461040a9264b7e44444c53d26480b438eda6
tree87cbfd816dc4a7b9bcae5ae13e0ec2fe0dd745ed
parentd228814b1913444dfdd9a25519ed7b38a19653e2
x86/efistub: Remap kernel text read-only before dropping NX attribute

Currently, the EFI stub invokes the EFI memory attributes protocol to
strip any NX restrictions from the entire loaded kernel, resulting in
all code and data being mapped read-write-execute.

The point of the EFI memory attributes protocol is to remove the need
for all memory allocations to be mapped with both write and execute
permissions by default, and make it the OS loader's responsibility to
transition data mappings to code mappings where appropriate.

Even though the UEFI specification does not appear to leave room for
denying memory attribute changes based on security policy, let's be
cautious and avoid relying on the ability to create read-write-execute
mappings. This is trivially achievable, given that the amount of kernel
code executing via the firmware's 1:1 mapping is rather small and
limited to the .head.text region. So let's drop the NX restrictions only
on that subregion, but not before remapping it as read-only first.

Signed-off-by: Ard Biesheuvel <ardb@kernel.org>
arch/x86/boot/compressed/Makefile
arch/x86/boot/compressed/misc.c
arch/x86/include/asm/boot.h
drivers/firmware/efi/libstub/x86-stub.c