Attention is currently required from: Arthur Heymans, Intel coreboot Reviewers.
Hello Arthur Heymans, Intel coreboot Reviewers, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/86038?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed: Verified+1 by build bot (Jenkins)
Change subject: cpu/x86/smm: Fix smm_get_save_state() returning invalid pointer ......................................................................
cpu/x86/smm: Fix smm_get_save_state() returning invalid pointer
The smm_get_save_state() function returns an invalid pointer (negative pointer) when the cpu variable is equal to the number of CPUs. This leads to a hang when the pointer is used to access the save state.
TEST=No unexpected hangs in System Management Mode (SMM) were detected on fatcat.
Change-Id: I09f969105190a004372c43cb1542f5b716da1eda Signed-off-by: Jeremy Compostella jeremy.compostella@intel.com --- M src/cpu/x86/smm/smm_module_handler.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/38/86038/2