Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 22 Jun 2001 13:32:21 -0400
From:      Bosko Milekic <bmilekic@technokratis.com>
To:        John Baldwin <jhb@FreeBSD.org>
Cc:        Alfred Perlstein <bright@sneakerz.org>, freebsd-current@FreeBSD.org, "Alexander N. Kabaev" <ak03@gte.com>
Subject:   Re: [HEADS-UP]: Mbuf allocator changes
Message-ID:  <20010622133221.A2616@technokratis.com>
In-Reply-To: <XFMail.010622103532.jhb@FreeBSD.org>; from jhb@FreeBSD.org on Fri, Jun 22, 2001 at 10:35:32AM -0700
References:  <20010622121725.X4543@sneakerz.org> <XFMail.010622103532.jhb@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help

On Fri, Jun 22, 2001 at 10:35:32AM -0700, John Baldwin wrote:
> 
> On 22-Jun-01 Alfred Perlstein wrote:
> > * Alexander N. Kabaev <ak03@gte.com> [010622 10:46] wrote:
> >> UP kernel can not be compiled in -CURRENT after your changes because
> >> kern/subr_mbuf.c references mp_ncpus variable, which is defined only in SMP
> >> case. Should this variable be moved out of #ifdef SMP?
> > 
> > Yes, I asked for this months ago, I thought it was already done.
> 
> mp_npcus is not initialized, etc. in the UP case.  I suppose it could be
> statically initialized to 1 and moved, but in that case it needs renaming, as
> mp_ncpus implies SMP (mp_ prefix).  If you want to make it ncpus and move it to
> sys/systm.h and stick it somewhere MI initialized to 1 that is fine.  Then
> hw.ncpus can reference that (well, it's called hw.ncpu right now, perhaps it
> should be renamed to match the variable *shrug*) and kern.smp.cpus can die as
> it won't be needed any longer.

	Note that we already have a (machdep, I think) sysctl exported ncpu
variable.
 
> > -Alfred
> 
> -- 
> 
> John Baldwin <jhb@FreeBSD.org> -- http://www.FreeBSD.org/~jhb/
> PGP Key: http://www.baldwin.cx/~john/pgpkey.asc
> "Power Users Use the Power to Serve!"  -  http://www.FreeBSD.org/
> 

-- 
 Bosko Milekic
 bmilekic@technokratis.com


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message




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