diff options
author | Jiri Slaby <jslaby@suse.cz> | 2014-02-24 13:59:58 -0500 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2014-03-07 10:06:13 -0500 |
commit | af64a7cb09db77344c596a0bf3d57d77257e8bf5 (patch) | |
tree | 9bfca2ea8870ed23eb49dca85303ec49752217a7 /kernel/trace/ftrace.c | |
parent | c867ccd8388d1c1a31bef9c54544b2ef32f0ebca (diff) |
ftrace: Pass retval through return in ftrace_dyn_arch_init()
No architecture uses the "data" parameter in ftrace_dyn_arch_init() in any
way, it just sets the value to 0. And this is used as a return value
in the caller -- ftrace_init, which just checks the retval against
zero.
Note there is also "return 0" in every ftrace_dyn_arch_init. So it is
enough to check the retval and remove all the indirect sets of data on
all archs.
Link: http://lkml.kernel.org/r/1393268401-24379-3-git-send-email-jslaby@suse.cz
Cc: linux-arch@vger.kernel.org
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace/ftrace.c')
-rw-r--r-- | kernel/trace/ftrace.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c index 76b6ed29d856..083c6d5fce25 100644 --- a/kernel/trace/ftrace.c +++ b/kernel/trace/ftrace.c | |||
@@ -4379,11 +4379,9 @@ void __init ftrace_init(void) | |||
4379 | addr = (unsigned long)ftrace_stub; | 4379 | addr = (unsigned long)ftrace_stub; |
4380 | 4380 | ||
4381 | local_irq_save(flags); | 4381 | local_irq_save(flags); |
4382 | ftrace_dyn_arch_init(&addr); | 4382 | ret = ftrace_dyn_arch_init(&addr); |
4383 | local_irq_restore(flags); | 4383 | local_irq_restore(flags); |
4384 | 4384 | if (ret) | |
4385 | /* ftrace_dyn_arch_init places the return code in addr */ | ||
4386 | if (addr) | ||
4387 | goto failed; | 4385 | goto failed; |
4388 | 4386 | ||
4389 | count = __stop_mcount_loc - __start_mcount_loc; | 4387 | count = __stop_mcount_loc - __start_mcount_loc; |