On Tue, Sep 09, 2014 at 11:49:05PM +0100, Geoff Levand wrote: > The arm64 implementation of kexec expects an operational cpu_die method of > struct cpu_operations, so add defined(CONFIG_KEXEC) to the preprocessor > conditional that enables cpu_die.
I very much do not like this, as I explained in previous postings and my comments on other patches in this series. Please use the hotplug infrastructure, and have kexec depend on CPU_HOTPLUG || !SMP. There is no reason for these files to have any knowledge of kexec whatsoever. Thanks, Mark > > Signed-off-by: Geoff Levand <[email protected]> > --- > arch/arm64/include/asm/cpu_ops.h | 2 +- > arch/arm64/kernel/psci.c | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/arch/arm64/include/asm/cpu_ops.h > b/arch/arm64/include/asm/cpu_ops.h > index d7b4b38..9fd7281 100644 > --- a/arch/arm64/include/asm/cpu_ops.h > +++ b/arch/arm64/include/asm/cpu_ops.h > @@ -50,7 +50,7 @@ struct cpu_operations { > int (*cpu_prepare)(unsigned int); > int (*cpu_boot)(unsigned int); > void (*cpu_postboot)(void); > -#ifdef CONFIG_HOTPLUG_CPU > +#if defined(CONFIG_HOTPLUG_CPU) || defined(CONFIG_KEXEC) > int (*cpu_disable)(unsigned int cpu); > void (*cpu_die)(unsigned int cpu); > int (*cpu_kill)(unsigned int cpu); > diff --git a/arch/arm64/kernel/psci.c b/arch/arm64/kernel/psci.c > index 5539547..81dbbc9 100644 > --- a/arch/arm64/kernel/psci.c > +++ b/arch/arm64/kernel/psci.c > @@ -380,7 +380,7 @@ static int cpu_psci_cpu_boot(unsigned int cpu) > return err; > } > > -#ifdef CONFIG_HOTPLUG_CPU > +#if defined(CONFIG_HOTPLUG_CPU) || defined(CONFIG_KEXEC) > static int cpu_psci_cpu_disable(unsigned int cpu) > { > /* Fail early if we don't have CPU_OFF support */ > @@ -442,7 +442,7 @@ const struct cpu_operations cpu_psci_ops = { > .cpu_init = cpu_psci_cpu_init, > .cpu_prepare = cpu_psci_cpu_prepare, > .cpu_boot = cpu_psci_cpu_boot, > -#ifdef CONFIG_HOTPLUG_CPU > +#if defined(CONFIG_HOTPLUG_CPU) || defined(CONFIG_KEXEC) > .cpu_disable = cpu_psci_cpu_disable, > .cpu_die = cpu_psci_cpu_die, > .cpu_kill = cpu_psci_cpu_kill, > -- > 1.9.1 > > > > _______________________________________________ > linux-arm-kernel mailing list > [email protected] > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > _______________________________________________ kexec mailing list [email protected] http://lists.infradead.org/mailman/listinfo/kexec
