Patchwork [3/3] CONFIG_HOTPLUG should be always on

login
register
mail settings
Submitter gregkh@linuxfoundation.org
Date Sept. 5, 2012, 12:01 a.m.
Message ID <20120904235155.218456468@clark.kroah.org>
Download mbox | patch
Permalink /patch/1404661/
State New, archived
Headers show

Comments

gregkh@linuxfoundation.org - Sept. 5, 2012, 12:01 a.m.
From: Greg KH <gregkh@linuxfoundation.org>

CONFIG_HOTPLUG is a very old option, back when we had static systems and it was
odd that any type of device would be removed or added after the system had
started up.  It is quite hard to disable it these days, and even if you do, it
only saves you about 200 bytes.  However, if it is disabled, lots of bugs show
up because it is almost never tested if the option is disabled.

This is a step to eventually just remove the option entirely, which will clean
up all of the devinit* variable and function pointer options, that everyone
(myself include) ends up getting wrong eventually, causing real problems when
memory segments are removed yet we don't expect them to be.

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Ralf Baechle <ralf@linux-mips.org>
Cc: Bjorn Helgaas <bhelgaas@google.com>

---
 init/Kconfig |    8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Patch

--- a/init/Kconfig
+++ b/init/Kconfig
@@ -1201,13 +1201,7 @@  config KALLSYMS_ALL
 	   Say N unless you really need all symbols.
 
 config HOTPLUG
-	bool "Support for hot-pluggable devices" if EXPERT
-	default y
-	help
-	  This option is provided for the case where no hotplug or uevent
-	  capabilities is wanted by the kernel.  You should only consider
-	  disabling this option for embedded systems that do not use modules, a
-	  dynamic /dev tree, or dynamic device discovery.  Just say Y.
+	def_bool y
 
 config PRINTK
 	default y