Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43809 )
Change subject: mb/asus/p2b: Drop select SMP ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/c/coreboot/+/43809/3//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/43809/3//COMMIT_MSG@11 PS3, Line 11: With MAX_CPUS==1, this has the effect of removing spinlock : implementation. But since is_smp_boot() evaluates false and : SMM uses separate smi_semaphore, there is no concurrency to : protect against with a spinlock.
Oh, sorry for not being clear. Just drop this paragraph. […]
My expectation was MAX_CPUS==1,SMP=n would have been identical to MAX_CPUS==1,SMP=y, but it is not. I thought it's worth commenting this odd case.