Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 26 Dec 2007 22:10:24 GMT
From:      John Birrell <jb@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 131732 for review
Message-ID:  <200712262210.lBQMAOvv078681@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=131732

Change 131732 by jb@jb_freebsd1 on 2007/12/26 22:09:42

	Use SMP_MAXCPU here rather than mp_maxid because this code can execute
	before the APs are enabled. We really just need the global array 
	initialised and this can be done regardless of whether or not all
	the possible CPUs are eventually put online.

Affected files ...

.. //depot/projects/dtrace/src/sys/cddl/kern/kern_cyclic.c#8 edit

Differences ...

==== //depot/projects/dtrace/src/sys/cddl/kern/kern_cyclic.c#8 (text+ko) ====

@@ -43,7 +43,7 @@
 	 * that the asserts work. On FreeBSD, if a CPU exists, it is
 	 * enabled.
 	 */
-	for (i = 0; i <= mp_maxid; i++) {
+	for (i = 0; i < SMP_MAXCPU; i++) {
 		cyclic_cpu[i].cpuid = i;
 		cyclic_cpu[i].cpu_flags &= CPU_ENABLE;
 	}



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200712262210.lBQMAOvv078681>