![Thomas Gleixner](/assets/img/avatar_default.png)
The conversion of the hotplug locking to a percpu rwsem unearthed lock ordering issues all over the place. The jump_label code has two issues: 1) Nested get_online_cpus() invocations 2) Ordering problems vs. the cpus rwsem and the jump_label_mutex To cure these, the following lock order has been established; cpus_rwsem -> jump_label_lock -> text_mutex Even if not all architectures need protection against CPU hotplug, taking cpus_rwsem before jump_label_lock is now mandatory in code pathes which actually modify code and therefor need text_mutex protection. Move the get_online_cpus() invocations into the core jump label code and establish the proper lock order where required. Signed-off-by: Thomas Gleixner <tglx@linutronix.de> Acked-by: Ingo Molnar <mingo@kernel.org> Acked-by: "David S. Miller" <davem@davemloft.net> Cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Cc: Chris Metcalf <cmetcalf@mellanox.com> Cc: Peter Zijlstra <peterz@infradead.org> Cc: Sebastian Siewior <bigeasy@linutronix.de> Cc: Steven Rostedt <rostedt@goodmis.org> Cc: Jason Baron <jbaron@akamai.com> Cc: Ralf Baechle <ralf@linux-mips.org> Link: http://lkml.kernel.org/r/20170524081549.025830817@linutronix.de
63 lines
1.6 KiB
C
63 lines
1.6 KiB
C
/*
|
|
* Copyright 2015 Tilera Corporation. All Rights Reserved.
|
|
*
|
|
* This program is free software; you can redistribute it and/or
|
|
* modify it under the terms of the GNU General Public License
|
|
* as published by the Free Software Foundation, version 2.
|
|
*
|
|
* This program is distributed in the hope that it will be useful, but
|
|
* WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY OR FITNESS FOR A PARTICULAR PURPOSE, GOOD TITLE or
|
|
* NON INFRINGEMENT. See the GNU General Public License for
|
|
* more details.
|
|
*
|
|
* jump label TILE-Gx support
|
|
*/
|
|
|
|
#include <linux/jump_label.h>
|
|
#include <linux/memory.h>
|
|
#include <linux/module.h>
|
|
#include <linux/mutex.h>
|
|
#include <linux/cpu.h>
|
|
|
|
#include <asm/cacheflush.h>
|
|
#include <asm/insn.h>
|
|
|
|
#ifdef HAVE_JUMP_LABEL
|
|
|
|
static void __jump_label_transform(struct jump_entry *e,
|
|
enum jump_label_type type)
|
|
{
|
|
tilegx_bundle_bits opcode;
|
|
/* Operate on writable kernel text mapping. */
|
|
unsigned long pc_wr = ktext_writable_addr(e->code);
|
|
|
|
if (type == JUMP_LABEL_JMP)
|
|
opcode = tilegx_gen_branch(e->code, e->target, false);
|
|
else
|
|
opcode = NOP();
|
|
|
|
*(tilegx_bundle_bits *)pc_wr = opcode;
|
|
/* Make sure that above mem writes were issued towards the memory. */
|
|
smp_wmb();
|
|
}
|
|
|
|
void arch_jump_label_transform(struct jump_entry *e,
|
|
enum jump_label_type type)
|
|
{
|
|
mutex_lock(&text_mutex);
|
|
|
|
__jump_label_transform(e, type);
|
|
flush_icache_range(e->code, e->code + sizeof(tilegx_bundle_bits));
|
|
|
|
mutex_unlock(&text_mutex);
|
|
}
|
|
|
|
__init_or_module void arch_jump_label_transform_static(struct jump_entry *e,
|
|
enum jump_label_type type)
|
|
{
|
|
__jump_label_transform(e, type);
|
|
}
|
|
|
|
#endif /* HAVE_JUMP_LABEL */
|