[coreboot-gerrit] Change in ...coreboot[master]: cpu/intel/model_1067x: Check for lock bit on IA32_FEATURE_CONTROL

[Arthur Heymans (Code Review)]

Hello Patrick Rudolph, build bot (Jenkins),

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/31255

to look at the new patch set (#2).

Change subject: cpu/intel/model_1067x: Check for lock bit on IA32_FEATURE_CONTROL
......................................................................

cpu/intel/model_1067x: Check for lock bit on IA32_FEATURE_CONTROL

df7aecd " cpu/intel: Configure IA32_FEATURE_CONTROL for alternative
SMRR" introduced a regression because it unconditionally writes to
IA32_FEATURE_CONTROL, which if it is already locked results in an
unhandled exception. The lock bit is already set on a system reboot.

Change-Id: I7d2df9e1b9d767809da7a61ccd877c6c40f132eb
Signed-off-by: Arthur Heymans 
---
M src/cpu/intel/model_1067x/mp_init.c
1 file changed, 14 insertions(+), 5 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/55/31255/2