From: Andi Kleen <andi@firstfloor.org>
To: Bodo Eggert <7eggert@gmx.de>
Cc: Andi Kleen <andi@firstfloor.org>,
Thomas Gleixner <tglx@linutronix.de>, Ingo Molnar <mingo@elte.hu>,
Chuck Ebbert <cebbert@redhat.com>,
linux-kernel@vger.kernel.org,
Arjan van de Ven <arjan@infradead.org>
Subject: Re: <PING> Re: [patch x86/core] x86: allow number of additional hotplug CPUs to be set at compile time
Date: Mon, 6 Oct 2008 15:22:52 +0200 [thread overview]
Message-ID: <20081006132252.GF3180@one.firstfloor.org> (raw)
In-Reply-To: <E1Kmno6-0003es-Te@be1.7eggert.dyndns.org>
On Mon, Oct 06, 2008 at 12:59:18PM +0200, Bodo Eggert wrote:
> I don't have a machine with pluggable CPUs, but I'd imagine if you'd take out
> some CPUs, the number of additional CPUs you can plug in will increase by the
> same number (forcing me to change the kernel command lince if I do), while the
It only costs you some memory to have more hotpluggable CPUs (on my
kernel here around 40k/possible CPU), so there's very little "force" in
practice unless you set excessive values.
Some other architectures invented a "possible_cpus=..." parameter to address
this. I presume this could be done on x86 too, but it doesn't seem
like a very pressing issue. Even if it was done it would be better
to keep additional_cpus for compatibility.
> number of slots (and the number of possible CPUs) stays the same unless
> somebody offers a new kind of CPU card. Therefore, I'd expect "maxcpus=" to be
> the only interface I want for this purpose. Put in a value bigger than the
> amount plugged in -> voila.
maxcpus=... doesn't affect the cpu_possible_mask. It really is only
good for limiting CPUs.
> OTOH, looking at Thomas' patch, I'd guess it would not work as expected ...
> and looking at the code seems to confirm this. Besides that, I'd possibly
> want a way to limit the number of online CPUs at boot saying something like
> "onlinecpus=", which would not limit the number of CPUs I can plug in.
That is exactly what maxcpus=... does.
Anyways I don't see the additional_cpus=... removal patch in tip
so hopefully things are fine now
-Andi
--
ak@linux.intel.com
next prev parent reply other threads:[~2008-10-06 13:16 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <bijiX-86S-5@gated-at.bofh.it>
[not found] ` <bisvP-3es-3@gated-at.bofh.it>
[not found] ` <biC2k-7cR-17@gated-at.bofh.it>
[not found] ` <biCbU-7lA-15@gated-at.bofh.it>
[not found] ` <biCOy-8ep-3@gated-at.bofh.it>
[not found] ` <biD7T-5N-1@gated-at.bofh.it>
[not found] ` <bjiEh-2MC-21@gated-at.bofh.it>
[not found] ` <bjnXc-1ec-11@gated-at.bofh.it>
[not found] ` <bjz2s-78A-13@gated-at.bofh.it>
[not found] ` <bjDfK-40E-19@gated-at.bofh.it>
[not found] ` <bjDIN-4L8-31@gated-at.bofh.it>
[not found] ` <bjEbU-5eX-19@gated-at.bofh.it>
[not found] ` <bjIIi-2Oi-1@gated-at.bofh.it>
[not found] ` <bjJOk-49Q-5@gated-at.bofh.it>
[not found] ` <bjKAv-5bK-17@gated-at.bofh.it>
2008-10-06 10:59 ` <PING> Re: [patch x86/core] x86: allow number of additional hotplug CPUs to be set at compile time Bodo Eggert
2008-10-06 13:22 ` Andi Kleen [this message]
2008-10-01 23:19 Chuck Ebbert
2008-10-02 9:12 ` Andi Kleen
2008-10-02 19:25 ` Chuck Ebbert
2008-10-02 19:44 ` Andi Kleen
2008-10-02 20:09 ` Chuck Ebbert
2008-10-02 20:40 ` Andi Kleen
2008-10-04 16:52 ` <PING> " Andi Kleen
2008-10-04 22:30 ` Chuck Ebbert
2008-10-05 10:28 ` Ingo Molnar
2008-10-05 14:52 ` Thomas Gleixner
2008-10-05 15:20 ` Ingo Molnar
2008-10-05 15:51 ` Thomas Gleixner
2008-10-05 15:56 ` Ingo Molnar
2008-10-05 20:39 ` Andi Kleen
2008-10-05 21:49 ` Thomas Gleixner
2008-10-05 22:45 ` Andi Kleen
2008-10-05 20:28 ` Andi Kleen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20081006132252.GF3180@one.firstfloor.org \
--to=andi@firstfloor.org \
--cc=7eggert@gmx.de \
--cc=arjan@infradead.org \
--cc=cebbert@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=tglx@linutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.