Get rid of phycpus_attached bitmask; it's maintained but not used and trunk
authorbouyer <bouyer@NetBSD.org>
Fri, 24 Feb 2012 11:31:23 +0000
branchtrunk
changeset 209449 83c37edf1e7c
parent 209448 39776f34c16e
child 209450 17d264b48f84
Get rid of phycpus_attached bitmask; it's maintained but not used and will limit the number of physical CPUs to 32 without good reasons.
sys/arch/xen/x86/cpu.c
--- a/sys/arch/xen/x86/cpu.c	Fri Feb 24 08:44:44 2012 +0000
+++ b/sys/arch/xen/x86/cpu.c	Fri Feb 24 11:31:23 2012 +0000
@@ -1,4 +1,4 @@
-/*	$NetBSD: cpu.c,v 1.86 2012/02/24 08:06:07 cherry Exp $	*/
+/*	$NetBSD: cpu.c,v 1.87 2012/02/24 11:31:23 bouyer Exp $	*/
 /* NetBSD: cpu.c,v 1.18 2004/02/20 17:35:01 yamt Exp  */
 
 /*-
@@ -66,7 +66,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: cpu.c,v 1.86 2012/02/24 08:06:07 cherry Exp $");
+__KERNEL_RCSID(0, "$NetBSD: cpu.c,v 1.87 2012/02/24 11:31:23 bouyer Exp $");
 
 #include "opt_ddb.h"
 #include "opt_multiprocessor.h"
@@ -184,9 +184,6 @@
 uint32_t cpus_attached = 1;
 uint32_t cpus_running = 1;
 
-uint32_t phycpus_attached = 0;
-uint32_t phycpus_running = 0;
-
 uint32_t cpu_feature[5]; /* X86 CPUID feature bits
 			  *	[0] basic features %edx
 			  *	[1] basic features %ecx
@@ -222,11 +219,6 @@
 
 	sc->sc_dev = self;
 
-	if (phycpus_attached == ~0) {
-		aprint_error(": increase MAXCPUS\n");
-		return;
-	}
-
 	/*
 	 * If we're an Application Processor, allocate a cpu_info
 	 * If we're the first attached CPU use the primary cpu_info,
@@ -260,8 +252,6 @@
 	ci->ci_index = nphycpu++;
 	ci->ci_cpumask = (1 << cpu_index(ci));
 
-	atomic_or_32(&phycpus_attached, ci->ci_cpumask);
-
 	if (!pmf_device_register(self, NULL, NULL))
 		aprint_error_dev(self, "couldn't establish power handler\n");