Hi all,

please take a quick look at attached patch (IMHO it can be usefull :)

The code below is a common code pattern in Linux kernel: 

static int __init foo(void)
{
        . . .
        printk("%s: blah blah blah\n", bar);
        . . .
}

this is bad because "blah blah blah\n" goes to .rodata section and plagues 
the memory. With this small patch it's possibe to rewrite this fragment as:

static int __init foo(void)
{
        . . .
        printk(__init_msg("%s: blah blah blah\n"), bar);
        . . .
}

and thus "blah blah blah\n" goes to .data.init and then to bitbucket.
IMHO it can save some extra memory for us.

Best regadrs.

-- 
Andrey Panin            | Embedded systems software engineer
[EMAIL PROTECTED]        | PGP key: http://www.orbita1.ru/~pazke/AndreyPanin.asc
diff -ur linux.vanilla/include/linux/init.h linux/include/linux/init.h
--- linux.vanilla/include/linux/init.h  Mon May 14 15:51:20 2001
+++ linux/include/linux/init.h  Mon May 14 15:54:05 2001
@@ -155,4 +155,9 @@
 #define __devexitdata __exitdata
 #endif
 
+#define __init_msg(x) ({ static char msg[] __initdata = (x); msg; })
+#define __exit_msg(x) ({ static char msg[] __exitdata = (x); msg; })
+#define __devinit_msg(x) ({ static char msg[] __devinitdata = (x); msg; })
+#define __devexit_msg(x) ({ static char msg[] __devexitdata = (x); msg; })
+
 #endif /* _LINUX_INIT_H */

PGP signature

Reply via email to