Messages in this thread Patch in this message |  | | Date | Mon, 14 May 2001 16:21:18 +0400 | Subject | [PATCH] small addition to <linux/init.h> | From | <> |
| |
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 pazke@orbita1.ru | 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 */[unhandled content-type:application/pgp-signature] |  |