[coreboot-gerrit] Patch set updated for coreboot: 885786c Print segment clean up information only when required.
Marc Jones (marc.jones@se-eng.com)
gerrit at coreboot.org
Wed Dec 31 04:54:02 CET 2014
Marc Jones (marc.jones at se-eng.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7937
-gerrit
commit 885786cb1092b31404bcfdb52944436ffc8da32c
Author: Vadim Bendebury <vbendeb at chromium.org>
Date: Tue May 13 15:43:58 2014 -0700
Print segment clean up information only when required.
Eliminate duplicated printout and if needed, print only changed
information.
BUG=none
TEST=verified that the 'New segment dstaddr...' message is not
duplicated anymore
Original-Change-Id: Ia13593394fccbb225f2bd9ab2b9228bac29d50fb
Original-Signed-off-by: Vadim Bendebury <vbendeb at chromium.org>
Original-Reviewed-on: https://chromium-review.googlesource.com/199672
Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
(cherry picked from commit aadf018821ebfa63d6ac9d2429ae1fb483dd6cb3)
Signed-off-by: Marc Jones <marc.jones at se-eng.com>
Change-Id: I7544bddc4026191395cfe3b8ac66256ec223391e
---
src/lib/selfboot.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/lib/selfboot.c b/src/lib/selfboot.c
index 8e9e0de..de059b0 100644
--- a/src/lib/selfboot.c
+++ b/src/lib/selfboot.c
@@ -252,9 +252,10 @@ static int build_self_segment_list(
/* Clean up the values */
if (new->s_filesz > new->s_memsz) {
new->s_filesz = new->s_memsz;
+ printk(BIOS_DEBUG,
+ " cleaned up filesize 0x%lx\n",
+ new->s_filesz);
}
- printk(BIOS_DEBUG, " (cleaned up) New segment addr 0x%lx size 0x%lx offset 0x%lx filesize 0x%lx\n",
- new->s_dstaddr, new->s_memsz, new->s_srcaddr, new->s_filesz);
break;
case PAYLOAD_SEGMENT_BSS:
More information about the coreboot-gerrit
mailing list