diff options
author | Marcin Nowakowski <marcin.nowakowski@imgtec.com> | 2016-12-02 03:58:28 -0500 |
---|---|---|
committer | Ralf Baechle <ralf@linux-mips.org> | 2017-01-03 10:34:44 -0500 |
commit | c83c2eed67e578576acf08611bfb630bd199714b (patch) | |
tree | aa4ba36a2f1ca8cc8a6057f257a57c81e6669281 | |
parent | 109c32ffd89d64dd99a775f2f50443bee38b63e9 (diff) |
MIPS: kexec: remove SMP_DUMP
SMP_DUMP has been added as a new IPI signal when kexec support was added
for Cavium Octeon CPUs ('commit 7aa1c8f47e7e ("MIPS: kdump: Add support")'.
However, the new signal doesn't appear to ever have a proper handler
added (octeon_message_functions[] array has an empty handler for it),
and generic IPI handlers now trigger a BUG() on unhandled signal.
As the method is unused remove it completely and replace its only
invocation with a smp_call_function().
[ralf@linux-mips.org: Renumber SMP_ASK_C0COUNT to avoid numbering gaps.]
Signed-off-by: Marcin Nowakowski <marcin.nowakowski@imgtec.com>
Cc: linux-mips@linux-mips.org
Patchwork: https://patchwork.linux-mips.org/patch/14630/
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
-rw-r--r-- | arch/mips/include/asm/smp.h | 8 | ||||
-rw-r--r-- | arch/mips/kernel/crash.c | 2 | ||||
-rw-r--r-- | arch/mips/kernel/smp.c | 17 |
3 files changed, 2 insertions, 25 deletions
diff --git a/arch/mips/include/asm/smp.h b/arch/mips/include/asm/smp.h index f8c5faa93584..98a117a05fbc 100644 --- a/arch/mips/include/asm/smp.h +++ b/arch/mips/include/asm/smp.h | |||
@@ -42,9 +42,7 @@ extern int __cpu_logical_map[NR_CPUS]; | |||
42 | #define SMP_CALL_FUNCTION 0x2 | 42 | #define SMP_CALL_FUNCTION 0x2 |
43 | /* Octeon - Tell another core to flush its icache */ | 43 | /* Octeon - Tell another core to flush its icache */ |
44 | #define SMP_ICACHE_FLUSH 0x4 | 44 | #define SMP_ICACHE_FLUSH 0x4 |
45 | /* Used by kexec crashdump to save all cpu's state */ | 45 | #define SMP_ASK_C0COUNT 0x8 |
46 | #define SMP_DUMP 0x8 | ||
47 | #define SMP_ASK_C0COUNT 0x10 | ||
48 | 46 | ||
49 | /* Mask of CPUs which are currently definitely operating coherently */ | 47 | /* Mask of CPUs which are currently definitely operating coherently */ |
50 | extern cpumask_t cpu_coherent_mask; | 48 | extern cpumask_t cpu_coherent_mask; |
@@ -111,8 +109,4 @@ static inline void arch_send_call_function_ipi_mask(const struct cpumask *mask) | |||
111 | mp_ops->send_ipi_mask(mask, SMP_CALL_FUNCTION); | 109 | mp_ops->send_ipi_mask(mask, SMP_CALL_FUNCTION); |
112 | } | 110 | } |
113 | 111 | ||
114 | #if defined(CONFIG_KEXEC) | ||
115 | extern void (*dump_ipi_function_ptr)(void *); | ||
116 | void dump_send_ipi(void (*dump_ipi_callback)(void *)); | ||
117 | #endif | ||
118 | #endif /* __ASM_SMP_H */ | 112 | #endif /* __ASM_SMP_H */ |
diff --git a/arch/mips/kernel/crash.c b/arch/mips/kernel/crash.c index 1723b1762297..5a71518be0f1 100644 --- a/arch/mips/kernel/crash.c +++ b/arch/mips/kernel/crash.c | |||
@@ -56,7 +56,7 @@ static void crash_kexec_prepare_cpus(void) | |||
56 | 56 | ||
57 | ncpus = num_online_cpus() - 1;/* Excluding the panic cpu */ | 57 | ncpus = num_online_cpus() - 1;/* Excluding the panic cpu */ |
58 | 58 | ||
59 | dump_send_ipi(crash_shutdown_secondary); | 59 | smp_call_function(crash_shutdown_secondary, NULL, 0); |
60 | smp_wmb(); | 60 | smp_wmb(); |
61 | 61 | ||
62 | /* | 62 | /* |
diff --git a/arch/mips/kernel/smp.c b/arch/mips/kernel/smp.c index 0a831f63b0ec..8c60a296294c 100644 --- a/arch/mips/kernel/smp.c +++ b/arch/mips/kernel/smp.c | |||
@@ -638,23 +638,6 @@ void flush_tlb_one(unsigned long vaddr) | |||
638 | EXPORT_SYMBOL(flush_tlb_page); | 638 | EXPORT_SYMBOL(flush_tlb_page); |
639 | EXPORT_SYMBOL(flush_tlb_one); | 639 | EXPORT_SYMBOL(flush_tlb_one); |
640 | 640 | ||
641 | #if defined(CONFIG_KEXEC) | ||
642 | void (*dump_ipi_function_ptr)(void *) = NULL; | ||
643 | void dump_send_ipi(void (*dump_ipi_callback)(void *)) | ||
644 | { | ||
645 | int i; | ||
646 | int cpu = smp_processor_id(); | ||
647 | |||
648 | dump_ipi_function_ptr = dump_ipi_callback; | ||
649 | smp_mb(); | ||
650 | for_each_online_cpu(i) | ||
651 | if (i != cpu) | ||
652 | mp_ops->send_ipi_single(i, SMP_DUMP); | ||
653 | |||
654 | } | ||
655 | EXPORT_SYMBOL(dump_send_ipi); | ||
656 | #endif | ||
657 | |||
658 | #ifdef CONFIG_GENERIC_CLOCKEVENTS_BROADCAST | 641 | #ifdef CONFIG_GENERIC_CLOCKEVENTS_BROADCAST |
659 | 642 | ||
660 | static DEFINE_PER_CPU(atomic_t, tick_broadcast_count); | 643 | static DEFINE_PER_CPU(atomic_t, tick_broadcast_count); |