diff options
author | Anna-Maria Gleixner <anna-maria@linutronix.de> | 2016-04-06 10:42:28 +0200 |
---|---|---|
committer | Will Deacon <will.deacon@arm.com> | 2016-04-14 18:13:03 +0100 |
commit | 499c81507f599d4e753cf6401f921e68efab235a (patch) | |
tree | 6c40972e707ec9efc9c25f38b58f176e57932c57 | |
parent | 97740051dd31d200a0efaa84544fe5e4713aac40 (diff) | |
download | op-kernel-dev-499c81507f599d4e753cf6401f921e68efab235a.zip op-kernel-dev-499c81507f599d4e753cf6401f921e68efab235a.tar.gz |
arm64/debug: Remove superfluous SMP function call
Since commit 1cf4f629d9d2 ("cpu/hotplug: Move online calls to
hotplugged cpu") 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
clear_os_lock(). The function writes the OSLAR register to clear OS
locking. This does not require to be called with interrupts disabled,
therefore the smp_call_function_single() calling convention is not
preserved.
Cc: Catalin Marinas <catalin.marinas@arm.com>
Cc: Will Deacon <will.deacon@arm.com>
Cc: linux-arm-kernel@lists.infradead.org
Signed-off-by: Anna-Maria Gleixner <anna-maria@linutronix.de>
Signed-off-by: Will Deacon <will.deacon@arm.com>
-rw-r--r-- | arch/arm64/kernel/debug-monitors.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/arch/arm64/kernel/debug-monitors.c b/arch/arm64/kernel/debug-monitors.c index c45f296..4fbf3c5 100644 --- a/arch/arm64/kernel/debug-monitors.c +++ b/arch/arm64/kernel/debug-monitors.c @@ -135,9 +135,8 @@ static void clear_os_lock(void *unused) static int os_lock_notify(struct notifier_block *self, unsigned long action, void *data) { - int cpu = (unsigned long)data; if ((action & ~CPU_TASKS_FROZEN) == CPU_ONLINE) - smp_call_function_single(cpu, clear_os_lock, NULL, 1); + clear_os_lock(NULL); return NOTIFY_OK; } |