[coreboot-gerrit] New patch to review for coreboot: a44fecb cbmem console: reduce temporary buffer size for dynamic CBMEM

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Wed Nov 27 21:30:29 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/4293

-gerrit

commit a44fecb246503cfe4f43c11303d16255fb3c1bbd
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Mon Sep 9 11:22:18 2013 -0700

    cbmem console: reduce temporary buffer size for dynamic CBMEM
    
    When using dynamic CBMEM the CBMEM area is initialized before
    entering ram stage, and so we need a way smaller temporary buffer
    for the CBMEM console during early bits of ram stage. In practice
    around 256 bytes are needed, but keep the buffer at 1k so we make
    sure we don't run out.
    
    TEST=Boot tested on pit
    BRANCH=none
    BUG=none
    
    Change-Id: I462810b7bafbcc57f8e5f9b1d1f38cfdf85fa630
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
    Reviewed-on: https://chromium-review.googlesource.com/168575
    Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    [km: cherry-pick 7fd1bbc0 from chromium git]
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
 src/lib/cbmem_console.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/lib/cbmem_console.c b/src/lib/cbmem_console.c
index ac1d8bb..fec11f7 100644
--- a/src/lib/cbmem_console.c
+++ b/src/lib/cbmem_console.c
@@ -53,7 +53,13 @@ static struct cbmem_console car_cbmem_console CAR_CBMEM;
  * to be concatenated with the CBMEM console buffer contents accumulated
  * during the ROM stage, once CBMEM becomes available at RAM stage.
  */
-static u8 static_console[40000];
+
+#if CONFIG_DYNAMIC_CBMEM
+#define STATIC_CONSOLE_SIZE 1024
+#else
+#define STATIC_CONSOLE_SIZE 40000
+#endif
+static u8 static_console[STATIC_CONSOLE_SIZE];
 #endif
 
 static inline struct cbmem_console *current_console(void)



More information about the coreboot-gerrit mailing list