[coreboot-gerrit] Patch set updated for coreboot: 2a6ca04 libpayload: Set heap's header size to 64-bit

Nico Huber (nico.h@gmx.de) gerrit at coreboot.org
Fri Sep 6 11:31:23 CEST 2013

Nico Huber (nico.h at gmx.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3889


commit 2a6ca04efc0260d3d37c6d73fe5496c41971fc8f
Author: Nico Huber <nico.huber at secunet.com>
Date:   Fri Jun 14 15:34:59 2013 +0200

    libpayload: Set heap's header size to 64-bit
    For libpayload clients with larger memory needs (eg. FILO with integrated
    flashrom) the current configuration isn't enough.
    Change-Id: Ic82d6477c53da62a1325400f2e596d7d557d5d1e
    Signed-off-by: Patrick Georgi <patrick.georgi at secunet.com>
    Signed-off-by: Nico Huber <nico.huber at secunet.com>
 payloads/libpayload/libc/malloc.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/payloads/libpayload/libc/malloc.c b/payloads/libpayload/libc/malloc.c
index c03cc73..851ef57 100644
--- a/payloads/libpayload/libc/malloc.c
+++ b/payloads/libpayload/libc/malloc.c
@@ -37,9 +37,6 @@
  * We're also susceptible to the usual buffer overrun poisoning, though the
  * risk is within acceptable ranges for this implementation (don't overrun
  * your buffers, kids!).
- *
- * The header format (hdrtype_t) supports heaps of up to 32MiB (given that int
- * is 32 bits long).
 #define IN_MALLOC_C
@@ -50,7 +47,7 @@ extern char _heap, _eheap;	/* Defined in the ldscript. */
 static void *hstart = (void *)&_heap;
 static void *hend = (void *)&_eheap;
-typedef unsigned int hdrtype_t;
+typedef u64 hdrtype_t;
 #define HDRSIZE (sizeof(hdrtype_t))
 #define SIZE_BITS ((HDRSIZE << 3) - 7)

More information about the coreboot-gerrit mailing list