[coreboot-gerrit] Patch merged into coreboot/master: b02c873 cbfstool: Fix initial empty space in image creation.

gerrit at coreboot.org gerrit at coreboot.org
Fri Mar 22 00:21:42 CET 2013


the following patch was just integrated into master:
commit b02c873190990698350a7c2a9bce52ce81c0f1b2
Author: Hung-Te Lin <hungte at chromium.org>
Date:   Fri Mar 15 17:40:08 2013 +0800

    cbfstool: Fix initial empty space in image creation.
    
    When calculating initial CBFS empty entry space, the size of header itself must
    be not included (with the reserved space for entry name). This is a regression
    of the old cbfstool size bug.
    
    Before this fix, in build process we see:
     OBJCOPY    cbfs/fallback/romstage_null.bin
     W: CBFS image was created with old cbfstool with size bug.
        Fixing size in last entry...
    
    And checking the output binary:
     cbfstool build/coreboot.pre1 print -v -v
     DEBUG: read_cbfs_image: build/coreboot.pre1 (262144 bytes)
     DEBUG: x86sig: 0xfffffd30, offset: 0x3fd30
     W: CBFS image was created with old cbfstool with size bug.
        Fixing size in last entry...
     DEBUG: Last entry has been changed from 0x3fd40 to 0x3fd00.
     coreboot.pre1: 256 kB, bootblksz 688, romsize 262144, offset 0x0 align: 64
     Name                           Offset     Type         Size
     (empty)                        0x0        null         261296
     DEBUG:  cbfs_file=0x0, offset=0x28, content_address=0x28+0x3fcb0
    
    After this fix, no more alerts in build process.
    Verified to build successfully on x86/qemu and arm/snow configurations.
    
    Change-Id: I35c96f4c10a41bae671148a0e08988fa3bf6b7d3
    Signed-off-by: Hung-Te Lin <hungte at chromium.org>
    Reviewed-on: http://review.coreboot.org/2731
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>

Build-Tested: build bot (Jenkins) at Fri Mar 15 18:01:05 2013, giving +1
Reviewed-By: Stefan Reinauer <stefan.reinauer at coreboot.org> at Fri Mar 22 00:20:03 2013, giving +2
See http://review.coreboot.org/2731 for details.

-gerrit



More information about the coreboot-gerrit mailing list