[coreboot-gerrit] [S] Change in coreboot[main]: commonlib/bsd/lz4_wrapper.c: Fix misaligned access

[Maximilian Brune (Code Review)]

Attention is currently required from: Angel Pons, Arthur Heymans, Julius Werner, Nico Huber, Philipp Hug, Ron Minnich.

Maximilian Brune has posted comments on this change by Maximilian Brune. ( https://review.coreboot.org/c/coreboot/+/81910?usp=email )

Change subject: commonlib/bsd/lz4_wrapper.c: Fix misaligned access
......................................................................

Patch Set 6:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/81910/comment/f8dd42b3_3b307e70?usp... :
PS1, Line 18: The coreboot LZ4 decompression code does some misaligned access during
            : decompression
> Hmmm... fine. […]

Done

File src/commonlib/bsd/lz4_wrapper.c:

https://review.coreboot.org/c/coreboot/+/81910/comment/11278048_7535c71a?usp... :
PS5, Line 10: /* LZ4 comes with its own supposedly portable memory access functions, but they
            :  * seem to be very inefficient in practice (at least on ARM64). Since coreboot
            :  * knows about endianness and allows some basic assumptions (such as unaligned
            :  * access support), we can easily write the ones we need ourselves. */
> This comment seems to be inaccurate now.

They now reflect the discussions in the comments.