<p><a href="https://review.coreboot.org/27870">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/27870/2/src/cpu/intel/smm/gen1/smmrelocate.c">File src/cpu/intel/smm/gen1/smmrelocate.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27870/2/src/cpu/intel/smm/gen1/smmrelocate.c@180">Patch Set #2, Line 180:</a> <code style="font-family:monospace,monospace">BIOS_WARNING</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">warning or error?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">looking at the description of both, it feels like BIOS_WARNING is better since it is expected to still work if SMRR is not set. Not sure to what extend TSEG is accessible accessible with SMRR being set, so it is a security issue at most.</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/27870">change 27870</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/27870"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I48f55cdac5f4b16b9a8d7a8ef3a84918e756e315 </div>
<div style="display:none"> Gerrit-Change-Number: 27870 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Arthur Heymans <arthur@aheymans.xyz> </div>
<div style="display:none"> Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz> </div>
<div style="display:none"> Gerrit-Reviewer: Felix Held <felix-coreboot@felixheld.de> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 07 Aug 2018 13:25:18 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>