x86: Fix .brk attribute in linker script
authorJuergen Gross <jgross@suse.com>
Thu, 30 Jun 2022 07:14:41 +0000 (09:14 +0200)
committerBorislav Petkov <bp@suse.de>
Fri, 1 Jul 2022 09:12:43 +0000 (11:12 +0200)
commit7e09ac27f43b382f5fe9bb7c7f4c465ece1f8a23
treea78ec4ad54498d4f57fba9dd167ebfc6db3a86ef
parent38fa5479b41376dc9d7f57e71c83514285a25ca0
x86: Fix .brk attribute in linker script

Commit in Fixes added the "NOLOAD" attribute to the .brk section as a
"failsafe" measure.

Unfortunately, this leads to the linker no longer covering the .brk
section in a program header, resulting in the kernel loader not knowing
that the memory for the .brk section must be reserved.

This has led to crashes when loading the kernel as PV dom0 under Xen,
but other scenarios could be hit by the same problem (e.g. in case an
uncompressed kernel is used and the initrd is placed directly behind
it).

So drop the "NOLOAD" attribute. This has been verified to correctly
cover the .brk section by a program header of the resulting ELF file.

Fixes: e32683c6f7d2 ("x86/mm: Fix RESERVE_BRK() for older binutils")
Signed-off-by: Juergen Gross <jgross@suse.com>
Signed-off-by: Borislav Petkov <bp@suse.de>
Reviewed-by: Josh Poimboeuf <jpoimboe@kernel.org>
Link: https://lore.kernel.org/r/20220630071441.28576-4-jgross@suse.com
arch/x86/kernel/vmlinux.lds.S