[coreboot] Patch merged into coreboot/master: 8b93059 Pass the CPU index as a parameter to startup.
gerrit at coreboot.org
gerrit at coreboot.org
Tue Nov 13 16:07:46 CET 2012
the following patch was just integrated into master:
commit 8b93059eccedc528443c06eb86c58bd320dca203
Author: Ronald G. Minnich <rminnich at chromium.org>
Date: Tue Jun 5 14:41:27 2012 -0700
Pass the CPU index as a parameter to startup.
This addition is in support of future multicore support in
coreboot. It also will allow us to remove some asssembly code.
The CPU "index" -- i.e., its order in the sequence in which
cores are brought up, NOT its APIC id -- is passed into the
secondary start. We modify the function to specify regparm(0).
We also take this opportunity to do some cleanup:
indexes become unsigned ints, not unsigned longs, for example.
Build and boot on a multicore system, with pcserial enabled.
Capture the output. Observe that the messages
Initializing CPU #0
Initializing CPU #1
Initializing CPU #2
Initializing CPU #3
appear exactly as they do prior to this change.
Change-Id: I5854d8d957c414f75fdd63fb017d2249330f955d
Signed-off-by: Ronald G. Minnich <rminnich at chromium.org>
Reviewed-on: http://review.coreboot.org/1820
Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
Tested-by: build bot (Jenkins)
Build-Tested: build bot (Jenkins) at Tue Nov 13 02:33:03 2012, giving +1
Reviewed-By: Stefan Reinauer <stefan.reinauer at coreboot.org> at Tue Nov 13 02:15:50 2012, giving +2
See http://review.coreboot.org/1820 for details.
-gerrit
More information about the coreboot
mailing list