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

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31255 )

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

Patch Set 4:

(1 comment)

https://review.coreboot.org/#/c/31255/4/src/cpu/intel/model_1067x/mp_init.c 
File src/cpu/intel/model_1067x/mp_init.c:

https://review.coreboot.org/#/c/31255/4/src/cpu/intel/model_1067x/mp_init.c@... 
PS4, Line 66: printk(BIOS_DEBUG, "IA32_FEATURE_CONTROL already locked\n");
> Is that expected? If not, a lower log level like NOTICE might be useful.

It is expected on reboots at least. It is the same loglevel as cpu/intel/common/common_init.c which does something similar.