diff options
author | Paul Mackerras <paulus@samba.org> | 2005-10-19 07:44:51 -0400 |
---|---|---|
committer | Paul Mackerras <paulus@samba.org> | 2005-10-19 07:44:51 -0400 |
commit | 7ed476d17f04473f70d796cb6c172bdcfcc9b8e5 (patch) | |
tree | bafacf9813cdbb27a9e4a125340c98415cdcb903 /arch/ppc/kernel | |
parent | 46aab8c584ea69cdbe1060f5eae5bf092763f710 (diff) |
ppc: Minor smp changes for consistency with ppc64
This makes platform code use the smp_ops variable directly instead
of ppc_md.smp_ops, removes the two unused `data' and `wait' arguments
from the *_message_pass() functions, and removes the call to the
never-implemented smp_ops->space_timers() function.
Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to 'arch/ppc/kernel')
-rw-r--r-- | arch/ppc/kernel/smp.c | 20 |
1 files changed, 8 insertions, 12 deletions
diff --git a/arch/ppc/kernel/smp.c b/arch/ppc/kernel/smp.c index 801c793fdec8..bc5bf1124836 100644 --- a/arch/ppc/kernel/smp.c +++ b/arch/ppc/kernel/smp.c | |||
@@ -51,7 +51,7 @@ EXPORT_SYMBOL(cpu_online_map); | |||
51 | EXPORT_SYMBOL(cpu_possible_map); | 51 | EXPORT_SYMBOL(cpu_possible_map); |
52 | 52 | ||
53 | /* SMP operations for this machine */ | 53 | /* SMP operations for this machine */ |
54 | static struct smp_ops_t *smp_ops; | 54 | struct smp_ops_t *smp_ops; |
55 | 55 | ||
56 | /* all cpu mappings are 1-1 -- Cort */ | 56 | /* all cpu mappings are 1-1 -- Cort */ |
57 | volatile unsigned long cpu_callin_map[NR_CPUS]; | 57 | volatile unsigned long cpu_callin_map[NR_CPUS]; |
@@ -74,11 +74,11 @@ extern void __save_cpu_setup(void); | |||
74 | #define PPC_MSG_XMON_BREAK 3 | 74 | #define PPC_MSG_XMON_BREAK 3 |
75 | 75 | ||
76 | static inline void | 76 | static inline void |
77 | smp_message_pass(int target, int msg, unsigned long data, int wait) | 77 | smp_message_pass(int target, int msg) |
78 | { | 78 | { |
79 | if (smp_ops){ | 79 | if (smp_ops) { |
80 | atomic_inc(&ipi_sent); | 80 | atomic_inc(&ipi_sent); |
81 | smp_ops->message_pass(target,msg,data,wait); | 81 | smp_ops->message_pass(target, msg); |
82 | } | 82 | } |
83 | } | 83 | } |
84 | 84 | ||
@@ -119,7 +119,7 @@ void smp_message_recv(int msg, struct pt_regs *regs) | |||
119 | void smp_send_tlb_invalidate(int cpu) | 119 | void smp_send_tlb_invalidate(int cpu) |
120 | { | 120 | { |
121 | if ( PVR_VER(mfspr(SPRN_PVR)) == 8 ) | 121 | if ( PVR_VER(mfspr(SPRN_PVR)) == 8 ) |
122 | smp_message_pass(MSG_ALL_BUT_SELF, PPC_MSG_INVALIDATE_TLB, 0, 0); | 122 | smp_message_pass(MSG_ALL_BUT_SELF, PPC_MSG_INVALIDATE_TLB); |
123 | } | 123 | } |
124 | 124 | ||
125 | void smp_send_reschedule(int cpu) | 125 | void smp_send_reschedule(int cpu) |
@@ -135,13 +135,13 @@ void smp_send_reschedule(int cpu) | |||
135 | */ | 135 | */ |
136 | /* This is only used if `cpu' is running an idle task, | 136 | /* This is only used if `cpu' is running an idle task, |
137 | so it will reschedule itself anyway... */ | 137 | so it will reschedule itself anyway... */ |
138 | smp_message_pass(cpu, PPC_MSG_RESCHEDULE, 0, 0); | 138 | smp_message_pass(cpu, PPC_MSG_RESCHEDULE); |
139 | } | 139 | } |
140 | 140 | ||
141 | #ifdef CONFIG_XMON | 141 | #ifdef CONFIG_XMON |
142 | void smp_send_xmon_break(int cpu) | 142 | void smp_send_xmon_break(int cpu) |
143 | { | 143 | { |
144 | smp_message_pass(cpu, PPC_MSG_XMON_BREAK, 0, 0); | 144 | smp_message_pass(cpu, PPC_MSG_XMON_BREAK); |
145 | } | 145 | } |
146 | #endif /* CONFIG_XMON */ | 146 | #endif /* CONFIG_XMON */ |
147 | 147 | ||
@@ -224,7 +224,7 @@ static int __smp_call_function(void (*func) (void *info), void *info, | |||
224 | spin_lock(&call_lock); | 224 | spin_lock(&call_lock); |
225 | call_data = &data; | 225 | call_data = &data; |
226 | /* Send a message to all other CPUs and wait for them to respond */ | 226 | /* Send a message to all other CPUs and wait for them to respond */ |
227 | smp_message_pass(target, PPC_MSG_CALL_FUNCTION, 0, 0); | 227 | smp_message_pass(target, PPC_MSG_CALL_FUNCTION); |
228 | 228 | ||
229 | /* Wait for response */ | 229 | /* Wait for response */ |
230 | timeout = 1000000; | 230 | timeout = 1000000; |
@@ -294,7 +294,6 @@ void __init smp_prepare_cpus(unsigned int max_cpus) | |||
294 | smp_store_cpu_info(smp_processor_id()); | 294 | smp_store_cpu_info(smp_processor_id()); |
295 | cpu_callin_map[smp_processor_id()] = 1; | 295 | cpu_callin_map[smp_processor_id()] = 1; |
296 | 296 | ||
297 | smp_ops = ppc_md.smp_ops; | ||
298 | if (smp_ops == NULL) { | 297 | if (smp_ops == NULL) { |
299 | printk("SMP not supported on this machine.\n"); | 298 | printk("SMP not supported on this machine.\n"); |
300 | return; | 299 | return; |
@@ -308,9 +307,6 @@ void __init smp_prepare_cpus(unsigned int max_cpus) | |||
308 | /* Backup CPU 0 state */ | 307 | /* Backup CPU 0 state */ |
309 | __save_cpu_setup(); | 308 | __save_cpu_setup(); |
310 | 309 | ||
311 | if (smp_ops->space_timers) | ||
312 | smp_ops->space_timers(num_cpus); | ||
313 | |||
314 | for_each_cpu(cpu) { | 310 | for_each_cpu(cpu) { |
315 | if (cpu == smp_processor_id()) | 311 | if (cpu == smp_processor_id()) |
316 | continue; | 312 | continue; |