diff options
author | Fernando Vazquez <fernando@intellilink.co.jp> | 2006-10-01 02:29:10 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-10-01 03:39:31 -0400 |
commit | bc03613decef0cc4d2f3a24f19fa5a868745715f (patch) | |
tree | 7c3343b100ae399df29815d709e4ab01768141ca /arch/i386 | |
parent | ce53af9496c625a8ae33526713be34a23756db19 (diff) |
[PATCH] stack overflow safe kdump: safe smp_send_nmi_allbutself()
Re-implement smp_send_nmi_allbutself() so that calls to smp_processor_id
(through send_IPI_allbutself) can be replaced with safe_smp_processor_id
without affecting other parts of the kernel (as suggested by Eric Biederman).
Signed-off-by: Fernando Vazquez <fernando@intellilink.co.jp>
Looks-reasonable-to: Andi Kleen <ak@muc.de>
Acked-by: "Eric W. Biederman" <ebiederm@xmission.com>
Cc: Vivek Goyal <vgoyal@in.ibm.com>
Cc: James Bottomley <James.Bottomley@steeleye.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'arch/i386')
-rw-r--r-- | arch/i386/kernel/crash.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/arch/i386/kernel/crash.c b/arch/i386/kernel/crash.c index 2dfc049dafa..144b4328896 100644 --- a/arch/i386/kernel/crash.c +++ b/arch/i386/kernel/crash.c | |||
@@ -134,7 +134,10 @@ static int crash_nmi_callback(struct notifier_block *self, | |||
134 | 134 | ||
135 | static void smp_send_nmi_allbutself(void) | 135 | static void smp_send_nmi_allbutself(void) |
136 | { | 136 | { |
137 | send_IPI_allbutself(NMI_VECTOR); | 137 | cpumask_t mask = cpu_online_map; |
138 | cpu_clear(safe_smp_processor_id(), mask); | ||
139 | if (!cpus_empty(mask)) | ||
140 | send_IPI_mask(mask, NMI_VECTOR); | ||
138 | } | 141 | } |
139 | 142 | ||
140 | static struct notifier_block crash_nmi_nb = { | 143 | static struct notifier_block crash_nmi_nb = { |