[coreboot-gerrit] Patch merged into coreboot/master: 7c7b5ff libpayload: Add a new "die" function to fatally signal programming errors.

gerrit at coreboot.org gerrit at coreboot.org
Mon Sep 15 17:37:15 CEST 2014


the following patch was just integrated into master:
commit 7c7b5ffabbaa55faa17adb18ebbd1de79a9c61eb
Author: Gabe Black <gabeblack at google.com>
Date:   Sat Nov 23 00:38:49 2013 -0800

    libpayload: Add a new "die" function to fatally signal programming errors.
    
    If a programming error is detected, die can be used to print a message and
    stop execution similar to failing an assert. There's also a "die_if" function
    which is conditional.
    
    die functions, like asserts, should be used to trap programming errors and not
    when the hardware does something wrong. If all code was written perfectly, no
    die function would ever be called. In other words, it would be appropriate to
    use die if a function was called with a value that was out of bounds or if
    malloc failed. It wouldn't be appropriate if an external device doesn't
    respond.
    
    In the future, the die family of functions might print a stack trace or show
    other debugging info.
    
    Old-Change-Id: I653fc8cb0b4e459522f1b86f7fac280836d57916
    Signed-off-by: Gabe Black <gabeblack at google.com>
    Reviewed-on: https://chromium-review.googlesource.com/178000
    Reviewed-by: Gabe Black <gabeblack at chromium.org>
    Commit-Queue: Gabe Black <gabeblack at chromium.org>
    Tested-by: Gabe Black <gabeblack at chromium.org>
    (cherry picked from commit 59df109d56a0f5346562de9b3124666a4443adf0)
    
    libpayload: Fix the license in some files which were accidentally made GPL.
    
    Some files were accidentally made GPL when they were added to libpayload. This
    change changes them over to a BSD license to be in line with the intended
    license of libpayload.
    
    Old-Change-Id: Ia95ac4951b173dcb93cb489705680e7313df3c92
    Signed-off-by: Gabe Black <gabeblack at google.com>
    Reviewed-on: https://chromium-review.googlesource.com/182202
    Reviewed-by: Ronald Minnich <rminnich at chromium.org>
    Commit-Queue: Gabe Black <gabeblack at chromium.org>
    Tested-by: Gabe Black <gabeblack at chromium.org>
    (cherry picked from commit 5f47600e50e82de226f2fa6ea81d4a3d1c56277b)
    
    Squashed the initial patch for "die" functions and a later update to
    the license header.
    
    Change-Id: I3a62cd820e676f4458e61808733d81edd3d76e87
    Signed-off-by: Isaac Christensen <isaac.christensen at se-eng.com>
    Reviewed-on: http://review.coreboot.org/6889
    Tested-by: build bot (Jenkins)
    Reviewed-by: David Hendricks <dhendrix at chromium.org>


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

-gerrit



More information about the coreboot-gerrit mailing list