[coreboot-gerrit] New patch to review for coreboot: ccc16cf cbfstool error: -v gives THOUSANDS of lines of bad output.

Martin Roth (gaumless@gmail.com) gerrit at coreboot.org
Tue Aug 5 01:23:29 CEST 2014


Martin Roth (gaumless at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/6496

-gerrit

commit ccc16cfa1224d025d41a66978f30fc10781c8147
Author: Martin Roth <martin.roth at se-eng.com>
Date:   Mon Aug 4 17:23:06 2014 -0600

    cbfstool error: -v gives THOUSANDS of lines of bad output.
    
    This is not intended for submission.  I just ran cbfstool with the
    -v option, trying to debug an issue I was seeing, and suddenly I had
    TWO issues...
    
    There's a problem in cbfstool here, and I don't have time to debug it
    further than this right now.  If anyone feels like looking at it, I
    offer this up to them.
    
    Here's part of the output:
    PAYLOAD    build/coreboot.pre add-payload -f payloads/memtest86+/memtest.elf -n img/memtest  -c LZMA  -v
    img/memtest                    0xe09700   payload      46951
       0x41544144 ((unknown) compression, offset: 0x38, load: 0x10000, length: 46895/192808
       0x52544e45 (none compression, offset: 0x0, load: 0x10000, length: 0/0
       0x3000001 ((unknown) compression, offset: 0x0, load: 0x7e4081b787ec, length: 2020815957/363341946
       0x314cc703 ((unknown) compression, offset: 0x42eeb380, load: 0x90b3ccab6100c86, length: 1916715941/-378419934
       0xeb8788ab ((unknown) compression, offset: 0xf3b21c77, load: 0x62e8dea3da480ad0, length: -1342285081/-1389218217
       0x246d75f4 ((unknown) compression, offset: 0xa881bd58, load: 0x6ee3e7be0c88cc21, length: -504708229/1612905668
    ... (removed 400,000+ lines)
       0x0 (none compression, offset: 0x0, load: 0x0, length: 0/0
       0x0 (none compression, offset: 0x0, load: 0x0, length: 0/0
       0x0 (none compression, offset: 0x0, load: 0x0, length: 0/0
       0x0 (none compression, offset: 0x0, load: 0x0, length: 0/0
       0x0 (none compression, offset: 0x0, load: 0x0, length: 0/0
    make: *** [payloads/memtest86+/memtest.elf] Segmentation fault (core dumped)
    
    Change-Id: Ie2effd76894e824db4012baf9ac10dd79a79a113
    Signed-off-by: Martin Roth <martin.roth at se-eng.com>
---
 util/cbfstool/cbfs_image.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/util/cbfstool/cbfs_image.c b/util/cbfstool/cbfs_image.c
index 12bc2fe..234be5f 100644
--- a/util/cbfstool/cbfs_image.c
+++ b/util/cbfstool/cbfs_image.c
@@ -684,13 +684,13 @@ int cbfs_print_entry_info(struct cbfs_image *image, struct cbfs_file *entry,
 		case CBFS_COMPONENT_PAYLOAD:
 			payload  = (struct cbfs_payload_segment *)
 					CBFS_SUBHEADER(entry);
-			while (payload) {
+//			while (payload) {
 				cbfs_print_payload_segment_info(payload, fp);
-				if (payload->type == PAYLOAD_SEGMENT_ENTRY)
-					break;
-				else
-					payload ++;
-			}
+//				if (payload->type == PAYLOAD_SEGMENT_ENTRY)
+//					break;
+//				else
+//					payload ++;
+//			}
 			break;
 		default:
 			break;



More information about the coreboot-gerrit mailing list