Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 4 Jun 2011 13:07:56 +0200
From:      Fabian Keil <freebsd-listen@fabiankeil.de>
To:        freebsd-current@freebsd.org
Subject:   Re: [PATCH] Add the infrastructure for supporting an infinite number of CPUs
Message-ID:  <20110604130756.1d3b71ba@fabiankeil.de>
In-Reply-To: <BANLkTi=xgP64i2S=SE1zz-p07b7cTA06Zg@mail.gmail.com>
References:  <BANLkTi=xgP64i2S=SE1zz-p07b7cTA06Zg@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
--Sig_/I44SqPU4/HIKzH2CTFHypNO
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: quoted-printable

Attilio Rao <attilio@freebsd.org> 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--



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