[coreboot-gerrit] Patch merged into coreboot/master: a7d9244 timestamps: You can never have enough of them!

gerrit at coreboot.org gerrit at coreboot.org
Tue Apr 14 09:03:51 CEST 2015


the following patch was just integrated into master:
commit a7d924412a4300e91b3bd6eb3581b954b02f8d37
Author: Julius Werner <jwerner at chromium.org>
Date:   Tue Dec 2 20:51:19 2014 -0800

    timestamps: You can never have enough of them!
    
    Now that we have timestamps in pre-RAM stages, let's actually make use
    of them. This patch adds several timestamps to both the bootblock and
    especially the verstage to allow more fine-grained boot time tracking.
    
    Some of the introduced timestamps can appear more than once per boot.
    This doesn't seem to be a problem for both coreboot and the cbmem
    utility, and the context makes it clear which operation was timestamped
    at what point.
    
    Also simplifies cbmem's timestamp printing routine a bit, fixing a
    display bug when a timestamp had a section of exactly ",000," in it
    (e.g. 1,000,185).
    
    BRANCH=None
    BUG=None
    TEST=Booted Pinky, Blaze and Falco, confirmed that all timestamps show
    up and contained sane values. Booted Storm (no timestamps here since it
    doesn't support pre-RAM timestamps yet).
    
    Change-Id: I7f4d6aba3ebe3db0d003c7bcb2954431b74961b3
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 7a2ce81722aba85beefcc6c81f9908422b8da8fa
    Original-Change-Id: I5979bfa9445a9e0aba98ffdf8006c21096743456
    Original-Signed-off-by: Julius Werner <jwerner at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/234063
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Reviewed-on: http://review.coreboot.org/9608
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


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

-gerrit



More information about the coreboot-gerrit mailing list