[coreboot] r3343 - trunk/coreboot-v2/src/boot

svn at coreboot.org svn at coreboot.org
Thu May 22 00:10:39 CEST 2008


Author: myles
Date: 2008-05-22 00:10:38 +0200 (Thu, 22 May 2008)
New Revision: 3343

Modified:
   trunk/coreboot-v2/src/boot/elfboot.c
Log:
This is a simple patch which allows payloads to be placed in memory in
the range of 0xf0000-0x100000, where the Coreboot tables live in v2.
As long as the payload doesn't need the tables, it seems harmless, so
why not just print a warning?

This allows v2 to load "legacybios" without having to have a separate loader.

Signed-off-by: Myles Watson <mylesgw at gmail.com>
It'll be fine for testing and doesn't really break anything that did
work before...

Acked-by: Stefan Reinauer <stepan at coresystems.de>


Modified: trunk/coreboot-v2/src/boot/elfboot.c
===================================================================
--- trunk/coreboot-v2/src/boot/elfboot.c	2008-05-21 13:49:03 UTC (rev 3342)
+++ trunk/coreboot-v2/src/boot/elfboot.c	2008-05-21 22:10:38 UTC (rev 3343)
@@ -227,6 +227,10 @@
 		if ((mtype == LB_MEM_RAM) && (start < mend) && (end > mstart)) {
 			break;
 		}
+		if ((mtype == LB_MEM_TABLE) && (start < mend) && (end > mstart)) {
+			printk_err("Payload is overwriting Coreboot tables.\n");
+			break;
+		}
 	}
 	if (i == mem_entries) {
 		printk_err("No matching ram area found for range:\n");





More information about the coreboot mailing list