[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/cannonlake: select VMX configurabiltity

[Arthur Heymans (Code Review)]

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

Change subject: soc/intel/cannonlake: select VMX configurabiltity
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/#/c/31442/2/src/soc/intel/cannonlake/cpu.c 
File src/soc/intel/cannonlake/cpu.c:

https://review.coreboot.org/#/c/31442/2/src/soc/intel/cannonlake/cpu.c@464 
PS2, Line 464: mp_run_on_all_cpus(vmx_configure, NULL, 2 * USECS_PER_MSEC);
             : }
could you explain in the commit message and here in the code why this is run here and not simply during cpu_driver ops? Also why is the wheel reinvented with vmx_configure in soc/intel while there has been code that does the more or less the same in cpu/intel/common?