linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: will.deacon@arm.com (Will Deacon)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC PATCH 3.7.0-rc4] ARM:smp: introduce smp_notify_cpu_stop to fix kexec smp case
Date: Mon, 19 Nov 2012 10:58:48 +0000	[thread overview]
Message-ID: <20121119105848.GF3205@mudshark.cambridge.arm.com> (raw)
In-Reply-To: <50A5FBB7.9040001@st.com>

On Fri, Nov 16, 2012 at 08:39:19AM +0000, Srinivas KANDAGATLA wrote:
> On 15/11/12 21:18, Will Deacon wrote:
> >
> >  The bigger issue is co-ordinating with the
> > secondaries so you know when they're safely out of the way and you can
> > proceed with the kexec. Remember that you won't have working locks and you
> > can't cross-call a function because it won't actually return.
>
> Yes I agree and understand the limitation.
> The callback should simple and just relocate the secondary core to a
> safe place which in our case is a holding pen, which is safe and not in
> the system ram.
> After the callback, the secondary core will be spinning in the holding
> pen which will be released once they get a matching cpu-id.
> And this approach works perfectly ok on our chips and I guess should
> work for other chips aswell.

But how do you know when the callback is complete? That's the part that is
tricky as you need to avoid clobbering the kernel image before you know for
sure that all the secondaries are out of the way. I think you either need
some horrible homebrew locking primitives or something in hardware to signal
back to the primary CPU.

> > If you can solve the synchronisation problem then we can think about adding
> > these hooks.
>
> I think the call back code should not do anything complex other than few
> lines of assembly or jump to a holding pen, this way it does not need
> any synchronization calls or system infrastructure.

See above.

Will

  reply	other threads:[~2012-11-19 10:58 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-15 18:17 [RFC PATCH 3.7.0-rc4] ARM:smp: introduce smp_notify_cpu_stop to fix kexec smp case Srinivas KANDAGATLA
2012-11-15 21:18 ` Will Deacon
2012-11-16  8:39   ` Srinivas KANDAGATLA
2012-11-19 10:58     ` Will Deacon [this message]
2012-11-19 12:07       ` Srinivas KANDAGATLA
2012-11-19 12:16         ` Will Deacon

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=20121119105848.GF3205@mudshark.cambridge.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).