[coreboot-gerrit] Change in coreboot[master]: cpu/intel/car: Skip stack integrity check if FSP_USES_CB_STACK is enable

[Aaron Durbin (Code Review)]

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

Change subject: cpu/intel/car: Skip stack integrity check if FSP_USES_CB_STACK is enable
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35165/1/src/cpu/intel/car/romstage.... 
File src/cpu/intel/car/romstage.c:

https://review.coreboot.org/c/coreboot/+/35165/1/src/cpu/intel/car/romstage.... 
PS1, Line 63: 	if (!CONFIG(FSP_USES_CB_STACK)) {
If we do need to go this route, I think we can make this a little cleaner with some helper functions:

static void setup_stack_guards(uintptr_t base, size_t num_guards);
static void check_stack_guards(uintptr_t base, size_t num_guards);

We can conditionally bail early in those based on the Kconfig once the behavior is properly documented out FSP.