diff options
author | Alexey Dobriyan <adobriyan@sw.ru> | 2007-05-08 03:28:47 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-05-08 14:15:08 -0400 |
commit | a5c43dae7ae38c2a6b3e9a819bcf45f010bf6a4a (patch) | |
tree | b30da7a4541e803e35a6a74ad33e836442c3f6c8 /include/linux | |
parent | 9d65cb4a1718a072898c7a57a3bc61b2dc4bcd4d (diff) |
Fix race between cat /proc/slab_allocators and rmmod
Same story as with cat /proc/*/wchan race vs rmmod race, only
/proc/slab_allocators want more info than just symbol name.
Signed-off-by: Alexey Dobriyan <adobriyan@sw.ru>
Acked-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'include/linux')
-rw-r--r-- | include/linux/kallsyms.h | 6 | ||||
-rw-r--r-- | include/linux/module.h | 6 |
2 files changed, 12 insertions, 0 deletions
diff --git a/include/linux/kallsyms.h b/include/linux/kallsyms.h index ae0117a95cfd..12178d2c882b 100644 --- a/include/linux/kallsyms.h +++ b/include/linux/kallsyms.h | |||
@@ -31,6 +31,7 @@ extern int sprint_symbol(char *buffer, unsigned long address); | |||
31 | extern void __print_symbol(const char *fmt, unsigned long address); | 31 | extern void __print_symbol(const char *fmt, unsigned long address); |
32 | 32 | ||
33 | int lookup_symbol_name(unsigned long addr, char *symname); | 33 | int lookup_symbol_name(unsigned long addr, char *symname); |
34 | int lookup_symbol_attrs(unsigned long addr, unsigned long *size, unsigned long *offset, char *modname, char *name); | ||
34 | 35 | ||
35 | #else /* !CONFIG_KALLSYMS */ | 36 | #else /* !CONFIG_KALLSYMS */ |
36 | 37 | ||
@@ -65,6 +66,11 @@ static inline int lookup_symbol_name(unsigned long addr, char *symname) | |||
65 | return -ERANGE; | 66 | return -ERANGE; |
66 | } | 67 | } |
67 | 68 | ||
69 | static inline int lookup_symbol_attrs(unsigned long addr, unsigned long *size, unsigned long *offset, char *modname, char *name) | ||
70 | { | ||
71 | return -ERANGE; | ||
72 | } | ||
73 | |||
68 | /* Stupid that this does nothing, but I didn't create this mess. */ | 74 | /* Stupid that this does nothing, but I didn't create this mess. */ |
69 | #define __print_symbol(fmt, addr) | 75 | #define __print_symbol(fmt, addr) |
70 | #endif /*CONFIG_KALLSYMS*/ | 76 | #endif /*CONFIG_KALLSYMS*/ |
diff --git a/include/linux/module.h b/include/linux/module.h index 099ae5932c68..6d3dc9c4ff96 100644 --- a/include/linux/module.h +++ b/include/linux/module.h | |||
@@ -455,6 +455,7 @@ const char *module_address_lookup(unsigned long addr, | |||
455 | unsigned long *offset, | 455 | unsigned long *offset, |
456 | char **modname); | 456 | char **modname); |
457 | int lookup_module_symbol_name(unsigned long addr, char *symname); | 457 | int lookup_module_symbol_name(unsigned long addr, char *symname); |
458 | int lookup_module_symbol_attrs(unsigned long addr, unsigned long *size, unsigned long *offset, char *modname, char *name); | ||
458 | 459 | ||
459 | /* For extable.c to search modules' exception tables. */ | 460 | /* For extable.c to search modules' exception tables. */ |
460 | const struct exception_table_entry *search_module_extables(unsigned long addr); | 461 | const struct exception_table_entry *search_module_extables(unsigned long addr); |
@@ -531,6 +532,11 @@ static inline int lookup_module_symbol_name(unsigned long addr, char *symname) | |||
531 | return -ERANGE; | 532 | return -ERANGE; |
532 | } | 533 | } |
533 | 534 | ||
535 | static inline int lookup_module_symbol_attrs(unsigned long addr, unsigned long *size, unsigned long *offset, char *modname, char *name) | ||
536 | { | ||
537 | return -ERANGE; | ||
538 | } | ||
539 | |||
534 | static inline int module_get_kallsym(unsigned int symnum, unsigned long *value, | 540 | static inline int module_get_kallsym(unsigned int symnum, unsigned long *value, |
535 | char *type, char *name, | 541 | char *type, char *name, |
536 | char *module_name, int *exported) | 542 | char *module_name, int *exported) |