Messages in this thread Patch in this message | | | Date | Mon, 9 Feb 2009 11:16:57 -0700 | From | Alex Chiang <> | Subject | [PATCH v2 2/2] 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().
Cc: stable@kernel.org Signed-off-by: Alex Chiang <achiang@hp.com> --- 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 2ec5bbf..5229054 100644 --- a/arch/ia64/kernel/smpboot.c +++ b/arch/ia64/kernel/smpboot.c @@ -740,11 +740,10 @@ int __cpu_disable(void) if (migrate_platform_irqs(cpu)) { cpu_set(cpu, cpu_online_map); - return (-EBUSY); + return -EBUSY; } remove_siblinginfo(cpu); - cpu_clear(cpu, cpu_online_map); fixup_irqs(); local_flush_tlb_all(); cpu_clear(cpu, cpu_callin_map); -- 1.6.0.1.161.g7f314
| |