[XS] Change in coreboot[main]: soc/intel/mtl: Increase CAR_STACK_SIZE by 31KB for coreboot compatibi...

[Nico Huber (Code Review)]

Attention is currently required from: Eran Mitrani, Jakub Czapiga, Julius Werner, Kapil Porwal, Paul Menzel, Rishika Raj, Subrata Banik, Tarun.

Nico Huber has posted comments on this change by Rishika Raj. ( https://review.coreboot.org/c/coreboot/+/83540?usp=email )

Change subject: soc/intel/mtl: Increase CAR_STACK_SIZE by 31KB for coreboot compatibility
......................................................................

Patch Set 8:

(1 comment)

Commit Message:

https://review.coreboot.org/c/coreboot/+/83540/comment/f8e06304_9fe93ecc?usp... :
PS5, Line 7: soc/intel/meteorlake: Increase CAR STACK_SIZE by 31KB to meet coreboot requirements
> To clarify, there's no reason to make it 32 specifically, it just seems like a number that's big eno […]

Thanks, that's much clearer. I'm much more used to know how much stack we need.
We used to measure that and I believe static analysis would be possible too.
Maybe it's worth to check at least a little canary after FSP-M ran.

Having something 4KiB big on the CAR stack seems uncommon these days, as we
fully support .bss know AFAIK. So I'd rather move the UPD struct there, but
that might just be personal preference.