Attention is currently required from: Angel Pons. Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51186 )
Change subject: cpu/x86/smm_loaderv2: Use the permanent stack top during relocation ......................................................................
Patch Set 2:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/51186/comment/4e40f75a_033d0a5a PS2, Line 13: 0x30000 when the number of CPUs is too large. Currently the code would
I can't parse this part (I can't find the main verb): […]
Done
File src/cpu/x86/mp_init.c:
https://review.coreboot.org/c/coreboot/+/51186/comment/d934b51e_c6322238 PS2, Line 787: if (smm_setup_relocation_handler((void*)perm_smbase, &smm_params)) {
Jenkins is right
Done
File src/cpu/x86/smm/smm_module_loader.c:
https://review.coreboot.org/c/coreboot/+/51186/comment/bed904b9_0cac2d81 PS2, Line 270:
no space after unary *
Done