All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arjan van de Ven <arjan@infradead.org>
To: Ingo Molnar <mingo@elte.hu>
Cc: linux-kernel@vger.kernel.org, Milton Miller <miltonm@bga.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	arjanvandeven@gmail.com,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Peter Zijlstra <a.p.zijlstra@chello.nl>,
	Thomas Gleixner <tglx@linutronix.de>,
	"H. Peter Anvin" <hpa@zytor.com>
Subject: Re: smp: Start up non-boot CPUs asynchronously
Date: Tue, 31 Jan 2012 05:41:55 -0800	[thread overview]
Message-ID: <20120131054155.371e8307@infradead.org> (raw)
In-Reply-To: <20120131125232.GD4408@elte.hu>

On Tue, 31 Jan 2012 13:52:32 +0100
Ingo Molnar <mingo@elte.hu> wrote:
> 
> The real fix would be to make the init code depend less on each 
> other, i.e. have less hotplug lock dependencies. Or, if it's 
> such a hot lock for a good reason, why does spinning on it slow 
> down the boot process? It really shouldnt.

by inspection, anything that calls get_online_cpus()/put_online_cpus()
will block while a CPU is coming up. This is used in things like 
kmem_cache_create()... which is used about everywhere.
(there's various other places... more or less it's a requirement
for using the for_each_online_cpu() api correctly)



-- 
Arjan van de Ven 	Intel Open Source Technology Centre
For development, discussion and tips for power savings, 
visit http://www.lesswatts.org

  reply	other threads:[~2012-01-31 13:40 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-31  4:54 smp: Start up non-boot CPUs asynchronously Arjan van de Ven
2012-01-31 12:52 ` Ingo Molnar
2012-01-31 13:41   ` Arjan van de Ven [this message]
2012-01-31 14:31     ` Ingo Molnar
2012-01-31 15:22       ` Arjan van de Ven
2012-01-31 16:12         ` Ingo Molnar
2012-01-31 16:24           ` Arjan van de Ven
2012-02-01 22:55             ` Andrew Morton
     [not found]               ` <CADyApD0yVOePmaLznks_h6xR_BCUjzEFUB7VtsL9vvsoHwCOVw@mail.gmail.com>
2012-02-01 23:31                 ` Linus Torvalds
2012-02-14  8:17             ` Srivatsa S. Bhat
2012-02-14  8:17               ` Srivatsa S. Bhat
2012-02-14  9:48               ` Srivatsa S. Bhat
2012-02-14  9:48                 ` Srivatsa S. Bhat
2012-02-14 14:31                 ` Arjan van de Ven
2012-02-14 15:20                   ` Peter Zijlstra
2012-02-14 15:20                     ` Peter Zijlstra
2012-02-14 16:01                     ` Arjan van de Ven
2012-02-14 19:57                   ` Srivatsa S. Bhat
2012-02-14 19:57                     ` Srivatsa S. Bhat
2012-02-14 20:00                     ` Peter Zijlstra
2012-02-14 20:00                       ` Peter Zijlstra
2012-02-14 21:02                     ` Arjan van de Ven
2012-02-14 21:02                       ` Arjan van de Ven
2012-02-14 19:32                 ` Srivatsa S. Bhat
2012-02-14 19:32                   ` Srivatsa S. Bhat
2012-02-14 21:28                 ` Benjamin Herrenschmidt
2012-02-14 21:28                   ` Benjamin Herrenschmidt

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=20120131054155.371e8307@infradead.org \
    --to=arjan@infradead.org \
    --cc=a.p.zijlstra@chello.nl \
    --cc=akpm@linux-foundation.org \
    --cc=arjanvandeven@gmail.com \
    --cc=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=miltonm@bga.com \
    --cc=mingo@elte.hu \
    --cc=tglx@linutronix.de \
    --cc=torvalds@linux-foundation.org \
    /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.