[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