[coreboot-gerrit] Patch merged into coreboot/master: c270bc7 elog: Get rid of the staging_header variable.

gerrit at coreboot.org gerrit at coreboot.org
Mon Dec 2 15:16:21 CET 2013


the following patch was just integrated into master:
commit c270bc7c7bb67bfc1204661a4c9af6c9419a6bf6
Author: Gabe Black <gabeblack at google.com>
Date:   Tue Apr 23 19:36:01 2013 -0700

    elog: Get rid of the staging_header variable.
    
    The header is at the start of the log. There's no reason to either keep a
    seperate pointer to it, or to keep a copy of it in some other bit of memory.
    
    Built and booted on Link and used 'mosys eventlog list' to list the
    contents of the log. Ran
    
    for x in $(seq 1 2000); do
      cat elog.event.kernel_clean > /sys/firmware/gsmi/append_to_eventlog;
    done
    
    And ran mosys eventlog list again to verify that the log had been shrunk
    correctly.
    
    Change-Id: I2afcd52c0ce5bbb662ac56f2895cdbea28d5c2ce
    Signed-off-by: Gabe Black <gabeblack at google.com>
    Reviewed-on: https://gerrit.chromium.org/gerrit/49304
    Reviewed-by: Duncan Laurie <dlaurie at chromium.org>
    Commit-Queue: Gabe Black <gabeblack at chromium.org>
    Tested-by: Gabe Black <gabeblack at chromium.org>


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

-gerrit



More information about the coreboot-gerrit mailing list