diff options
| author |   Juergen Gross <jgross@suse.com> | 2022-06-30 09:14:40 +0200 |
|---|---|---|
| committer |   Borislav Petkov <bp@suse.de> | 2022-07-01 11:11:34 +0200 |
| commit | 38fa5479b41376dc9d7f57e71c83514285a25ca0 (patch) | |
| tree | 121800fdca14dd13f8787bb13c559a738688b1e6 | |
| parent | x86/xen: Use clear_bss() for Xen PV guests (diff) | |
| download | linux-dev-38fa5479b41376dc9d7f57e71c83514285a25ca0.tar.xz linux-dev-38fa5479b41376dc9d7f57e71c83514285a25ca0.zip | |
x86: Clear .brk area at early boot
The .brk section has the same properties as .bss: it is an alloc-only
section and should be cleared before being used.
Not doing so is especially a problem for Xen PV guests, as the
hypervisor will validate page tables (check for writable page tables
and hypervisor private bits) before accepting them to be used.
Make sure .brk is initially zero by letting clear_bss() clear the brk
area, too.
Signed-off-by: Juergen Gross <jgross@suse.com>
Signed-off-by: Borislav Petkov <bp@suse.de>
Link: https://lore.kernel.org/r/20220630071441.28576-3-jgross@suse.com
| -rw-r--r-- | arch/x86/kernel/head64.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/arch/x86/kernel/head64.c b/arch/x86/kernel/head64.c index e7e233209a8c..6a3cfaf6b72a 100644 --- a/arch/x86/kernel/head64.c +++ b/arch/x86/kernel/head64.c @@ -430,6 +430,8 @@ void __init clear_bss(void) { memset(__bss_start, 0, (unsigned long) __bss_stop - (unsigned long) __bss_start); + memset(__brk_base, 0, + (unsigned long) __brk_limit - (unsigned long) __brk_base); } static unsigned long get_cmd_line_ptr(void) |