Attention is currently required from: Arthur Heymans, Maximilian Brune, Nico Huber.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80776?usp=email )
Change subject: lib: Declare heap in assembly ......................................................................
Patch Set 3:
(1 comment)
File src/lib/program.ld:
https://review.coreboot.org/c/coreboot/+/80776/comment/025e7559_ab9a2cb4 : PS3, Line 141: }
I think a simpler solution is to just add a `: to_load` here. (We already have that for the `. […]
`to_load` is just how we call the program segment, it could be any name. I'm not exactly sure what's happening here but there seems to be some heuristic that sections which are explicitly marked as NOLOAD and don't contain any actual objects aren't automatically added to a suitable program segment like all sections normally are. Adding the segment assignment explicitly works around that. (In general linker scripts seem to rely on a lot of these "magic defaults" for backwards compatibility with how things were "commonly done" in the past, it's probably better to rely less on that and write more stuff explicitly where possible.)
-
Julius Werner (Code Review)