Reorder modutils config options & fix yet more dependency

- modprobe can indirectly benefit from FEATURE_2_4_MODULES and
  FEATURE_INSMOD_TRY_MAP options.
- The position of config FEATURE_INSMOD_TRY_MMAP prevented some other
  config options from indenting under FEATURE_2_4_MODULES. Reorder to
  fix this.
- FEATURE_MODPROBE_SMALL_CHECK_ALREADY_LOADED is now moved to
  Config.src under "Common options" section. (I wished to edit this
  config so that it also work with "big" modutils, but it's not done at
  the moment. Sorry.)

Signed-off-by: Kang-Che Sung <explorer09@gmail.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>

diff --git a/modutils/Config.src b/modutils/Config.src
index d0bae2ea374d4ed704438400666a41e90fc9ebee..5f0b0cec4ad320899f80fda4ad245cc189cb7dc6 100644 (file)
--- a/modutils/Config.src
+++ b/modutils/Config.src
@@ -45,31 +45,22 @@ config FEATURE_CMDLINE_MODULE_OPTIONS
          Allow insmod and modprobe take module options from the applets'
          command line.
 
+config FEATURE_MODPROBE_SMALL_CHECK_ALREADY_LOADED
+       bool "Skip loading of already loaded modules"
+       default y
+       depends on MODPROBE_SMALL && (DEPMOD || INSMOD || MODPROBE)
+       help
+         Check if the module is already loaded.
+
 config FEATURE_2_4_MODULES
        bool "Support version 2.2/2.4 Linux kernels"
        default n
-       depends on (INSMOD || RMMOD || LSMOD) && !MODPROBE_SMALL
+       depends on (INSMOD || LSMOD || MODPROBE || RMMOD) && !MODPROBE_SMALL
        help
          Support module loading for 2.2.x and 2.4.x Linux kernels.
          This increases size considerably. Say N unless you plan
          to run ancient kernels.
 
-config FEATURE_INSMOD_TRY_MMAP
-       bool "Try to load module from a mmap'ed area"
-       default n
-       depends on INSMOD && !MODPROBE_SMALL
-       help
-         This option causes module loading code to try to mmap
-         module first. If it does not work (for example,
-         it does not work for compressed modules), module will be read
-         (and unpacked if needed) into a memory block allocated by malloc.
-
-         The only case when mmap works but malloc does not is when
-         you are trying to load a big module on a very memory-constrained
-         machine. Malloc will momentarily need 2x as much memory as mmap.
-
-         Choosing N saves about 250 bytes of code (on 32-bit x86).
-
 config FEATURE_INSMOD_VERSION_CHECKING
        bool "Enable module version checking"
        default n
@@ -113,7 +104,7 @@ config FEATURE_INSMOD_LOAD_MAP
 config FEATURE_INSMOD_LOAD_MAP_FULL
        bool "Symbols in load map"
        default y
-       depends on FEATURE_INSMOD_LOAD_MAP && !MODPROBE_SMALL
+       depends on FEATURE_INSMOD_LOAD_MAP
        help
          Without this option, -m will only output section
          load map. With this option, -m will also output
@@ -129,6 +120,22 @@ config FEATURE_CHECK_TAINTED_MODULE
          support request.
          This option is required to support GPLONLY modules.
 
+config FEATURE_INSMOD_TRY_MMAP
+       bool "Try to load module from a mmap'ed area"
+       default n
+       depends on (INSMOD || MODPROBE) && !MODPROBE_SMALL
+       help
+         This option causes module loading code to try to mmap
+         module first. If it does not work (for example,
+         it does not work for compressed modules), module will be read
+         (and unpacked if needed) into a memory block allocated by malloc.
+
+         The only case when mmap works but malloc does not is when
+         you are trying to load a big module on a very memory-constrained
+         machine. Malloc will momentarily need 2x as much memory as mmap.
+
+         Choosing N saves about 250 bytes of code (on 32-bit x86).
+
 config FEATURE_MODUTILS_ALIAS
        bool "Support module.aliases file"
        default y

diff --git a/modutils/modprobe-small.c b/modutils/modprobe-small.c
index 726a3108c7482b4d2b7cdea0a943af75f563f948..d6f4814157513791c6f112f72d9c03e3e7cb8114 100644 (file)
--- a/modutils/modprobe-small.c
+++ b/modutils/modprobe-small.c
@@ -8,14 +8,8 @@
  * Licensed under GPLv2, see file LICENSE in this source tree.
  */
 
-/* config MODPROBE_SMALL is defined in Config.src to ensure better "make config" order */
-
-//config:config FEATURE_MODPROBE_SMALL_CHECK_ALREADY_LOADED
-//config:      bool "Skip loading of already loaded modules"
-//config:      default y
-//config:      depends on MODPROBE_SMALL && (DEPMOD || INSMOD || MODPROBE)
-//config:      help
-//config:        Check if the module is already loaded.
+/* modprobe-small configs are defined in Config.src to ensure better
+ * "make config" order */
 
 //applet:IF_LSMOD(   IF_MODPROBE_SMALL(APPLET(lsmod,    BB_DIR_SBIN, BB_SUID_DROP)))
 //applet:IF_MODPROBE(IF_MODPROBE_SMALL(APPLET(modprobe, BB_DIR_SBIN, BB_SUID_DROP)))