Name of user not set #1003839 has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/57088 )
Change subject: cpu/intel: Make doubly sure we don't write to IA32_FEATURE_CTRL if its lock bit is set. ......................................................................
cpu/intel: Make doubly sure we don't write to IA32_FEATURE_CTRL if its lock bit is set.
Change-Id: Id21cc304b1e1e2871b684e287713af1c6a603efe Signed-off-by: Darius Goad mszoopers@protonmail.com --- M src/cpu/intel/model_1067x/mp_init.c 1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/88/57088/1
diff --git a/src/cpu/intel/model_1067x/mp_init.c b/src/cpu/intel/model_1067x/mp_init.c index fd6a82a..7b26875 100644 --- a/src/cpu/intel/model_1067x/mp_init.c +++ b/src/cpu/intel/model_1067x/mp_init.c @@ -59,6 +59,8 @@ printk(BIOS_DEBUG, "IA32_FEATURE_CONTROL already locked\n"); printk(BIOS_DEBUG, "SMRR status: %senabled\n", ia32_ft_ctrl.lo & (1 << 3) ? "" : "not "); + smm_relocate(); + return; } else { if (!CONFIG(SET_IA32_FC_LOCK_BIT)) printk(BIOS_INFO,