Tweak some config defaults; fix MODPROBE_SMALL ordering in "make config"
authorDenys Vlasenko <vda.linux@googlemail.com>
Thu, 22 Dec 2016 13:36:49 +0000 (14:36 +0100)
committerDenys Vlasenko <vda.linux@googlemail.com>
Sun, 1 Jan 2017 12:01:04 +0000 (13:01 +0100)
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
include/applets.h.sh
init/init.c
modutils/Config.src
modutils/modprobe-small.c
shell/ash.c

index bab4e0d7214c65786b69b5a3000e387aea5d158b..be8b92404ace8603aeb805d97a45d7eaa0584d76 100755 (executable)
@@ -6,9 +6,6 @@
 # CONFIG_applet names
 grep ^IF_ applets.h | grep -v IF_FEATURE_ | sed 's/IF_\([A-Z0-9._-]*\)(.*/\1/' \
 | grep -v MODPROBE_SMALL \
-| sed 's/BB_SYSCTL/SYSCTL/' \
-| sed 's/TEST1/[/' \
-| sed 's/TEST2/[[/' \
 | sort | uniq \
 >applets_APP1
 
@@ -17,8 +14,11 @@ grep ^IF_ applets.h | sed -e's/ //g' -e's/.*(\([a-z[][^,]*\),.*/\1/' \
 | grep -v '^bash$' \
 | grep -v '^sh$' \
 | tr a-z A-Z \
+| sed 's/^SYSCTL$/BB_SYSCTL/' \
+| sed 's/^\[\[$/TEST1/' \
+| sed 's/^\[$/TEST2/' \
 | sort | uniq \
 >applets_APP2
 
 diff -u applets_APP1 applets_APP2 >applets_APP.diff
-rm applets_APP1 applets_APP2
+#rm applets_APP1 applets_APP2
index 0813d1b5f0fa3177c5827a004ff4546594bb713f..08cfa2f8ce3dfce057d6666d00813541cec76df1 100644 (file)
@@ -83,7 +83,7 @@
 //config:
 //config:config FEATURE_INIT_COREDUMPS
 //config:      bool "Support dumping core for child processes (debugging only)"
-//config:      default y
+//config:      default n       # not Y because this is a debug option
 //config:      depends on INIT || LINUXRC
 //config:      help
 //config:        If this option is enabled and the file /.init_enable_core
index 4227f356a72cc817715ab2ac2a1798169ba694eb..84ff34a08224845907f824d47a3190527698dd75 100644 (file)
@@ -5,6 +5,37 @@
 
 menu "Linux Module Utilities"
 
+config MODPROBE_SMALL
+       bool "Simplified modutils"
+       default y
+       select PLATFORM_LINUX
+       help
+         Simplified modutils.
+
+         With this option modprobe does not require modules.dep file
+         and does not use /etc/modules.conf file.
+         It scans module files in /lib/modules/`uname -r` and
+         determines dependencies and module alias names on the fly.
+         This may make module loading slower, most notably
+         when one needs to load module by alias (this requires
+         scanning through module _bodies_).
+
+         At the first attempt to load a module by alias modprobe
+         will try to generate modules.dep.bb file in order to speed up
+         future loads by alias. Failure to do so (read-only /lib/modules,
+         etc) is not reported, and future modprobes will be slow too.
+
+         NB: modules.dep.bb file format is not compatible
+         with modules.dep file as created/used by standard module tools.
+
+         Additional module parameters can be stored in
+         /etc/modules/$module_name files.
+
+         Apart from modprobe, other utilities are also provided:
+         - insmod is an alias to modprobe
+         - rmmod is an alias to modprobe -r
+         - depmod generates modules.dep.bb
+
 INSERT
 
 comment "Options common to multiple modutils"
index 51ba42f7ac135f36c5cde4f965753797cc0dcbd6..e3a349b4edf242bc43e9c6310fc8bc8fdbbd107f 100644 (file)
@@ -7,37 +7,9 @@
  *
  * Licensed under GPLv2, see file LICENSE in this source tree.
  */
-//config:config MODPROBE_SMALL
-//config:      bool "Simplified modutils"
-//config:      default y
-//config:      select PLATFORM_LINUX
-//config:      help
-//config:        Simplified modutils.
-//config:
-//config:        With this option modprobe does not require modules.dep file
-//config:        and does not use /etc/modules.conf file.
-//config:        It scans module files in /lib/modules/`uname -r` and
-//config:        determines dependencies and module alias names on the fly.
-//config:        This may make module loading slower, most notably
-//config:        when one needs to load module by alias (this requires
-//config:        scanning through module _bodies_).
-//config:
-//config:        At the first attempt to load a module by alias modprobe
-//config:        will try to generate modules.dep.bb file in order to speed up
-//config:        future loads by alias. Failure to do so (read-only /lib/modules,
-//config:        etc) is not reported, and future modprobes will be slow too.
-//config:
-//config:        NB: modules.dep.bb file format is not compatible
-//config:        with modules.dep file as created/used by standard module tools.
-//config:
-//config:        Additional module parameters can be stored in
-//config:        /etc/modules/$module_name files.
-//config:
-//config:        Apart from modprobe, other utilities are also provided:
-//config:        - insmod is an alias to modprobe
-//config:        - rmmod is an alias to modprobe -r
-//config:        - depmod generates modules.dep.bb
-//config:
+
+/* config MODPROBE_SMALL is defined in Config.src to ensure better "make config" order */
+
 //config:config FEATURE_MODPROBE_SMALL_OPTIONS_ON_CMDLINE
 //config:      bool "Accept module options on modprobe command line"
 //config:      default y
index 802626d4ef0a9be01c1009668fb903e66b74faba..89339182e03d777e962d4b8b6dd1382b52e1b978 100644 (file)
 //config:
 //config:config ASH_INTERNAL_GLOB
 //config:      bool "Use internal glob() implementation"
-//config:      default n
+//config:      default y       # Y is bigger, but because of uclibc glob() bug, let Y be default for now
 //config:      depends on ASH
 //config:      help
 //config:        Do not use glob() function from libc, use internal implementation.
 //config:
 //config:config ASH_MAIL
 //config:      bool "Check for new mail on interactive shells"
-//config:      default n
+//config:      default y
 //config:      depends on ASH
 //config:      help
 //config:        Enable "check for new mail" function in the ash shell.
-//config:
 
 //applet:IF_ASH(APPLET(ash, BB_DIR_BIN, BB_SUID_DROP))
 //applet:IF_FEATURE_SH_IS_ASH(APPLET_ODDNAME(sh, ash, BB_DIR_BIN, BB_SUID_DROP, sh))