Re: [PATCH] Print function names during do_initcall debugging

[Andrew Morton <akpm@osdl.org>]

Jan-Benedict Glaw <jbglaw@lug-owl.de> wrote:
>
> Please merge the following patch. It prints __init function names while
> all calls are executed at do_initcalls().

Nice, thanks.  I tried to do this in the original patch but the kallsyms
stuff wasn't set up at that stage.  Someone must have moved something.

However I suspect you didn't test it with CONFIG_KALLSYMS=n: it will be
missing newlines in the output.



 25-akpm/init/main.c |    8 ++++++--
 1 files changed, 6 insertions(+), 2 deletions(-)

diff -puN init/main.c~initcall_debug-print_symbol init/main.c
--- 25/init/main.c~initcall_debug-print_symbol	Tue Mar  9 15:07:16 2004
+++ 25-akpm/init/main.c	Tue Mar  9 15:07:59 2004
@@ -37,6 +37,7 @@
 #include <linux/profile.h>
 #include <linux/rcupdate.h>
 #include <linux/moduleparam.h>
+#include <linux/kallsyms.h>
 #include <linux/writeback.h>
 #include <linux/cpu.h>
 #include <linux/efi.h>
@@ -513,8 +514,11 @@ static void __init do_initcalls(void)
 	for (call = &__initcall_start; call < &__initcall_end; call++) {
 		char *msg;
 
-		if (initcall_debug)
-			printk("calling initcall 0x%p\n", *call);
+		if (initcall_debug) {
+			printk(KERN_DEBUG "Calling initcall 0x%p", *call);
+			print_symbol(": %s()", (unsigned long) *call);
+			printk("\n");
+		}
 
 		(*call)();
 

_