[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