[coreboot-gerrit] Change in coreboot[master]: Kconfig: Allow x86 to compress pre-ram stages if not run XIP

[Julius Werner (Code Review)]

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36613 )

Change subject: Kconfig: Allow x86 to compress pre-ram stages if not run XIP
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36613/1/src/Kconfig 
File src/Kconfig:

https://review.coreboot.org/c/coreboot/+/36613/1/src/Kconfig@1220 
PS1, Line 1220: 	depends on NO_XIP_EARLY_STAGES
> Sizewise it indeed seem like a big win. Any idea how compute intensive decompressing is?

It should be barely measurable (e.g. a millisecond or less), LZ4 is super fast. It should always be a win compared to loading more bytes over SPI. The only exceptions would be really pathological cases like running on uncached memory or with a super low CPU clock or something like that.