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@gmail.com It'll be fine for testing and doesn't really break anything that did work before...
Acked-by: Stefan Reinauer stepan@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");