diff options
author | Ralf Baechle <ralf@linux-mips.org> | 2011-05-28 10:27:59 -0400 |
---|---|---|
committer | Ralf Baechle <ralf@linux-mips.org> | 2011-07-20 18:12:10 -0400 |
commit | 852fe3105e94ca26d1b3df7e2cb6878ebdd67608 (patch) | |
tree | b38949922e762e4310b105eec24a9df5352804e4 /arch/mips/include | |
parent | b12acf163f6e52ff7d41aca51382dde17c506068 (diff) |
MIPS: Malta: Fix crash in SMP kernel on non-CMP systems.
Since 6be63bbbdab66b9185dc6f67c8b1bacb6f37f946 (lmo) rsp.
af3a1f6f4813907e143f87030cde67a9971db533 (kernel.org) the Malta code does
no longer probe for presence of GCMP if CMP is not configured. This means
that the variable gcmp_present well be left at its default value of -1
which normally is meant to indicate that GCMP has not yet been mmapped.
This non-zero value is now interpreted as GCMP being present resulting
in a write attempt to a GCMP register resulting in a crash.
Reported and a build fix on top of my fix by Rob Landley <rob@landley.net>.
Reported-by: Rob Landley <rob@landley.net>
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
Patchwork: https://patchwork.linux-mips.org/patch/2413/
Diffstat (limited to 'arch/mips/include')
-rw-r--r-- | arch/mips/include/asm/smp-ops.h | 41 |
1 files changed, 38 insertions, 3 deletions
diff --git a/arch/mips/include/asm/smp-ops.h b/arch/mips/include/asm/smp-ops.h index 9e09af34c8a8..48b03fff31e5 100644 --- a/arch/mips/include/asm/smp-ops.h +++ b/arch/mips/include/asm/smp-ops.h | |||
@@ -56,8 +56,43 @@ static inline void register_smp_ops(struct plat_smp_ops *ops) | |||
56 | 56 | ||
57 | #endif /* !CONFIG_SMP */ | 57 | #endif /* !CONFIG_SMP */ |
58 | 58 | ||
59 | extern struct plat_smp_ops up_smp_ops; | 59 | static inline int register_up_smp_ops(void) |
60 | extern struct plat_smp_ops cmp_smp_ops; | 60 | { |
61 | extern struct plat_smp_ops vsmp_smp_ops; | 61 | #ifdef CONFIG_SMP_UP |
62 | extern struct plat_smp_ops up_smp_ops; | ||
63 | |||
64 | register_smp_ops(&up_smp_ops); | ||
65 | |||
66 | return 0; | ||
67 | #else | ||
68 | return -ENODEV; | ||
69 | #endif | ||
70 | } | ||
71 | |||
72 | static inline int register_cmp_smp_ops(void) | ||
73 | { | ||
74 | #ifdef CONFIG_MIPS_CMP | ||
75 | extern struct plat_smp_ops cmp_smp_ops; | ||
76 | |||
77 | register_smp_ops(&cmp_smp_ops); | ||
78 | |||
79 | return 0; | ||
80 | #else | ||
81 | return -ENODEV; | ||
82 | #endif | ||
83 | } | ||
84 | |||
85 | static inline int register_vsmp_smp_ops(void) | ||
86 | { | ||
87 | #ifdef CONFIG_MIPS_MT_SMP | ||
88 | extern struct plat_smp_ops vsmp_smp_ops; | ||
89 | |||
90 | register_smp_ops(&vsmp_smp_ops); | ||
91 | |||
92 | return 0; | ||
93 | #else | ||
94 | return -ENODEV; | ||
95 | #endif | ||
96 | } | ||
62 | 97 | ||
63 | #endif /* __ASM_SMP_OPS_H */ | 98 | #endif /* __ASM_SMP_OPS_H */ |