[coreboot-gerrit] New patch to review for coreboot: 68ef243 CBMEM console: increase temporary buffer size for non-dynamic CBMEM

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Wed Nov 27 21:56:03 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 68ef2430f5f91a231db26d0ddcafe881a9abf9d3
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 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