[coreboot-gerrit] Change in coreboot[master]: arch/x86: Move stack with CAR_GLOBAL_MIGRATION

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34882 )

Change subject: arch/x86: Move stack with CAR_GLOBAL_MIGRATION
......................................................................

Patch Set 22: Code-Review-2

The linker script does not reflect reality before or after this commit.

Implementation of amd/stoneyridge, even though it has C_ENVIRONMENT_BOOTBLOCK, places BSP stack right below _car_region_end. No stack guards are used. CB:34883

FSP2.0 (FSP_USES_CB_STACK=n) places shared FSP heap+stack right below _car_region_end. There is runtime checking (die() in case of failure), but build-time assertion would be nicer.

I don't see why any of the followup works would depend on this, so I am suggesting to not merge this in its current state.