David Guckian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/25431 )
Change subject: soc/intel/denverton_ns: Enable Fast Strings
......................................................................
Patch Set 7:
As per Intel Architecture SW Developers Manual:
Vol. 1, Section 7.3.9.3
Fast-String Operation
To improve performance, more recent processors support modifications to the processor’s operation during the string store operations initiated with the MOVS, MOVSB, STOS, and STOSB instructions. This optimized operation, called fast-string operation...
Software can disable fast-string operation by clearing the fast-string-enable bit (bit 0) of IA32_MISC_ENABLE MSR. However, Intel recommends that system software always enable fast-string operation.
It's a short paragraph, but a bit long to add all of it here I think.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/25431
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I7cee3c40299abf14a24128b1ac14f1823f87a0e1
Gerrit-Change-Number: 25431
Gerrit-PatchSet: 7
Gerrit-Owner: Julien Viard de Galbert
coreboot-review-ju@vdg.name
Gerrit-Reviewer: David Guckian
Gerrit-Reviewer: David Guckian
david.guckian@intel.com
Gerrit-Reviewer: Jay Talbott
JayTalbott@sysproconsulting.com
Gerrit-Reviewer: Julien Viard de Galbert
coreboot-review-ju@vdg.name
Gerrit-Reviewer: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Reviewer: Shine Liu
shine.liu@intel.com
Gerrit-Reviewer: Vanny E
vanessa.f.eusebio@intel.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Angel Pons
th3fanbus@gmail.com
Gerrit-Comment-Date: Wed, 16 Jan 2019 12:20:24 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment