Kexec/Kdump: kdump_disable_iosapic needs CONFIG_IOSAPIC

> -----Original Message-----
> From: linux-ia64-ownervger.kernel.org
> [mailto:linux-ia64-ownervger.kernel.org] On Behalf
Of Horms
> Sent: 2006Äê9ÔÂ20ÈÕ 10:43
> To: Zou, Nanhai
> Cc: Luck, Tony; Linux-IA64; fastboot
> Subject: Kexec/Kdump: kdump_disable_iosapic needs
CONFIG_IOSAPIC
> 
> kdump_disable_iosapic() doesn't exist without
CONFIG_IOSAPIC,
> calling it needs to be gaurded by a corresponding
#ifdef
> 
> This patch has been updated for IA64 Kexec/Kdump patch
for 2.6.18-rc6,
> http://permalink.gmane.org/gmane.linux.ports.ia64/14988
> 
> Signed-Off-By: Simon Horman <hormsverge.net.au>
> 
> diff --git a/arch/ia64/kernel/crash.c
b/arch/ia64/kernel/crash.c
> index ccc898c..b3f4915 100644
> --- a/arch/ia64/kernel/crash.c
> +++ b/arch/ia64/kernel/crash.c
>  -121,7 +121,9  machine_crash_shutdown(struct pt_regs *p
>  	 * In practice this means shooting down the other
cpus in
>  	 * an SMP system.
>  	 */
> +#ifdef CONFIG_IOSAPIC
>  	kexec_disable_iosapic();
> +#endif
>  #ifdef CONFIG_SMP
>  	kdump_smp_send_stop();
>  	if (kdump_wait_cpu_freeze() && kdump_on_init)
	{
>  -135,7 +137,9  static void
>  machine_kdump_on_init(void)
>  {
>  	local_irq_disable();
> +#ifdef CONFIG_IOSAPIC
>  	kexec_disable_iosapic();
> +#endif
>  	machine_kexec(ia64_kimage);
>  }
> 
> 
 CONFIG_IOSAPIC is only disabled when configure kernel for
simulator. 
 So I think we do not need this #ifdef unless we are using
kexec/kdump on a simulator.
 Thanks
 Zou Nan hai
> --
> Horms
>   H: http://www.vergenet.n
et/~horms/
>   W: http://www.valinux.co.jp
/en/
> 
> -
> To unsubscribe from this list: send the line
"unsubscribe linux-ia64" in
> the body of a message to majordomovger.kernel.org
> More majordomo info at  http://vge
r.kernel.org/majordomo-info.html
-
To unsubscribe from this list: send the line
"unsubscribe linux-ia64" in
the body of a message to majordomovger.kernel.org
More majordomo info at  http://vge
r.kernel.org/majordomo-info.html