diff options
| author | Heiko Carstens <heiko.carstens@de.ibm.com> | 2014-04-04 05:23:03 -0400 |
|---|---|---|
| committer | Martin Schwidefsky <schwidefsky@de.ibm.com> | 2014-04-09 04:19:12 -0400 |
| commit | e7c46c66dbd1727f7de52f54c4acefb692b24f37 (patch) | |
| tree | 8b72c013eaad430227ef9284697085d884a46de6 | |
| parent | a8a934e44f2bd0ed613e1aa0471e3478c6a9228a (diff) | |
s390/smp: fix smp_stop_cpu() for !CONFIG_SMP
smp_stop_cpu() should stop the current cpu even for !CONFIG_SMP.
Otherwise machine_halt() will return and and the machine generates a
panic instread of simply stopping the current cpu:
Kernel panic - not syncing: Attempted to kill init! exitcode=0x00000000
CPU: 0 PID: 1 Comm: systemd-shutdow Not tainted 3.14.0-01527-g2b6ef16a6bc5 #10
[...]
Call Trace:
([<0000000000110db0>] show_trace+0xf8/0x158)
[<0000000000110e7a>] show_stack+0x6a/0xe8
[<000000000074dba8>] panic+0xe4/0x268
[<0000000000140570>] do_exit+0xa88/0xb2c
[<000000000016e12c>] SyS_reboot+0x1f0/0x234
[<000000000075da70>] sysc_nr_ok+0x22/0x28
[<000000007d5a09b4>] 0x7d5a09b4
Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
| -rw-r--r-- | arch/s390/include/asm/sigp.h | 19 | ||||
| -rw-r--r-- | arch/s390/include/asm/smp.h | 13 | ||||
| -rw-r--r-- | arch/s390/kernel/smp.c | 15 |
3 files changed, 31 insertions, 16 deletions
diff --git a/arch/s390/include/asm/sigp.h b/arch/s390/include/asm/sigp.h index d091aa1aaf11..bf9c823d4020 100644 --- a/arch/s390/include/asm/sigp.h +++ b/arch/s390/include/asm/sigp.h | |||
| @@ -31,4 +31,23 @@ | |||
| 31 | #define SIGP_STATUS_INCORRECT_STATE 0x00000200UL | 31 | #define SIGP_STATUS_INCORRECT_STATE 0x00000200UL |
| 32 | #define SIGP_STATUS_NOT_RUNNING 0x00000400UL | 32 | #define SIGP_STATUS_NOT_RUNNING 0x00000400UL |
| 33 | 33 | ||
| 34 | #ifndef __ASSEMBLY__ | ||
| 35 | |||
| 36 | static inline int __pcpu_sigp(u16 addr, u8 order, u32 parm, u32 *status) | ||
| 37 | { | ||
| 38 | register unsigned int reg1 asm ("1") = parm; | ||
| 39 | int cc; | ||
| 40 | |||
| 41 | asm volatile( | ||
| 42 | " sigp %1,%2,0(%3)\n" | ||
| 43 | " ipm %0\n" | ||
| 44 | " srl %0,28\n" | ||
| 45 | : "=d" (cc), "+d" (reg1) : "d" (addr), "a" (order) : "cc"); | ||
| 46 | if (status && cc == 1) | ||
| 47 | *status = reg1; | ||
| 48 | return cc; | ||
| 49 | } | ||
| 50 | |||
| 51 | #endif /* __ASSEMBLY__ */ | ||
| 52 | |||
| 34 | #endif /* __S390_ASM_SIGP_H */ | 53 | #endif /* __S390_ASM_SIGP_H */ |
diff --git a/arch/s390/include/asm/smp.h b/arch/s390/include/asm/smp.h index 160779394096..21703f85b48d 100644 --- a/arch/s390/include/asm/smp.h +++ b/arch/s390/include/asm/smp.h | |||
| @@ -7,6 +7,8 @@ | |||
| 7 | #ifndef __ASM_SMP_H | 7 | #ifndef __ASM_SMP_H |
| 8 | #define __ASM_SMP_H | 8 | #define __ASM_SMP_H |
| 9 | 9 | ||
| 10 | #include <asm/sigp.h> | ||
| 11 | |||
| 10 | #ifdef CONFIG_SMP | 12 | #ifdef CONFIG_SMP |
| 11 | 13 | ||
| 12 | #include <asm/lowcore.h> | 14 | #include <asm/lowcore.h> |
| @@ -50,9 +52,18 @@ static inline int smp_store_status(int cpu) { return 0; } | |||
| 50 | static inline int smp_vcpu_scheduled(int cpu) { return 1; } | 52 | static inline int smp_vcpu_scheduled(int cpu) { return 1; } |
| 51 | static inline void smp_yield_cpu(int cpu) { } | 53 | static inline void smp_yield_cpu(int cpu) { } |
| 52 | static inline void smp_yield(void) { } | 54 | static inline void smp_yield(void) { } |
| 53 | static inline void smp_stop_cpu(void) { } | ||
| 54 | static inline void smp_fill_possible_mask(void) { } | 55 | static inline void smp_fill_possible_mask(void) { } |
| 55 | 56 | ||
| 57 | static inline void smp_stop_cpu(void) | ||
| 58 | { | ||
| 59 | u16 pcpu = stap(); | ||
| 60 | |||
| 61 | for (;;) { | ||
| 62 | __pcpu_sigp(pcpu, SIGP_STOP, 0, NULL); | ||
| 63 | cpu_relax(); | ||
| 64 | } | ||
| 65 | } | ||
| 66 | |||
| 56 | #endif /* CONFIG_SMP */ | 67 | #endif /* CONFIG_SMP */ |
| 57 | 68 | ||
| 58 | #ifdef CONFIG_HOTPLUG_CPU | 69 | #ifdef CONFIG_HOTPLUG_CPU |
diff --git a/arch/s390/kernel/smp.c b/arch/s390/kernel/smp.c index 512ce1cde2a4..86e65ec3422b 100644 --- a/arch/s390/kernel/smp.c +++ b/arch/s390/kernel/smp.c | |||
| @@ -82,21 +82,6 @@ DEFINE_MUTEX(smp_cpu_state_mutex); | |||
| 82 | /* | 82 | /* |
| 83 | * Signal processor helper functions. | 83 | * Signal processor helper functions. |
| 84 | */ | 84 | */ |
| 85 | static inline int __pcpu_sigp(u16 addr, u8 order, u32 parm, u32 *status) | ||
| 86 | { | ||
| 87 | register unsigned int reg1 asm ("1") = parm; | ||
| 88 | int cc; | ||
| 89 | |||
| 90 | asm volatile( | ||
| 91 | " sigp %1,%2,0(%3)\n" | ||
| 92 | " ipm %0\n" | ||
| 93 | " srl %0,28\n" | ||
| 94 | : "=d" (cc), "+d" (reg1) : "d" (addr), "a" (order) : "cc"); | ||
| 95 | if (status && cc == 1) | ||
| 96 | *status = reg1; | ||
| 97 | return cc; | ||
| 98 | } | ||
| 99 | |||
| 100 | static inline int __pcpu_sigp_relax(u16 addr, u8 order, u32 parm, u32 *status) | 85 | static inline int __pcpu_sigp_relax(u16 addr, u8 order, u32 parm, u32 *status) |
| 101 | { | 86 | { |
| 102 | int cc; | 87 | int cc; |
