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

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak 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//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/35165/1//COMMIT_MSG@12 
PS1, Line 12: Don't need to run code logic to check the integrity of stack if FSP and
            : coreboot both likely to make use of common stack.
> I agree with Aaron. We should properly document what the FSP utilization of the stack is. […]

There still shouldn't be anything wrong with checking that the stack is a) used as an actual stack, and b) no unallocated memory is being touched.