[coreboot-gerrit] [XS] Change in coreboot[main]: soc/intel/meteorlake: Increase CAR STACK_SIZE by 31KB to meet coreboo...

[Subrata Banik (Code Review)]

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

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

Change subject: soc/intel/meteorlake: Increase CAR STACK_SIZE by 31KB to meet coreboot requirements
......................................................................

Patch Set 5:

(1 comment)

Commit Message:

https://review.coreboot.org/c/coreboot/+/83540/comment/d38adda1_1ee6782d?usp... :
PS5, Line 7: soc/intel/meteorlake: Increase CAR STACK_SIZE by 31KB to meet coreboot requirements
> Why the 32KiB? this makes it sound like coreboot would need that much.
> Do we ever expect it to be more than 2KiB? If not, I'd guess that 4KiB
> would provide enough margin (unless something is wrong with the 512).

The CAR stack defined in coreboot is also used by coreboot to fill up the FSP-M UPDs. It is evident that the existing romstage stack size (1KB) is too small to fulfill its intended purpose, as the 1KB reserved for coreboot's portion of the stack cannot even accommodate the stack-allocated FSPM_UPD structure itself.

Based on our debugging, we have observed instances where the vboot structure is also linked into CAR. As a result, we believe that 32KB is a reasonable size for us, given all of the factors involved.