[coreboot-gerrit] Patch set updated for coreboot: 2e0c3dc CBMEM console: increase temporary buffer size for non-dynamic CBMEM

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Thu Nov 28 13:05:32 CET 2013


Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/4295

-gerrit

commit 2e0c3dc80cd6235ffd8dbf489e0e07e76b9c307b
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Wed Nov 27 22:54:11 2013 +0200

    CBMEM console: increase temporary buffer size for non-dynamic CBMEM
    
    Make temporary buffer allocation equal with the allocation in CBMEM and
    let copy_console_buffer() handle possible truncation.
    
    When not using dynamic CBMEM the CBMEM area is initialized late in the
    ramstage and should be able to hold almost as many characters as the
    CBMEM can hold. We have seen 40000 was not always enough with logging
    level set to spew, new default size is 0x10000.
    
    Change-Id: If4b143fdf807e28b6766b8b99db5216b767948d5
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
 src/lib/cbmem_console.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lib/cbmem_console.c b/src/lib/cbmem_console.c
index fec11f7..83d44e3 100644
--- a/src/lib/cbmem_console.c
+++ b/src/lib/cbmem_console.c
@@ -57,7 +57,7 @@ static struct cbmem_console car_cbmem_console CAR_CBMEM;
 #if CONFIG_DYNAMIC_CBMEM
 #define STATIC_CONSOLE_SIZE 1024
 #else
-#define STATIC_CONSOLE_SIZE 40000
+#define STATIC_CONSOLE_SIZE CONFIG_CONSOLE_CBMEM_BUFFER_SIZE
 #endif
 static u8 static_console[STATIC_CONSOLE_SIZE];
 #endif



More information about the coreboot-gerrit mailing list