mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
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

  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                             ` 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome