Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37728 )
Change subject: [RFC] treewide: move config MAX_CPUS to chipset scope
......................................................................
Patch Set 1:
Patch Set 1:
are you sure? If we ever get back to supporting servers, then max_cpus is number sockets * cpus per socket, I figure. Or am I missing something
Historical note: back in the earliest days, before more than one core per socket, MAX_CPUS was indeed just the number of sockets, and hence purely a mainboard thing. How times change :-)
Then I would just add another Kconfig option, e.g. NUM_SOCKETS or something like that. Then, platforms which can support more than one CPU would just need to override NUM_SOCKETS.
In any case, even the current approach can account for exceptions. The platform default is meant to be the max number of CPU cores that can possibly exist as per chipset limitations, but of course some boards might have additional limitations. For example, consider a LGA1155 AIO mainboard: suppose that it has a VRM and thermal solution that can only handle CPUs with a TDP of up to 35W. This means no quad-core CPUs can run on it, so MAX_CPUS could be 4.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/37728
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I7ff908a850aac4251bd27958be67c66ce4eea6b6
Gerrit-Change-Number: 37728
Gerrit-PatchSet: 1
Gerrit-Owner: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Alexander Couzens
lynxis@fe80.eu
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Damien Zammit
Gerrit-Reviewer: Evgeny Zinoviev
me@ch1p.io
Gerrit-Reviewer: Idwer Vollering
vidwer@gmail.com
Gerrit-Reviewer: Patrick Rudolph
siro@das-labor.org
Gerrit-Reviewer: Tristan Corrick
tristan@corrick.kiwi
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-CC: ron minnich
rminnich@gmail.com
Gerrit-Comment-Date: Wed, 18 Dec 2019 22:40:10 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment