From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Yong Zhang Subject: [PATCH 2/2] [S390] irq: fix show_interrupts() vs cpu hotplug Date: Wed, 27 Jul 2011 12:56:23 +0800 Message-Id: <1311742583-18633-2-git-send-email-yong.zhang0@gmail.com> In-Reply-To: <1311742583-18633-1-git-send-email-yong.zhang0@gmail.com> References: <20110726163424.GC2576@osiris.boeblingen.de.ibm.com> <1311742583-18633-1-git-send-email-yong.zhang0@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-Archive: List-Post: To: linux-s390@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Martin Schwidefsky , Heiko Carstens List-ID: The current usage of get_online_cpus()/put_online_cpus() in show_interrupts() is not correct since show_interrupts() will be called nr_irqs times, during that period, cpu hotplug could also happen. Since have moved the protection to upper(patch#1), the pair of get_online_cpus()/put_online_cpus() could be removed here. Signed-off-by: Yong Zhang Cc: Martin Schwidefsky Cc: Heiko Carstens --- arch/s390/kernel/irq.c | 2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/arch/s390/kernel/irq.c b/arch/s390/kernel/irq.c index e3264f6..57c036d 100644 --- a/arch/s390/kernel/irq.c +++ b/arch/s390/kernel/irq.c @@ -63,7 +63,6 @@ int show_interrupts(struct seq_file *p, void *v) { int i = *(loff_t *) v, j; - get_online_cpus(); if (i == 0) { seq_puts(p, " "); for_each_online_cpu(j) @@ -83,7 +82,6 @@ int show_interrupts(struct seq_file *p, void *v) seq_printf(p, " %s", intrclass_names[i].desc); seq_putc(p, '\n'); } - put_online_cpus(); return 0; } -- 1.7.4.1