[coreboot-gerrit] Change in coreboot[master]: soc/intel/cannonlake: Enable FSP to use coreboot stack for cometlake

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34130 )

Change subject: soc/intel/cannonlake: Enable FSP to use coreboot stack for cometlake
......................................................................

Patch Set 5:
> > > It seems to affect FSP-M only, but that's not documented.
> > > It requires a huge amount of stack, why?
> > > The Documentation wasn't updated to explain this new feature. It seems independent of EFI_MP_SERVICES_PPI.
> >
> > Yes, it is required for FSP-M to execute, we'll have to allocate the stack out of CAR.
> > The stack requirement is due to FSP_M execution, configuring it for lesser value, we see hang during FSP-M execution. The value configured here is actually less the than the stack size that was reserved by FSP for the implementation when FSP was setting up different stack(0x28000).
> >
> > yes this is independent of the EFI_MP_SERVICES_PPI.
>
> Adding Nate if we have FSP2.1 spec available for Patrick.

The FSP2_1 selection was removed with patch set 4, so this is not
about FSP 2.1 any more. Maybe you are somewhere in between?