From owner-p4-projects@FreeBSD.ORG Wed Dec 26 22:10:24 2007 Return-Path: Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id C8BCB16A468; Wed, 26 Dec 2007 22:10:24 +0000 (UTC) Delivered-To: perforce@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 8D75D16A41B for ; Wed, 26 Dec 2007 22:10:24 +0000 (UTC) (envelope-from jb@freebsd.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 82C3C13C455 for ; Wed, 26 Dec 2007 22:10:24 +0000 (UTC) (envelope-from jb@freebsd.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.1/8.14.1) with ESMTP id lBQMAOSD078684 for ; Wed, 26 Dec 2007 22:10:24 GMT (envelope-from jb@freebsd.org) Received: (from perforce@localhost) by repoman.freebsd.org (8.14.1/8.14.1/Submit) id lBQMAOvv078681 for perforce@freebsd.org; Wed, 26 Dec 2007 22:10:24 GMT (envelope-from jb@freebsd.org) Date: Wed, 26 Dec 2007 22:10:24 GMT Message-Id: <200712262210.lBQMAOvv078681@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: perforce set sender to jb@freebsd.org using -f From: John Birrell To: Perforce Change Reviews Cc: Subject: PERFORCE change 131732 for review X-BeenThere: p4-projects@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: p4 projects tree changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 26 Dec 2007 22:10:25 -0000 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; }