From owner-freebsd-current@FreeBSD.ORG Sat Jun 4 11:23:08 2011 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id A33B91065673 for ; Sat, 4 Jun 2011 11:23:08 +0000 (UTC) (envelope-from freebsd-listen@fabiankeil.de) Received: from smtprelay04.ispgateway.de (smtprelay04.ispgateway.de [80.67.31.31]) by mx1.freebsd.org (Postfix) with ESMTP id 636BF8FC0A for ; Sat, 4 Jun 2011 11:23:08 +0000 (UTC) Received: from [78.35.65.229] (helo=fabiankeil.de) by smtprelay04.ispgateway.de with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.68) (envelope-from ) id 1QSohr-0002Qc-HC for freebsd-current@freebsd.org; Sat, 04 Jun 2011 13:07:52 +0200 Date: Sat, 4 Jun 2011 13:07:56 +0200 From: Fabian Keil To: freebsd-current@freebsd.org Message-ID: <20110604130756.1d3b71ba@fabiankeil.de> In-Reply-To: References: Mime-Version: 1.0 Content-Type: multipart/signed; micalg=PGP-SHA1; boundary="Sig_/I44SqPU4/HIKzH2CTFHypNO"; protocol="application/pgp-signature" X-Df-Sender: 775067 Subject: Re: [PATCH] Add the infrastructure for supporting an infinite number of CPUs X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 04 Jun 2011 11:23:08 -0000 --Sig_/I44SqPU4/HIKzH2CTFHypNO Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Attilio Rao wrote: > Current maximum number of CPUs supported by the FreeBSD kernel is 32. > That number cames from indirectly by the fact that we have a cpumask_t > type, representing a mask of CPUs, which is an unsigned int right now. > I then made a patch that removes the cpumask_t type and uses cpuset_t > type for characterizing a generic mask of CPUs: > http://www.freebsd.org/~attilio/largeSMP/largeSMP-patchset-beta-0.diff =20 > I really need to commit this patch before code slush happens, thus I > plan to commit it on June 7th, if no one reports bugs or can make good > point on his reviews. Please note that the patch has been greatly > tested and reviewed on all FreeBSD tier-1 and tier-2 architectures. > Anyway more testing and reviews are welcome to happen. In the few days I've been using the patch, I haven't experienced any problems on a dual core AMD64 system using SCHED_ULE. I'm aware that two isn't that close to infinity, though. Not sure if it matters, but I noticed two places where the CPU_ABSENT() macro could be used, but isn't. There may be more, but as someone unfamiliar with the code it wasn't obvious to me if there are places where "all_cpus" and "cpus" are equal. Of course there also might be a difference in semantics that I'm not gettin= g. Fabian --Sig_/I44SqPU4/HIKzH2CTFHypNO Content-Type: application/pgp-signature; name=signature.asc Content-Disposition: attachment; filename=signature.asc -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.17 (FreeBSD) iEYEARECAAYFAk3qEg8ACgkQBYqIVf93VJ273ACdElM9wRQUM3tpoIFtRJ8kxB5a UFUAmgOMWymGdVTkWIq9jn0+BTR4XQK2 =nx8z -----END PGP SIGNATURE----- --Sig_/I44SqPU4/HIKzH2CTFHypNO--