[coreboot-gerrit] Change in coreboot[master]: cpu/x86/smm: SMM module loader version 2

[Patrick Georgi (Code Review)]

Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43684 )

Change subject: cpu/x86/smm: SMM module loader version 2
......................................................................

Patch Set 2:

(2 comments)

https://review.coreboot.org/c/coreboot/+/43684/2/src/cpu/x86/Kconfig 
File src/cpu/x86/Kconfig:

https://review.coreboot.org/c/coreboot/+/43684/2/src/cpu/x86/Kconfig@129 
PS2, Line 129: server
nit: maybe deemphasize the "server" mentions a bit? There are few non-server designs out there with more than 32 threads right now, but that may change over the next few years and nothing in here is strictly "server stuff".

https://review.coreboot.org/c/coreboot/+/43684/2/src/cpu/x86/smm/smm_module_... 
File src/cpu/x86/smm/smm_module_loaderv2.c:

https://review.coreboot.org/c/coreboot/+/43684/2/src/cpu/x86/smm/smm_module_... 
PS2, Line 123: not enough space in SMM to setup all CPUs
So how would a firmware developer start to solve this, or is SMM just not available on designs with too many threads?