powerpc/ftrace: Use patch_instruction instead of probe_kernel_write()
authorSteven Rostedt <srostedt@redhat.com>
Thu, 26 Apr 2012 08:31:19 +0000 (08:31 +0000)
committerBenjamin Herrenschmidt <benh@kernel.crashing.org>
Tue, 3 Jul 2012 04:14:39 +0000 (14:14 +1000)
The patch_instruction() interface is made to modify kernel text. It is
safer to use that then the probe_kernel_write() when modifying kernel
code.

Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
arch/powerpc/kernel/ftrace.c

index d56848afd2cfdbf2aeae3307af29245f26a865a6..6f33296a05701b95c8e65dfc29c502608c7a906d 100644 (file)
@@ -63,11 +63,9 @@ ftrace_modify_code(unsigned long ip, unsigned int old, unsigned int new)
                return -EINVAL;
 
        /* replace the text with the new text */
-       if (probe_kernel_write((void *)ip, &new, MCOUNT_INSN_SIZE))
+       if (patch_instruction((unsigned int *)ip, new))
                return -EPERM;
 
-       flush_icache_range(ip, ip + 8);
-
        return 0;
 }
 
@@ -212,12 +210,9 @@ __ftrace_make_nop(struct module *mod,
         */
        op = 0x48000008;        /* b +8 */
 
-       if (probe_kernel_write((void *)ip, &op, MCOUNT_INSN_SIZE))
+       if (patch_instruction((unsigned int *)ip, op))
                return -EPERM;
 
-
-       flush_icache_range(ip, ip + 8);
-
        return 0;
 }
 
@@ -286,11 +281,9 @@ __ftrace_make_nop(struct module *mod,
 
        op = PPC_INST_NOP;
 
-       if (probe_kernel_write((void *)ip, &op, MCOUNT_INSN_SIZE))
+       if (patch_instruction((unsigned int *)ip, op))
                return -EPERM;
 
-       flush_icache_range(ip, ip + 8);
-
        return 0;
 }
 #endif /* PPC64 */
@@ -426,11 +419,9 @@ __ftrace_make_call(struct dyn_ftrace *rec, unsigned long addr)
 
        pr_devel("write to %lx\n", rec->ip);
 
-       if (probe_kernel_write((void *)ip, &op, MCOUNT_INSN_SIZE))
+       if (patch_instruction((unsigned int *)ip, op))
                return -EPERM;
 
-       flush_icache_range(ip, ip + 8);
-
        return 0;
 }
 #endif /* CONFIG_PPC64 */