Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47598 )
Change subject: arch/x86: Remove ID_SECTION_OFFSET ......................................................................
Patch Set 2:
I don't understand the purpose of .id for !ENV_X86 case in lib/program.ld. The version strings will appear at arbitrary location near start of binary, Are the offsets used for anything? Is the purpose to just force the strings to be present in binary even with decompressor/bootblock console disabled? Some of those builds are extremely tight fits already.
I think this was just done to try to match what x86 does as closely as possible, and serves no programmatic purpose. It does allow you to visually identify images when looking at them in a hexdump, since the strings always appear near the start of the image. But I agree it's kinda pointless and I'm also in favor of getting rid of these in general, and replacing them with an architecture-independent and extensible CBFS file.