Date: Thu, 27 Mar 2008 19:37:32 +0000 From: John Birrell <jb@what-creek.com> To: Robert Watson <rwatson@FreeBSD.org> Cc: cvs-src@FreeBSD.org, John Birrell <jb@FreeBSD.org>, cvs-all@FreeBSD.org, src-committers@FreeBSD.org Subject: Re: cvs commit: src/sys/amd64/include param.h src/sys/arm/include param.h src/sys/i386/include param.h src/sys/ia64/include param.h src/sys/powerpc/include param.h src/sys/sparc64/include param.h src/sys/sun4v/include param.h Message-ID: <20080327193732.GA30621@what-creek.com> In-Reply-To: <20080327072329.U34007@fledge.watson.org> References: <200803270503.m2R53Qr7053473@repoman.freebsd.org> <20080327072329.U34007@fledge.watson.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Mar 27, 2008 at 07:52:42AM +0000, Robert Watson wrote: > Hmm. I think this probably is the right thing, but we'll need to be very > careful to watch for people defining foo[MAXCPU] in globally visible data > structures in the kernel in such a way that entries for the non-current CPU > are referenced by any modules. A casual glance suggests that, right now, > this is not generally an issue, but something to watch out for, since we do > use MAXCPU to size a number of kernel data structures. I should have added to the commit message: Suggested by: jhb -- John Birrell
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20080327193732.GA30621>