Commit c96f8385 authored by Torsten Duwe's avatar Torsten Duwe Committed by Michael Ellerman

powerpc/ftrace: Use generic ftrace_modify_all_code()

Convert powerpc's arch_ftrace_update_code() from its own version to use
the generic default functionality (without stop_machine -- our
instructions are properly aligned and the replacements atomic).

With this we gain error checking and the much-needed function_trace_op
handling.
Reviewed-by: default avatarBalbir Singh <bsingharora@gmail.com>
Reviewed-by: default avatarKamalesh Babulal <kamalesh@linux.vnet.ibm.com>
Signed-off-by: default avatarTorsten Duwe <duwe@suse.de>
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
parent 336a7b5d
...@@ -449,20 +449,13 @@ void ftrace_replace_code(int enable) ...@@ -449,20 +449,13 @@ void ftrace_replace_code(int enable)
} }
} }
/*
* Use the default ftrace_modify_all_code, but without
* stop_machine().
*/
void arch_ftrace_update_code(int command) void arch_ftrace_update_code(int command)
{ {
if (command & FTRACE_UPDATE_CALLS) ftrace_modify_all_code(command);
ftrace_replace_code(1);
else if (command & FTRACE_DISABLE_CALLS)
ftrace_replace_code(0);
if (command & FTRACE_UPDATE_TRACE_FUNC)
ftrace_update_ftrace_func(ftrace_trace_function);
if (command & FTRACE_START_FUNC_RET)
ftrace_enable_ftrace_graph_caller();
else if (command & FTRACE_STOP_FUNC_RET)
ftrace_disable_ftrace_graph_caller();
} }
int __init ftrace_dyn_arch_init(void) int __init ftrace_dyn_arch_init(void)
......
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