Attention is currently required from: Arthur Heymans, Julius Werner, Paul Menzel, Subrata Banik.
Hello Julius Werner, Subrata Banik, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/70175?usp=email
to look at the new patch set (#4).
The following approvals got outdated and were removed: Code-Review+2 by Julius Werner, Code-Review+2 by Subrata Banik, Verified+1 by build bot (Jenkins)
Change subject: lib/lzmadecode: Allow for 8 byte reads on 64bit ......................................................................
lib/lzmadecode: Allow for 8 byte reads on 64bit
This adds an optimization to lzma decode to also read from the boot medium in chunks of 8 bytes if that is the general purpose register length instead of always 4 bytes. It depends on the cache / memory / spi controller whether this is faster, but it's likely to be either the same or faster.
TESTED - google/vilboz: cached boot medium 64bit before - 32bit - 64bit after load FSP-M: 35,674 - 35,595 - 34,690 load ramstage: 42,134 - 43,378 - 40,882 load FSP-S: 24,954 - 25,496 - 24,368
- foxconn/g41m: uncached boot medium for testing 64bit before - 32bit - 64bit after load ramstage: 51,164 - 51,872 - 51,894
Change-Id: I890c075307c0aec877618d9902ea352ae42a3bfa Signed-off-by: Arthur Heymans arthur@aheymans.xyz --- M src/lib/lzmadecode.c M src/lib/lzmadecode.h 2 files changed, 10 insertions(+), 10 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/75/70175/4