Attention is currently required from: Arthur Heymans, Maximilian Brune, Philipp Hug, Ron Minnich.
Hello Philipp Hug, Ron Minnich, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/81910?usp=email
to look at the new patch set (#3).
The following approvals got outdated and were removed: Verified+1 by build bot (Jenkins)
Change subject: commonlib/bsd/lz4_wrapper.c: Fix alignment ......................................................................
commonlib/bsd/lz4_wrapper.c: Fix alignment
Currently the HiFive Unleashed produces the following exception: [DEBUG] Exception: Load address misaligned [DEBUG] Hart ID: 0 [DEBUG] Previous mode: machine [DEBUG] Bad instruction pc: 0x080010d0 [DEBUG] Bad address: 0x08026ab3 [DEBUG] Stored ra: 0x080010c8 [DEBUG] Stored sp: 0x08010cc8
The coreboot LZ4 decompression code does some misaligned access during decompression which the FU540 apparently does not support in SRAM.
Make the compiler generate code that adheres to natural alignment by fixing the LZ4_readLE16() function and creating LZ4_readLE32().
Signed-off-by: Maximilian Brune maximilian.brune@9elements.com Change-Id: Id165829bfd35be2bce2bbb019c208a304f627add --- M src/commonlib/bsd/lz4_wrapper.c 1 file changed, 13 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/10/81910/3