Attention is currently required from: Arthur Heymans, Jon Murphy, Jérémy Compostella, Nico Huber.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81886?usp=email )
Change subject: arch/x86: Prevent .text/.init overlap with older linkers ......................................................................
Patch Set 2:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/81886/comment/35a86e9b_657e4c31 : PS2, Line 11: (binutils 2.3x)
Are you sure this is used with the main branch? I was told ChromeOS always uses the newest toolchain (and hence we have to restrict toolchain updates to the releases). binutils was updated to 2.40 a year ago.
we are still using 2.37 version and raised an internal bug https://b.corp.google.com/issues/332445618 to perform an uprev to 2.42 to switch to the new binutil. But as said, the existing EDK2 builds are failing (as Intel FSP/EDK2 code are not supportive of the latest tool chain).
in case you don't have access to the chrome bug to share the context. Possibly one year ago we attempted 2.40 migration but since then we are blocked due to Intel EDK2 related build issue. So far we don't have resolution and it blocked the 2.40 migration for us. This time we reached out to EDK2 folks for fixing the issue. let see how it goes.
-
Subrata Banik (Code Review)