efi/libstub: arm64: Relax 2M alignment again for relocatable kernels
authorArd Biesheuvel <ardb@kernel.org>
Thu, 22 Jul 2021 10:10:31 +0000 (12:10 +0200)
committerArd Biesheuvel <ardb@kernel.org>
Tue, 3 Aug 2021 05:43:02 +0000 (07:43 +0200)
commit3a262423755b83a5f85009ace415d6e7f572dfe8
tree93763cf2704b930aebe885ee35d6f8fc45ade2af
parent5b94046efb4706b3429c9c8e7377bd8d1621d588
efi/libstub: arm64: Relax 2M alignment again for relocatable kernels

Commit 82046702e288 ("efi/libstub/arm64: Replace 'preferred' offset with
alignment check") simplified the way the stub moves the kernel image
around in memory before booting it, given that a relocatable image does
not need to be copied to a 2M aligned offset if it was loaded on a 64k
boundary by EFI.

Commit d32de9130f6c ("efi/arm64: libstub: Deal gracefully with
EFI_RNG_PROTOCOL failure") inadvertently defeated this logic by
overriding the value of efi_nokaslr if EFI_RNG_PROTOCOL is not
available, which was mistaken by the loader logic as an explicit request
on the part of the user to disable KASLR and any associated relocation
of an Image not loaded on a 2M boundary.

So let's reinstate this functionality, by capturing the value of
efi_nokaslr at function entry to choose the minimum alignment.

Fixes: d32de9130f6c ("efi/arm64: libstub: Deal gracefully with EFI_RNG_PROTOCOL failure")
Signed-off-by: Ard Biesheuvel <ardb@kernel.org>
Tested-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
drivers/firmware/efi/libstub/arm64-stub.c