[coreboot-gerrit] Patch merged into coreboot/master: 53721bd elog: Merge elog_validate_and_fill into elog_init_descriptor.

gerrit at coreboot.org gerrit at coreboot.org
Mon Dec 2 15:17:47 CET 2013


the following patch was just integrated into master:
commit 53721bdcc41c6b027a858efabe0c4cec34fe6920
Author: Gabe Black <gabeblack at google.com>
Date:   Wed Apr 24 17:54:37 2013 -0700

    elog: Merge elog_validate_and_fill into elog_init_descriptor.
    
    elog_validate_and_fill was called in exactly one place, in
    elog_init_descriptor. It didn't actually do what its name implied since the
    data in the event log was already "filled" by elog_init_descriptor. Likewise
    elog_init_descriptor was delegating an important part of its own job, scanning
    through the list of events, to elog_validate_and_fill.
    
    Since one function was basically just a displaced part of the other which
    couldn't really stand on its own, this change merges them together.
    
    Built and booted on Link. Ran mosys eventlog list. Added 2000 events with
    the SMI handler and ran mosys eventlog list again.
    
    Change-Id: Ic899eeb18146d0f127d0dded207d37d63cbc716f
    Signed-off-by: Gabe Black <gabeblack at google.com>
    Reviewed-on: https://gerrit.chromium.org/gerrit/49308
    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/4243 for details.

-gerrit



More information about the coreboot-gerrit mailing list