diff options
author | Denis Vlasenko | 2009-03-05 09:26:17 +0000 |
---|---|---|
committer | Denis Vlasenko | 2009-03-05 09:26:17 +0000 |
commit | c5741003e9cca3c0591ff44c4f27db71aab8be30 (patch) | |
tree | 02f67954a8ec9ef9ecdb42ba834c3dcf93261c1d | |
parent | 76140a77c9c47c41f244d90d4250d77b9504f7d4 (diff) | |
download | busybox-c5741003e9cca3c0591ff44c4f27db71aab8be30.zip busybox-c5741003e9cca3c0591ff44c4f27db71aab8be30.tar.gz |
modutils: delete unused global variable
function old new delta
modprobe_main 570 567 -3
insmod_main 101 98 -3
insmod_outputname 4 - -4
------------------------------------------------------------------------------
(add/remove: 0/1 grow/shrink: 0/2 up/down: 0/-10) Total: -10 bytes
text data bss dec hex filename
816784 476 7892 825152 c9740 busybox_old
816778 476 7888 825142 c9736 busybox_unstripped
-rw-r--r-- | modutils/Config.in | 7 | ||||
-rw-r--r-- | modutils/modutils.c | 2 | ||||
-rw-r--r-- | modutils/modutils.h | 21 |
3 files changed, 12 insertions, 18 deletions
diff --git a/modutils/Config.in b/modutils/Config.in index ce3bce1..ef8d969 100644 --- a/modutils/Config.in +++ b/modutils/Config.in @@ -44,7 +44,6 @@ config FEATURE_MODPROBE_SMALL_OPTIONS_ON_CMDLINE depends on MODPROBE_SMALL help Allow insmod and modprobe take module options from command line. - N.B. Very bloaty. config FEATURE_MODPROBE_SMALL_CHECK_ALREADY_LOADED bool "Skip loading of already loaded modules" @@ -52,7 +51,6 @@ config FEATURE_MODPROBE_SMALL_CHECK_ALREADY_LOADED depends on MODPROBE_SMALL help Check if the module is already loaded. - N.B. It's racy. config INSMOD bool "insmod" @@ -92,9 +90,6 @@ config MODPROBE Handle the loading of modules, and their dependencies on a high level. - Note that in the state, modprobe does not understand multiple - module options from the configuration file. See option below. - config FEATURE_MODPROBE_BLACKLIST bool prompt "Blacklist support" @@ -143,7 +138,7 @@ config FEATURE_INSMOD_KSYMOOPS_SYMBOLS By adding module symbols to the kernel symbol table, Oops messages occuring within kernel modules can be properly debugged. By enabling this feature, module symbols will always be added to the kernel symbol - table for properly debugging support. If you are not interested in + table for proper debugging support. If you are not interested in Oops messages from kernel modules, say N. config FEATURE_INSMOD_LOADINKMEM diff --git a/modutils/modutils.c b/modutils/modutils.c index 8836f7c..5a4ba84 100644 --- a/modutils/modutils.c +++ b/modutils/modutils.c @@ -17,8 +17,6 @@ extern int delete_module(const char *module, unsigned int flags); # define delete_module(mod, flags) syscall(__NR_delete_module, mod, flags) #endif -USE_FEATURE_2_4_MODULES(char *insmod_outputname); - /* a libbb candidate from ice age! */ diff --git a/modutils/modutils.h b/modutils/modutils.h index cc7c653..0a29ff2 100644 --- a/modutils/modutils.h +++ b/modutils/modutils.h @@ -27,9 +27,12 @@ int string_to_llist(char *string, llist_t **llist, const char *delim) FAST_FUNC; char *filename2modname(const char *filename, char *modname) FAST_FUNC; char *parse_cmdline_module_options(char **argv) FAST_FUNC; -#define INSMOD_OPTS "vq" USE_FEATURE_2_4_MODULES("sLo:fkx") \ +#define INSMOD_OPTS \ + "vq" \ + USE_FEATURE_2_4_MODULES("sLo:fkx") \ USE_FEATURE_INSMOD_LOAD_MAP("m") -#define INSMOD_ARGS USE_FEATURE_2_4_MODULES(, &insmod_outputname) + +#define INSMOD_ARGS USE_FEATURE_2_4_MODULES(, NULL) enum { INSMOD_OPT_VERBOSE = 0x0001, @@ -42,13 +45,13 @@ enum { INSMOD_OPT_NO_EXPORT = 0x0080 * ENABLE_FEATURE_2_4_MODULES, INSMOD_OPT_PRINT_MAP = 0x0100 * ENABLE_FEATURE_INSMOD_LOAD_MAP, #if ENABLE_FEATURE_2_4_MODULES -#if ENABLE_FEATURE_INSMOD_LOAD_MAP +# if ENABLE_FEATURE_INSMOD_LOAD_MAP INSMOD_OPT_UNUSED = 0x0200, -#else /* ENABLE_FEATURE_INSMOD_LOAD_MAP */ - INSMOD_OPT_UNUSED = 0x0100 -#endif -#else /* ENABLE_FEATURE_2_4_MODULES */ - INSMOD_OPT_UNUSED = 0x0004 +# else + INSMOD_OPT_UNUSED = 0x0100, +# endif +#else + INSMOD_OPT_UNUSED = 0x0004, #endif }; @@ -56,8 +59,6 @@ int FAST_FUNC bb_init_module(const char *module, const char *options); int FAST_FUNC bb_delete_module(const char *module, unsigned int flags); #if ENABLE_FEATURE_2_4_MODULES -extern char *insmod_outputname; - int FAST_FUNC bb_init_module_24(const char *module, const char *options); #endif |