[coreboot-gerrit] Change in coreboot[master]: arch/x86: Don't use .bss from car.ld if not running XIP

[Arthur Heymans (Code Review)]

Attention is currently required from: Furquan Shaikh, Kyösti Mälkki.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48156 )

Change subject: arch/x86: Don't use .bss from car.ld if not running XIP
......................................................................

Patch Set 2:

(1 comment)

File src/arch/x86/assembly_entry.S:

https://review.coreboot.org/c/coreboot/+/48156/comment/45a7040c_72b78bab 
PS2, Line 39: #if ENV_STAGE_XIP
> Same question for .data, although I don't see how one could re-initialise it. So .data becomes persistent across S3 suspend for bootblock(+romstage) ?

The resume path already clears .bss.

For the data section it looks like we would need to save it somewhere and restore it on the S3 path? Assuming we want a .data section in the first stage of course.