Commit 4394e282 authored by Rabin Vincent's avatar Rabin Vincent Committed by Russell King

ARM: 7330/1: ftrace: use canonical Thumb-2 wide instruction format

As commit 592201a9 (ARM: Thumb-2: Support Thumb-2 in undefined
instruction handler) says:

    32-bit Thumb instructions are specified in the form:
        ((first_half << 16 ) | second_half)
    which matches the layout used by the ARM ARM.

Convert the ftrace code to use the same format to avoid the usage of
different formats in kernel code.
Acked-by: default avatarDave Martin <dave.martin@linaro.org>
Signed-off-by: default avatarRabin Vincent <rabin@rab.in>
Signed-off-by: default avatarRussell King <rmk+kernel@arm.linux.org.uk>
parent dc283d70
...@@ -16,10 +16,11 @@ ...@@ -16,10 +16,11 @@
#include <linux/uaccess.h> #include <linux/uaccess.h>
#include <asm/cacheflush.h> #include <asm/cacheflush.h>
#include <asm/opcodes.h>
#include <asm/ftrace.h> #include <asm/ftrace.h>
#ifdef CONFIG_THUMB2_KERNEL #ifdef CONFIG_THUMB2_KERNEL
#define NOP 0xeb04f85d /* pop.w {lr} */ #define NOP 0xf85deb04 /* pop.w {lr} */
#else #else
#define NOP 0xe8bd4000 /* pop {lr} */ #define NOP 0xe8bd4000 /* pop {lr} */
#endif #endif
...@@ -88,7 +89,7 @@ static unsigned long ftrace_gen_branch(unsigned long pc, unsigned long addr, ...@@ -88,7 +89,7 @@ static unsigned long ftrace_gen_branch(unsigned long pc, unsigned long addr,
if (link) if (link)
second |= 1 << 14; second |= 1 << 14;
return (second << 16) | first; return __opcode_thumb32_compose(first, second);
} }
#else #else
static unsigned long ftrace_gen_branch(unsigned long pc, unsigned long addr, static unsigned long ftrace_gen_branch(unsigned long pc, unsigned long addr,
...@@ -125,6 +126,14 @@ static int ftrace_modify_code(unsigned long pc, unsigned long old, ...@@ -125,6 +126,14 @@ static int ftrace_modify_code(unsigned long pc, unsigned long old,
{ {
unsigned long replaced; unsigned long replaced;
if (IS_ENABLED(CONFIG_THUMB2_KERNEL)) {
old = __opcode_to_mem_thumb32(old);
new = __opcode_to_mem_thumb32(new);
} else {
old = __opcode_to_mem_arm(old);
new = __opcode_to_mem_arm(new);
}
if (validate) { if (validate) {
if (probe_kernel_read(&replaced, (void *)pc, MCOUNT_INSN_SIZE)) if (probe_kernel_read(&replaced, (void *)pc, MCOUNT_INSN_SIZE))
return -EFAULT; return -EFAULT;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment