[coreboot-gerrit] Patch merged into coreboot/master: intel: Add MMA feature in coreboot

gerrit at coreboot.org gerrit at coreboot.org
Fri Nov 20 18:21:30 CET 2015


the following patch was just integrated into master:
commit b90b94d3efbbe969c019ab8bb8316e2c08ac4968
Author: Pratik Prajapati <pratikkumar.v.prajapati at intel.com>
Date:   Fri Sep 11 13:51:38 2015 -0700

    intel: Add MMA feature in coreboot
    
    This patch implements Memory Margin Analysis feature in coreboot.
    
    Few things to note
    (1) the feature is enabled by setting CONFIG_MMA=y in the config file
    (2) coreboot reads mma_test_metadata.bin from cbfs during romstage and
    gets the name of MMA test name and test config name. Then coreboot finds
    these files in CBFS.
    If found, coreboot passes location and size of these files to FSP via
    UPD params.  Sets MrcFastBoot to 0 so that MRC happens and then MMA test
    would be executed during memory init.
    (3) FSP passes MMA results data in HOB and coreboot saves it in cbmem
    (4) when system boots to OS after test is executed cbmem tool is used
    to grab the MMA results data.
    
    BRANCH=none
    BUG=chrome-os-partner:43731
    TEST=Build and Boot kunimitsu (FAB3) and executed MMA tests
    Not tested on Glados
    
    CQ-DEPEND=CL:299476,CL:299475,CL:299474,CL:299473,CL:299509,CL:299508,CL:299507,CL:*230478,CL:*230479
    
    Change-Id: I0b4524abcf57db4d2440a06a79b5a0f4b60fa0ea
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 4aba9b728c263b9d5da5746ede3807927c9cc2a7
    Original-Change-Id: Ie2728154b49eac8695f707127334b12e345398dc
    Original-Signed-off-by: Pratik Prajapati <pratikkumar.v.prajapati at intel.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/299476
    Original-Commit-Ready: Pratikkumar V Prajapati <pratikkumar.v.prajapati at intel.com>
    Original-Tested-by: Pratikkumar V Prajapati <pratikkumar.v.prajapati at intel.com>
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Original-Reviewed-by: Pratikkumar V Prajapati <pratikkumar.v.prajapati at intel.com>
    Reviewed-on: http://review.coreboot.org/12481
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


See http://review.coreboot.org/12481 for details.

-gerrit



More information about the coreboot-gerrit mailing list