Re: [PATCH] i386 kexec: Allow the kexec on panic support to compile on voyager.

From: Horms
Date: Tue Jul 11 2006 - 08:28:38 EST


On Mon, 10 Jul 2006 16:37:49 -0600, Eric W. Biederman wrote:
>
> This patch removes the foolish assumption that SMP implied local
> apics. That assumption is not-true on the Voyager subarch. This
> makes that dependency explicit, and allows the code to build.

Doesn't only a small portion of the code in question rely
on CONFIG_X86_LOCAL_APIC? Is just a workaround until proper
voager support materialises?

> What gets disabled is just an optimization to get better crash
> dumps so the support should work if there is a kernel that will
> initialization on the voyager subarch under those harsh conditions.

By that do you mean, a crash kernel that is able to boot even
though the non-crashing CPUs have not been shutdown?

> Hopefully we can figure out how to initialize apics in init_IRQ
> and remove the need to disable io_apics and this dependency.

That does sound nice. Do you have any ideas on how that could be
made to happen?

> Signed-off-by: Eric W. Biederman <ebiederm@xxxxxxxxxxxx>
> ---
> arch/i386/kernel/crash.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/arch/i386/kernel/crash.c b/arch/i386/kernel/crash.c
> index f10da80..67d297d 100644
> --- a/arch/i386/kernel/crash.c
> +++ b/arch/i386/kernel/crash.c
> @@ -92,7 +92,7 @@ static void crash_save_self(struct pt_re
> crash_save_this_cpu(regs, cpu);
> }
>
> -#ifdef CONFIG_SMP
> +#if defined(CONFIG_SMP) && defined(CONFIG_X86_LOCAL_APIC)
> static atomic_t waiting_for_crash_ipi;
>
> static int crash_nmi_callback(struct notifier_block *self,



--
Horms
H: http://www.vergenet.net/~horms/
W: http://www.valinux.co.jp/en/

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/