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@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) {