[PATCH 1/8] arm64: Use cpu_ops for smp_stop
geoff at infradead.org
Tue May 13 15:27:10 PDT 2014
On Fri, 2014-05-09 at 09:44 +0100, Mark Rutland wrote:
> On Fri, May 09, 2014 at 01:48:17AM +0100, Geoff Levand wrote:
> > + /* If we have the cup_ops use them. */
> > +
> > + if (cpu_ops[cpu]->cpu_disable && cpu_ops[cpu]->cpu_die
> > + && !cpu_ops[cpu]->cpu_disable(cpu))
> > + cpu_ops[cpu]->cpu_die(cpu);
> For PSCI 0.2 support, we're going to need a cpu_kill callback which we
> can't call from the dying CPU. Specifically, we'll need to poll
> CPU_AFFINITY_INFO to ensure that secondaries have _actually_ left the
> kernel and aren't going to be adversely affected by the kernel text
> getting clobbered.
> As we're going to wire that up to the cpu hotplug infrastructure it
> would be nice to perform the hotplug for kexec by reusing the generic
> hotplug infrastructure rather than calling portions of the arm64
> implementation directly.
OK, is there somewhere I can see that new code, and when do you expect
it to be merged?
> > +
> > + /* Spin here if the cup_ops fail. */
> > +
> > while (1)
> > cpu_relax();
> This seems very dodgy to me. If a CPU doesn't actually die it's going to
> be spinning in some memory that we may later clobber. At that point the
> CPU will do arbitrarily bad things when it begins executing whatever its
> currently executing instructions (or vectors) were replaced by, and you
> will waste hours trying to figure out what went wrong (See 8121cf312a19
> "ARM: 7766/1: versatile: don't mark pen as __INIT" for a similar mess).
> If we fail to hotplug a CPU we at minimum need some acknowledgement that
> we failed. I would rather we failed to kexec entirely in that case.
This loop is for the non-hotplug power-off shutdown. This whole smp_stop
support needs to be reconsidered for a hotplug spin-table re-work.
More information about the kexec