<p>Matt DeVillier <strong>uploaded patch set #3</strong> to this change.</p><p><a href="https://review.coreboot.org/c/coreboot/+/30229">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">cpu/intel/common: decouple IA32_FEATURE_CONTROL lock from set_vmx()<br><br>Newer CPUs/SoCs need to configure other features via the<br>IA32_FEATURE_CONTROL msr, such as SGX, which cannot be done if the<br>msr is already locked. Create separate functions for setting the<br>vmx flag and lock bit, and rename existing function to indicate that<br>the lock bit will be set in addition to vmx flag (per Kconfig).<br><br>This will allow Skylake/Kabylake (and others?) to use the common<br>VMX code without breaking SGX, while ensuring no change in functionality<br>to existing platforms which current set both together.<br><br>Test: build/boot each affected platform, ensure no change in functionality<br><br>Change-Id: Iee772fe87306b4729ca012cef8640d3858e2cb06<br>Signed-off-by: Matt DeVillier <matt.devillier@gmail.com><br>---<br>M src/cpu/intel/common/Kconfig<br>M src/cpu/intel/common/common.h<br>M src/cpu/intel/common/common_init.c<br>M src/cpu/intel/fsp_model_406dx/model_406dx_init.c<br>M src/cpu/intel/haswell/haswell_init.c<br>M src/cpu/intel/model_1067x/model_1067x_init.c<br>M src/cpu/intel/model_106cx/model_106cx_init.c<br>M src/cpu/intel/model_2065x/model_2065x_init.c<br>M src/cpu/intel/model_206ax/model_206ax_init.c<br>M src/cpu/intel/model_6ex/model_6ex_init.c<br>M src/cpu/intel/model_6fx/model_6fx_init.c<br>M src/soc/intel/baytrail/cpu.c<br>M src/soc/intel/braswell/cpu.c<br>M src/soc/intel/broadwell/cpu.c<br>14 files changed, 46 insertions(+), 22 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/29/30229/3</pre><p>To view, visit <a href="https://review.coreboot.org/c/coreboot/+/30229">change 30229</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/c/coreboot/+/30229"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Iee772fe87306b4729ca012cef8640d3858e2cb06 </div>
<div style="display:none"> Gerrit-Change-Number: 30229 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Matt DeVillier <matt.devillier@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: David Guckian <david.guckian@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Matt DeVillier <matt.devillier@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Nico Huber <nico.h@gmx.de> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>