diff options
| author | Steven Miao <realmz6@gmail.com> | 2011-11-25 14:25:30 +0800 |
|---|---|---|
| committer | Bob Liu <lliubbo@gmail.com> | 2012-01-09 10:26:15 +0800 |
| commit | a2eff9dd8bca6d03bc3c87790bac3fdb4fe6dbf0 (patch) | |
| tree | 2218c5d6b9d1258f1f8c8c691ae210abe41c44e2 /arch/blackfin | |
| parent | 7d157fb02bc3f4dc74e6830725864ba501d92da7 (diff) | |
| download | lwn-a2eff9dd8bca6d03bc3c87790bac3fdb4fe6dbf0.tar.gz lwn-a2eff9dd8bca6d03bc3c87790bac3fdb4fe6dbf0.zip | |
blackfin: smp: fix msg queue overflow issue
disable preemption when icache flush and use wait mode cross call, hold the msg
queue lock while handle cross function call to avoid overflow
Signed-off-by: Steven Miao <realmz6@gmail.com>
Signed-off-by: Bob Liu <lliubbo@gmail.com>
Diffstat (limited to 'arch/blackfin')
| -rw-r--r-- | arch/blackfin/mach-common/smp.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/arch/blackfin/mach-common/smp.c b/arch/blackfin/mach-common/smp.c index d3464053dfdb..ac8f8a43158c 100644 --- a/arch/blackfin/mach-common/smp.c +++ b/arch/blackfin/mach-common/smp.c @@ -193,14 +193,10 @@ static irqreturn_t ipi_handler_int1(int irq, void *dev_instance) scheduler_ipi(); break; case BFIN_IPI_CALL_FUNC: - spin_unlock_irqrestore(&msg_queue->lock, flags); ipi_call_function(cpu, msg); - spin_lock_irqsave(&msg_queue->lock, flags); break; case BFIN_IPI_CPU_STOP: - spin_unlock_irqrestore(&msg_queue->lock, flags); ipi_cpu_stop(cpu); - spin_lock_irqsave(&msg_queue->lock, flags); break; default: printk(KERN_CRIT "CPU%u: Unknown IPI message 0x%lx\n", @@ -482,8 +478,10 @@ void smp_icache_flush_range_others(unsigned long start, unsigned long end) smp_flush_data.start = start; smp_flush_data.end = end; - if (smp_call_function(&ipi_flush_icache, &smp_flush_data, 0)) + preempt_disable(); + if (smp_call_function(&ipi_flush_icache, &smp_flush_data, 1)) printk(KERN_WARNING "SMP: failed to run I-cache flush request on other CPUs\n"); + preempt_enable(); } EXPORT_SYMBOL_GPL(smp_icache_flush_range_others); |