[coreboot-gerrit] Change in coreboot[master]: soc/skylake: lock AES-NI MSR

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35188 )

Change subject: soc/skylake: lock AES-NI MSR
......................................................................

Patch Set 5:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35188/5/src/soc/intel/skylake/cpu.c 
File src/soc/intel/skylake/cpu.c:

https://review.coreboot.org/c/coreboot/+/35188/5/src/soc/intel/skylake/cpu.c... 
PS5, Line 428: MSR_FEATURE_CONFIG
Before reading the MSR, don't you need to check CPUID to determine if the MSR is really supported?