x86: remove additional_cpus configurability
authorIngo Molnar <mingo@elte.hu>
Sun, 5 Oct 2008 15:12:36 +0000 (17:12 +0200)
committerIngo Molnar <mingo@elte.hu>
Mon, 13 Oct 2008 08:33:47 +0000 (10:33 +0200)
additional_cpus=<x> parameter is dangerous and broken: for example
if we boot additional_cpus=-2 on a stock dual-core system it will
crash the box on bootup.

So reduce the maze of code a bit by removingthe user-configurability
angle.

Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/x86/Kconfig
arch/x86/kernel/smpboot.c

index adaca6fa927d976ce0c7f1d21c630c31a1857cec..fc8351f374fd466b8f9b287a6f5d4a2ec3b0fd32 100644 (file)
@@ -1438,24 +1438,6 @@ config HOTPLUG_CPU
            automatically on SMP systems. )
          Say N if you want to disable CPU hotplug.
 
-config HOTPLUG_RESTRICT_TO_BOOTUP_CPUS
-       def_bool n
-       prompt "Restrict CPU hotplugging to processors found during boot" if HOTPLUG_CPU
-       ---help---
-         Say no here to use the default, which allows as many CPUs as are marked
-         "disabled" by ACPI or MPTABLES to be hotplugged after bootup.
-
-         Say yes if you do not want to allow CPUs to be added after booting, for
-         example if you only need CPU hotplugging enabled for suspend/resume.
-
-         If CPU_HOTPLUG is enabled this value may be overridden at boot time
-         with the "additional_cpus" kernel parameter.
-
-config HOTPLUG_ADDITIONAL_CPUS
-       int
-       default  0 if !HOTPLUG_CPU || HOTPLUG_RESTRICT_TO_BOOTUP_CPUS
-       default -1
-
 config COMPAT_VDSO
        def_bool y
        prompt "Compat VDSO support"
index 857a88bb9195a25d231fe34821eb56f8d7674a0a..8dd201c313296a6f127261e2f34a64abf26ee0ab 100644 (file)
@@ -1261,7 +1261,7 @@ void __init native_smp_cpus_done(unsigned int max_cpus)
        check_nmi_watchdog();
 }
 
-static int additional_cpus __initdata = CONFIG_HOTPLUG_ADDITIONAL_CPUS;
+static int additional_cpus = -1;
 
 /*
  * cpu_possible_map should be static, it cannot change as cpu's
@@ -1334,12 +1334,6 @@ static void remove_siblinginfo(int cpu)
        cpu_clear(cpu, cpu_sibling_setup_map);
 }
 
-static __init int setup_additional_cpus(char *s)
-{
-       return s && get_option(&s, &additional_cpus) ? 0 : -EINVAL;
-}
-early_param("additional_cpus", setup_additional_cpus);
-
 static void __ref remove_cpu_from_maps(int cpu)
 {
        cpu_clear(cpu, cpu_online_map);