[coreboot-gerrit] Change in coreboot[master]: arch/x86: Reserve some CAR memory for FSP single stack requirement

[Aaron Durbin (Code Review)]

Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35165 )

Change subject: arch/x86: Reserve some CAR memory for FSP single stack requirement
......................................................................

Patch Set 9:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35165/9/src/drivers/intel/fsp2_0/Kc... 
File src/drivers/intel/fsp2_0/Kconfig:

https://review.coreboot.org/c/coreboot/+/35165/9/src/drivers/intel/fsp2_0/Kc... 
PS9, Line 209: 	default 0x10000
> we had a sync with FSP team and finalize heap = 0x10000 and stack = 0x20000 for now on all FSP2.1 design and will get back on future soc if this requiremtn doesn't meet there.
>
> The same will be documented. Not sure where you see the gap. commit msg has been updated with the expectation.

128 KiB for stack and 64KiB for heap? That is massive. Where is the bloat coming from?