[lkml]   [2000]   [Jan]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [idea] request_module(const char *fmt, ...);
Tigran Aivazian wrote:
I've downloaded that patch, and compiled my kernel with modules

* your patch:

"size vmlinux.ta"
text 1743839
data 277738
total 2440873
* your patch, but
#define request_module(format, args...) ({-EINVAL;})
text 1743167
data 277738
total 2440201

--> an empty inline function saves around 670 byte decompressed image

Imho CONFIG_KMOD changes will be extremely rare, I'd prefer an inline
function. [or the gcc'ism ({-EINVAL;}), or any other trick].
* fs/nls/nls_base.c doesn't compile, AFAICS the 'if' statement in line
211 is now superflous.
* you could use the gcc'im " __attribute__ ((format(printf, 1, 2))))",
this would cause an error message if someone tries to load his new
'h%ell%o' module. printk already uses this this gcc'ism.
* several callers of request_module() do not include errno.h, ie I got
error messages during compilation with my patch :-(


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
Please read the FAQ at

 \ /
  Last update: 2005-03-22 13:55    [from the cache]
©2003-2014 Jasper Spaans. hosted at Digital Ocean