From owner-freebsd-current Fri Jun 22 10:34:40 2001 Delivered-To: freebsd-current@freebsd.org Received: from meow.osd.bsdi.com (meow.osd.bsdi.com [204.216.28.88]) by hub.freebsd.org (Postfix) with ESMTP id BBD0E37B408 for ; Fri, 22 Jun 2001 10:34:37 -0700 (PDT) (envelope-from jhb@FreeBSD.org) Received: from laptop.baldwin.cx (john@jhb-laptop.osd.bsdi.com [204.216.28.241]) by meow.osd.bsdi.com (8.11.3/8.11.2) with ESMTP id f5MHYUI71521; Fri, 22 Jun 2001 10:34:30 -0700 (PDT) (envelope-from jhb@FreeBSD.org) Message-ID: X-Mailer: XFMail 1.4.0 on FreeBSD X-Priority: 3 (Normal) Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 8bit MIME-Version: 1.0 In-Reply-To: <20010622133221.A2616@technokratis.com> Date: Fri, 22 Jun 2001 10:40:30 -0700 (PDT) From: John Baldwin To: Bosko Milekic Subject: Re: [HEADS-UP]: Mbuf allocator changes Cc: "Alexander N. Kabaev" , freebsd-current@FreeBSD.org, Alfred Perlstein Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On 22-Jun-01 Bosko Milekic wrote: > > On Fri, Jun 22, 2001 at 10:35:32AM -0700, John Baldwin wrote: >> >> On 22-Jun-01 Alfred Perlstein wrote: >> > * Alexander N. Kabaev [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. We already have a hw.ncpu. machdep.* should not be used by anything that is supposed to be MI. Offline it seems you missed part of the point of my post though. :) -- John Baldwin -- 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/ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message