[coreboot] [PATCH] v3: add dump_mem_range()

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Mon Feb 11 00:08:41 CET 2008


On 10.02.2008 23:52, ron minnich wrote:
> Why not have dump_mem_range accept a print level as the first arg?
> Then it's just like printk and banner.
>   

Done.

Add dump_mem_range to dump a memory range.

Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>

Index: LinuxBIOSv3-dumprange/include/console.h
===================================================================
--- LinuxBIOSv3-dumprange/include/console.h	(Revision 585)
+++ LinuxBIOSv3-dumprange/include/console.h	(Arbeitskopie)
@@ -49,5 +49,6 @@
 SHARED_WITH_ATTRIBUTES(printk, int, __attribute__((format (printf, 2, 3))),
 					int msg_level, const char *fmt, ...);
 SHARED(banner, void, int msg_level, const char *msg);
+SHARED(dump_mem_range, void, int msg_level, unsigned char *buf, int size);
 
 #endif /* CONSOLE_H */
Index: LinuxBIOSv3-dumprange/arch/x86/stage1.c
===================================================================
--- LinuxBIOSv3-dumprange/arch/x86/stage1.c	(Revision 585)
+++ LinuxBIOSv3-dumprange/arch/x86/stage1.c	(Arbeitskopie)
@@ -66,6 +66,19 @@
 	// FIXME check integrity
 
 }
+
+void dump_mem_range(int msg_level, unsigned char *buf, int size)
+{
+	int i;
+	printk(msg_level, "dumping memrange %p size %i:\n", buf, size);
+	for (i = 0; i < size; i++) {
+		printk(msg_level, "%02x ", buf[i]);
+		if (i % 16 == 15)
+			printk(msg_level, "\n");
+	}
+	return;
+}
+
 /* until we get rid of elf */
 int legacy(struct mem_file *archive, char *name, void *where, struct lb_memory *mem)
 {


-- 
http://www.hailfinger.org/





More information about the coreboot mailing list