[coreboot-gerrit] Patch merged into coreboot/master: intel/skylake: add function is_secondary_thread()

gerrit at coreboot.org gerrit at coreboot.org
Fri Feb 17 17:46:08 CET 2017


the following patch was just integrated into master:
commit 2b194d97411bd86303e0fec3a2edae2a718466bc
Author: Robbie Zhang <robbie.zhang at intel.com>
Date:   Tue Feb 14 15:12:17 2017 -0800

    intel/skylake: add function is_secondary_thread()
    
    There are MSRs that are programmable per-core not per-thread, so add
    a function to check whether current executing CPU is a primary core
    or a "hyperthreaded"/secondary core. For instance when trying to
    program Core PRMRR MSRs(per-core) with mp_init, cpu exception is thrown
    from the secondary thread. This function was used to avoid that.
    
    Potentially this function can be put to common code or arch/x86 or cpu/x86.
    
    BUG=chrome-os-partner:62438
    BRANCH=NONE
    TEST=Tested on Eve, verified core PRMRR MSRs get programmed only on primary
    thread avoiding exeception.
    
    Change-Id: Ic9648351fadf912164a39206788859baf3e5c173
    Signed-off-by: Robbie Zhang <robbie.zhang at intel.com>
    Reviewed-on: https://review.coreboot.org/18366
    Tested-by: build bot (Jenkins)
    Reviewed-by: Aaron Durbin <adurbin at chromium.org>


See https://review.coreboot.org/18366 for details.

-gerrit



More information about the coreboot-gerrit mailing list