[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.DEB.1.10.0911031324380.32136@V090114053VZO-1>
Date: Tue, 3 Nov 2009 13:34:32 -0500 (EST)
From: Christoph Lameter <cl@...ux-foundation.org>
To: Ingo Molnar <mingo@...e.hu>
cc: Ian Campbell <Ian.Campbell@...rix.com>, Tejun Heo <tj@...nel.org>,
"Paul E. McKenney" <paulmck@...ibm.com>,
Linus Torvalds <torvalds@...ux-foundation.org>,
Andrew Morton <akpm@...ux-foundation.org>,
Rusty Russell <rusty@...tcorp.com.au>,
linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] Correct nr_processes() when CPUs have been unplugged
On Tue, 3 Nov 2009, Ingo Molnar wrote:
> Sidenote: percpu areas currently are kept allocated on x86.
They must be kept allocated for all possible cpus. Arch code cannot decide
to not allocate per cpu areas.
Search for "for_each_possible_cpu" in the source tree if you want more
detail.
> That might change in the future though, especially with virtual systems
> where the possible range of CPUs can be very high - without us
> necessarily wanting to pay the percpu area allocation price for it. I.e.
> dynamic deallocation of percpu areas is something that could happen in
> the future.
Could be good but would not be as easy as you may think since core code
assumes that possible cpus have per cpu areas configured. There will be
the need for additional notifiers and more complex locking if we want to
have percpu areas for online cpus only. Per cpu areas are permanent at
this point which is a good existence guarantee that avoids all sorts of
complex scenarios.
> Nice one. I'm wondering why it was not discovered for such a long time.
Cpu hotplug is rarely used (what you listed are rare and unusual cases)
and therefore online cpus == possible cpus == present cpus.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists