From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753844Ab2IZIkj (ORCPT ); Wed, 26 Sep 2012 04:40:39 -0400 Received: from mga03.intel.com ([143.182.124.21]:9573 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752910Ab2IZIk3 (ORCPT ); Wed, 26 Sep 2012 04:40:29 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.80,489,1344236400"; d="scan'208";a="197264597" Subject: [PATCH RESEND] x86/fixup_irq: Clean the offlining CPU from the irq affinity mask From: Chuansheng Liu To: srivatsa.bhat@linux.vnet.ibm.com, tglx@linutronix.de, mingo@redhat.com, x86@kernel.org Cc: linux-kernel@vger.kernel.org, yanmin_zhang@linux.intel.com Content-Type: text/plain; charset="UTF-8" Date: Thu, 27 Sep 2012 01:38:12 +0800 Message-ID: <1348681092.19514.10.camel@cliu38-desktop-build> Mime-Version: 1.0 X-Mailer: Evolution 2.28.3 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org When one CPU is going offline, and fixup_irqs() will re-set the irq affinity in some cases, we should clean the offlining CPU from the irq affinity. The reason is setting offlining CPU as of the affinity is useless. Moreover, the smp_affinity value will be confusing when the offlining CPU come back again. Example: For irq 93 with 4 CPUS, the default affinity f(1111), normal cases: 4 CPUS will receive the irq93 interrupts. When echo 0 > /sys/devices/system/cpu/cpu3/online, just CPU0,1,2 will receive the interrupts. But after the CPU3 is online again, we will not set affinity,the result will be: the smp_affinity is f, but still just CPU0,1,2 can receive the interrupts. So we should clean the offlining CPU from irq affinity mask in fixup_irqs(). Reviewed-by: Srivatsa S. Bhat Signed-off-by: liu chuansheng --- arch/x86/kernel/irq.c | 8 ++++++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/arch/x86/kernel/irq.c b/arch/x86/kernel/irq.c index d44f782..08bb905 100644 --- a/arch/x86/kernel/irq.c +++ b/arch/x86/kernel/irq.c @@ -239,6 +239,7 @@ void fixup_irqs(void) struct irq_desc *desc; struct irq_data *data; struct irq_chip *chip; + int cpu = smp_processor_id(); for_each_irq_desc(irq, desc) { int break_affinity = 0; @@ -277,8 +278,11 @@ void fixup_irqs(void) if (!irqd_can_move_in_process_context(data) && chip->irq_mask) chip->irq_mask(data); - if (chip->irq_set_affinity) - chip->irq_set_affinity(data, affinity, true); + if ((chip->irq_set_affinity) && + !chip->irq_set_affinity(data, affinity, true)) { + if (cpumask_test_cpu(cpu, data->affinity)) + cpumask_clear_cpu(cpu, data->affinity); + } else if (!(warned++)) set_affinity = 0; -- 1.7.0.4