All of lore.kernel.org
 help / color / mirror / Atom feed
From: will.deacon@arm.com (Will Deacon)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH][ ARM cpu hotplug 1/2 ] extract common code for arm cpu	hotplug
Date: Tue, 30 Nov 2010 14:05:38 -0000	[thread overview]
Message-ID: <004701cb9097$ab173f40$0145bdc0$@deacon@arm.com> (raw)
In-Reply-To: <20101130115148.GB31054@n2100.arm.linux.org.uk>

Russell,

> On Tue, Nov 30, 2010 at 11:03:31AM +0000, Russell King - ARM Linux wrote:
> > I see no point to what is being proposed in this thread.  It's _soo_
> > little code that the platforms have to implement that it really is
> > not worth the effort.

[...]

> Note also that most implementations of this get it wrong.  Realview
> assumes ARMv6 MPcore layout of the auxcontrol register, which is
> cut'n'pasted into s5pv310 (ARMv7) and Tegra (ARMv7) without thinking.
> The only implementation which has put some thought into it is the
> UX500 one, which is extremely simple.
> 
> So, out of the four existing implementations, at least two need fixing
> to be different, and I'll bet that they do have some way to power down
> the offlined CPUs for greater power savings.

I have patches that fix this for RealView and Versatile Express, I posted
them to the list in the past (note that these are now out of date):

http://lists.infradead.org/pipermail/linux-arm-kernel/2010-September/026160.html

I have a newer version here:

git://linux-arm.org/linux-2.6-wd.git cpu-hotplug

but it relies on the Versatile Express multi-tile patches that I'm posting at
the moment. If the hotplug APIs aren't about to change fundamentally, then
I'll repost these patches once the multi-tile stuff is accepted.

Will

  reply	other threads:[~2010-11-30 14:05 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-29  9:48 [PATCH][ ARM cpu hotplug 1/2 ] extract common code for arm cpu hotplug Vincent Guittot
2010-11-29  9:54 ` Vincent Guittot
2010-11-29 10:41   ` Russell King - ARM Linux
2010-11-29 17:27     ` Vincent Guittot
2010-11-29 17:27       ` Vincent Guittot
2010-11-29 19:24       ` [PATCH][ ARM cpu hotplug 1/2 ] extract common code for arm cpu Russell King - ARM Linux
2010-11-29 19:24         ` [PATCH][ ARM cpu hotplug 1/2 ] extract common code for arm cpu hotplug Russell King - ARM Linux
2010-11-30 10:47         ` Amit Kucheria
2010-11-30 11:03           ` Russell King - ARM Linux
2010-11-30 11:51             ` Russell King - ARM Linux
2010-11-30 14:05               ` Will Deacon [this message]
2010-11-30 16:24             ` Rob Herring
2010-11-30 16:58               ` Russell King - ARM Linux

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='004701cb9097$ab173f40$0145bdc0$@deacon@arm.com' \
    --to=will.deacon@arm.com \
    --cc=linux-arm-kernel@lists.infradead.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.