Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

===================================================================
RCS file: /ftp/cvs/cvsroot/src/sys/kern/subr_pool.c,v
rcsdiff: /ftp/cvs/cvsroot/src/sys/kern/subr_pool.c,v: warning: Unknown phrases like `commitid ...;' are present.
retrieving revision 1.136
retrieving revision 1.137
diff -u -p -r1.136 -r1.137
--- src/sys/kern/subr_pool.c	2007/11/14 11:14:13	1.136
+++ src/sys/kern/subr_pool.c	2007/11/18 16:27:43	1.137
@@ -1,4 +1,4 @@
-/*	$NetBSD: subr_pool.c,v 1.136 2007/11/14 11:14:13 yamt Exp $	*/
+/*	$NetBSD: subr_pool.c,v 1.137 2007/11/18 16:27:43 ad Exp $	*/
 
 /*-
  * Copyright (c) 1997, 1999, 2000, 2002, 2007 The NetBSD Foundation, Inc.
@@ -38,7 +38,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: subr_pool.c,v 1.136 2007/11/14 11:14:13 yamt Exp $");
+__KERNEL_RCSID(0, "$NetBSD: subr_pool.c,v 1.137 2007/11/18 16:27:43 ad Exp $");
 
 #include "opt_pool.h"
 #include "opt_poollog.h"
@@ -2067,8 +2067,13 @@ pool_cache_bootstrap(pool_cache_t pc, si
 	/* Allocate per-CPU caches. */
 	memset(pc->pc_cpus, 0, sizeof(pc->pc_cpus));
 	pc->pc_ncpu = 0;
-	for (CPU_INFO_FOREACH(cii, ci)) {
-		pool_cache_cpu_init1(ci, pc);
+	if (ncpu == 0) {
+		/* XXX For sparc: boot CPU is not attached yet. */
+		pool_cache_cpu_init1(curcpu(), pc);
+	} else {
+		for (CPU_INFO_FOREACH(cii, ci)) {
+			pool_cache_cpu_init1(ci, pc);
+		}
 	}
 	
 	if (__predict_true(!cold)) {
@@ -2143,11 +2148,15 @@ static void
 pool_cache_cpu_init1(struct cpu_info *ci, pool_cache_t pc)
 {
 	pool_cache_cpu_t *cc;
+	int index;
 
+	index = ci->ci_index;
+
+	KASSERT(index < MAXCPUS);
 	KASSERT(((uintptr_t)pc->pc_cpus & (CACHE_LINE_SIZE - 1)) == 0);
 
-	if ((cc = pc->pc_cpus[ci->ci_index]) != NULL) {
-		KASSERT(cc->cc_cpu = ci);
+	if ((cc = pc->pc_cpus[index]) != NULL) {
+		KASSERT(cc->cc_cpuindex == index);
 		return;
 	}
 
@@ -2168,13 +2177,13 @@ pool_cache_cpu_init1(struct cpu_info *ci
 	cc->cc_ipl = pc->pc_pool.pr_ipl;
 	cc->cc_iplcookie = makeiplcookie(cc->cc_ipl);
 	cc->cc_cache = pc;
-	cc->cc_cpu = ci;
+	cc->cc_cpuindex = index;
 	cc->cc_hits = 0;
 	cc->cc_misses = 0;
 	cc->cc_current = NULL;
 	cc->cc_previous = NULL;
 
-	pc->pc_cpus[ci->ci_index] = cc;
+	pc->pc_cpus[index] = cc;
 }
 
 /*
@@ -2320,33 +2329,19 @@ static inline pool_cache_cpu_t *
 pool_cache_cpu_enter(pool_cache_t pc, int *s)
 {
 	pool_cache_cpu_t *cc;
-	struct cpu_info *ci;
 
 	/*
 	 * Prevent other users of the cache from accessing our
 	 * CPU-local data.  To avoid touching shared state, we
 	 * pull the neccessary information from CPU local data.
 	 */
-	ci = curcpu();
-	KASSERT(ci->ci_data.cpu_index < MAXCPUS);
-	cc = pc->pc_cpus[ci->ci_data.cpu_index];
+	crit_enter();
+	cc = pc->pc_cpus[curcpu()->ci_index];
 	KASSERT(cc->cc_cache == pc);
-	if (cc->cc_ipl == IPL_NONE) {
-		crit_enter();
-	} else {
+	if (cc->cc_ipl != IPL_NONE) {
 		*s = splraiseipl(cc->cc_iplcookie);
 	}
-
-	/* Moved to another CPU before disabling preemption? */
-	if (__predict_false(ci != curcpu())) {
-		ci = curcpu();
-		cc = pc->pc_cpus[ci->ci_data.cpu_index];
-	}
-
-#ifdef DIAGNOSTIC
-	KASSERT(cc->cc_cpu == ci);
 	KASSERT(((uintptr_t)cc & (CACHE_LINE_SIZE - 1)) == 0);
-#endif
 
 	return cc;
 }
@@ -2356,11 +2351,10 @@ pool_cache_cpu_exit(pool_cache_cpu_t *cc
 {
 
 	/* No longer need exclusive access to the per-CPU data. */
-	if (cc->cc_ipl == IPL_NONE) {
-		crit_exit();
-	} else {
+	if (cc->cc_ipl != IPL_NONE) {
 		splx(*s);
 	}
+	crit_exit();
 }
 
 #if __GNUC_PREREQ__(3, 0)