diff options
author | Arthur Heymans <arthur@aheymans.xyz> | 2024-02-02 18:35:00 +0100 |
---|---|---|
committer | Martin L Roth <gaumless@gmail.com> | 2024-03-09 23:15:34 +0000 |
commit | 7a51acfbe91c7f9d01837103341526abb6ea46f4 (patch) | |
tree | 2453f8188d5e1c8d5c160972a1ab4edcff9ca844 /src/cpu/x86/mp_init.c | |
parent | 1879b6a34a6e93a93d691a0d9f2457d6251a17c1 (diff) |
cpu/x86/smm: Set up page tables in safe SMRAM
Relying on page tables being in RO flash is not safe in every setup,
therefore set up some page tables in SMRAM that the permanent smihandler
can use.
Tested on QEMU.
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Change-Id: Icb3086abd577b9abb9966dd910a264a873ace4ed
Reviewed-on: https://review.coreboot.org/c/coreboot/+/80336
Reviewed-by: Patrick Rudolph <patrick.rudolph@9elements.com>
Reviewed-by: Benjamin Doron <benjamin.doron00@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/cpu/x86/mp_init.c')
-rw-r--r-- | src/cpu/x86/mp_init.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/cpu/x86/mp_init.c b/src/cpu/x86/mp_init.c index b336e9f2d1..dd5e94d4f2 100644 --- a/src/cpu/x86/mp_init.c +++ b/src/cpu/x86/mp_init.c @@ -790,7 +790,6 @@ static enum cb_err install_permanent_handler(int num_cpus, uintptr_t smbase, .num_cpus = num_cpus, .cpu_save_state_size = save_state_size, .num_concurrent_save_states = num_cpus, - .cr3 = read_cr3(), }; printk(BIOS_DEBUG, "Installing permanent SMM handler to 0x%08lx\n", smbase); |