Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33300 )
Change subject: [RFC and WIP]Add an option to align FMAP regions ......................................................................
Patch Set 1:
Patch Set 1:
Forgot this patch existed. I remember discussing this with someone else more recently too, don't quite remember with whom... maybe +Hung-Te? I think the solution we discussed there was rather to add another attribute "ALIGNED" that doesn't auto-align the section but instead just errors out when the section isn't aligned. So you would write
RW_VPD(PRESERVE,ALIGNED) 32K
and then if at any point you change something else in the file such that this ends up at a non-aligned boundary, you'll get a build-time error. The assumption there is that the only alignment we'd care about is 4K for the erase block.
That is probably the better approach.