Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34976 )
Change subject: cpu/intel/car: Make stack guards more useful on C_ENV_BOOTBLOCK ......................................................................
Patch Set 5:
(2 comments)
https://review.coreboot.org/c/coreboot/+/34976/2/src/cpu/intel/car/romstage.... File src/cpu/intel/car/romstage.c:
https://review.coreboot.org/c/coreboot/+/34976/2/src/cpu/intel/car/romstage.... PS2, Line 30: const u32 max_dcache_stack_size = CONFIG_DCACHE_BSP_STACK_SIZE ?
Better: […]
Done
https://review.coreboot.org/c/coreboot/+/34976/4/src/cpu/intel/car/romstage.... File src/cpu/intel/car/romstage.c:
https://review.coreboot.org/c/coreboot/+/34976/4/src/cpu/intel/car/romstage.... PS4, Line 35: size = MIN(size, stack_size);
There are 2 definitions of _car_stack_{start,end} in car.ld. […]
Ack