[coreboot] r857 - coreboot-v3/lib

svn at coreboot.org svn at coreboot.org
Thu Sep 4 17:35:50 CEST 2008


Author: hailfinger
Date: 2008-09-04 17:35:49 +0200 (Thu, 04 Sep 2008)
New Revision: 857

Modified:
   coreboot-v3/lib/console.c
Log:
Fix initial printk buffer address initialization. This was broken in
r785 (move from explicit asm to implicit C) and unfortunately qemu never
triggered the bug, only real hardware did. That's because qemu will not
spew an error on access to RAM before RAM is enabled.

Thanks to Ron for bisecting this.

Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
Acked-by: Ronald G. Minnich <rminnich at gmail.com>


Modified: coreboot-v3/lib/console.c
===================================================================
--- coreboot-v3/lib/console.c	2008-09-04 00:38:54 UTC (rev 856)
+++ coreboot-v3/lib/console.c	2008-09-04 15:35:49 UTC (rev 857)
@@ -82,7 +82,9 @@
 
 void printk_buffer_init(void)
 {
-	struct printk_buffer *buf = printk_buffer_addr();
+	struct printk_buffer *buf = (struct printk_buffer *)PRINTK_BUF_ADDR_CAR;
+	global_vars()->printk_buffer = buf;
+	buf = printk_buffer_addr();
 	buf->len = PRINTK_BUF_SIZE_CAR - sizeof(struct printk_buffer);
 	buf->readoffset = 0;
 	buf->writeoffset = 0;





More information about the coreboot mailing list