diff options
author | Alex Chiang <achiang@hp.com> | 2009-02-09 13:16:57 -0500 |
---|---|---|
committer | Tony Luck <aegl@agluck-desktop.(none)> | 2009-02-19 14:32:50 -0500 |
commit | c0acdea21437eff134cc3486bdc52907d9086af1 (patch) | |
tree | 08ad15f5c5acc26e2420cc6f46afceb487a9fa0b | |
parent | 66db2e6331612bbec193a358885854330596a92a (diff) |
[IA64] Remove redundant cpu_clear() in __cpu_disable path
The second call to cpu_clear() is redundant, as we've already removed
the CPU from cpu_online_map before calling migrate_platform_irqs().
Signed-off-by: Alex Chiang <achiang@hp.com>
Signed-off-by: Tony Luck <aegl@agluck-desktop.(none)>
-rw-r--r-- | arch/ia64/kernel/smpboot.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/arch/ia64/kernel/smpboot.c b/arch/ia64/kernel/smpboot.c index 2ec5bbff461e..52290547c85b 100644 --- a/arch/ia64/kernel/smpboot.c +++ b/arch/ia64/kernel/smpboot.c | |||
@@ -740,11 +740,10 @@ int __cpu_disable(void) | |||
740 | 740 | ||
741 | if (migrate_platform_irqs(cpu)) { | 741 | if (migrate_platform_irqs(cpu)) { |
742 | cpu_set(cpu, cpu_online_map); | 742 | cpu_set(cpu, cpu_online_map); |
743 | return (-EBUSY); | 743 | return -EBUSY; |
744 | } | 744 | } |
745 | 745 | ||
746 | remove_siblinginfo(cpu); | 746 | remove_siblinginfo(cpu); |
747 | cpu_clear(cpu, cpu_online_map); | ||
748 | fixup_irqs(); | 747 | fixup_irqs(); |
749 | local_flush_tlb_all(); | 748 | local_flush_tlb_all(); |
750 | cpu_clear(cpu, cpu_callin_map); | 749 | cpu_clear(cpu, cpu_callin_map); |