[coreboot-gerrit] Patch merged into coreboot/master: lib/timestamp.c: only log "Timestamp table full" once

gerrit at coreboot.org gerrit at coreboot.org
Tue Nov 24 21:01:30 CET 2015


the following patch was just integrated into master:
commit f5fd4c99d5b05fd9677cf81124d5ad5ef2270011
Author: Ben Gardner <gardner.ben at gmail.com>
Date:   Fri Nov 20 13:25:25 2015 -0600

    lib/timestamp.c: only log "Timestamp table full" once
    
    If the timestamp table gets corrupted (separate issue), the
    timestamp_sync_cache_to_cbmem() function may add a large number of bogus
    timestamp entries.
    
    This causes a flood of "ERROR: Timestamp table full".  With logs going
    to a serial console, this renders the system essentially unbootable.
    
    There really isn't a need to log that more than once, so log it when the
    last slot in the timestamp table is filled.
    
    Change-Id: I05d131183afceca31f4dac91c5edc95cfb1e443f
    Signed-off-by: Ben Gardner <gardner.ben at gmail.com>
    Reviewed-on: http://review.coreboot.org/12506
    Tested-by: build bot (Jenkins)
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-by: Patrick Georgi <pgeorgi at google.com>


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

-gerrit



More information about the coreboot-gerrit mailing list