Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1392151 > unrolled thread

[PATCH] ia64: Remove superfluous SMP function call

Started byAnna-Maria Gleixner <anna-maria@linutronix.de>
First post2016-05-02 13:50 +0200
Last post2016-05-02 22:30 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] ia64: Remove superfluous SMP function call Anna-Maria Gleixner <anna-maria@linutronix.de> - 2016-05-02 13:50 +0200
    RE: [PATCH] ia64: Remove superfluous SMP function call "Luck, Tony" <tony.luck@intel.com> - 2016-05-02 22:30 +0200

#1392151 — [PATCH] ia64: Remove superfluous SMP function call

FromAnna-Maria Gleixner <anna-maria@linutronix.de>
Date2016-05-02 13:50 +0200
Subject[PATCH] ia64: Remove superfluous SMP function call
Message-ID<rukwq-2DS-19@gated-at.bofh.it>
Since commit 3b9d6da67e11 ("cpu/hotplug: Fix rollback during error-out
in __cpu_disable()") it is ensured that callbacks of CPU_ONLINE and
CPU_DOWN_PREPARE are processed on the hotplugged CPU. Due to this SMP
function calls are no longer required.

Replace smp_call_function_single() with a direct call to
ia64_mca_cmc_vector_adjust(). The function itselfs handles disable and
enable interrupts, therefore the smp_call_function_single() calling
convention is not preserved.

Cc: Tony Luck <tony.luck@intel.com>
Cc: Fenghua Yu <fenghua.yu@intel.com>
Cc: linux-ia64@vger.kernel.org
Signed-off-by: Anna-Maria Gleixner <anna-maria@linutronix.de>
---
 arch/ia64/kernel/mca.c |    5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

--- a/arch/ia64/kernel/mca.c
+++ b/arch/ia64/kernel/mca.c
@@ -1904,13 +1904,10 @@ static int mca_cpu_callback(struct notif
 				      unsigned long action,
 				      void *hcpu)
 {
-	int hotcpu = (unsigned long) hcpu;
-
 	switch (action) {
 	case CPU_ONLINE:
 	case CPU_ONLINE_FROZEN:
-		smp_call_function_single(hotcpu, ia64_mca_cmc_vector_adjust,
-					 NULL, 0);
+		ia64_mca_cmc_vector_adjust(NULL);
 		break;
 	}
 	return NOTIFY_OK;

[toc] | [next] | [standalone]


#1392501

From"Luck, Tony" <tony.luck@intel.com>
Date2016-05-02 22:30 +0200
Message-ID<rusDE-1Qt-7@gated-at.bofh.it>
In reply to#1392151
> Replace smp_call_function_single() with a direct call to
> ia64_mca_cmc_vector_adjust(). The function itselfs handles disable and
> enable interrupts, therefore the smp_call_function_single() calling
> convention is not preserved.

Applied. Thanks.

-Tony

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web