[coreboot-gerrit] Change in coreboot[master]: src/cpu/x86/smm: remove debug message; not thread safe

[Nico Huber (Code Review)]

Attention is currently required from: Jonathan Zhang, Paul Menzel, David Hendricks, Rocky Phagura, Arthur Heymans, Aaron Durbin.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52518 )

Change subject: src/cpu/x86/smm: remove debug message; not thread safe
......................................................................

Patch Set 2: Code-Review+1

(3 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/52518/comment/adf20847_982decef 
PS2, Line 9: the console init
If you word it like that, it sounds like you're removing *the* SMM console
init. But this is merely about an odd error path.

How about

This patch removes a console_init() call and a debug...

?

https://review.coreboot.org/c/coreboot/+/52518/comment/0944bfce_8887cb22 
PS2, Line 10:  This prevents system hangs (soft hangs)
            : while in SMM if user drops in a new SOC with more cores or another
            : socket.
Shouldn't CONFIG_MAX_CPUS account for that?

https://review.coreboot.org/c/coreboot/+/52518/comment/5ab87c58_3f1eb85e 
PS2, Line 15: Tested on DeltaLake mainboard with SMM enabled and 52 CPU threads. .
With a low MAX_CPUS setting?