diff options
author | 2013-03-12 15:07:59 -0400 | |
---|---|---|
committer | 2013-03-15 00:36:02 -0400 | |
commit | e67efb93f0e9130174293ffaa5975f87b301b531 (patch) | |
tree | 344cadb55d5d50e0ae0473a8404a6893d8a3183b /include/linux/ftrace.h | |
parent | tracing: Fix comments for ftrace_event_file/call flags (diff) | |
download | linux-e67efb93f0e9130174293ffaa5975f87b301b531.tar.gz linux-e67efb93f0e9130174293ffaa5975f87b301b531.tar.bz2 linux-e67efb93f0e9130174293ffaa5975f87b301b531.zip |
ftrace: Clean up function probe methods
When a function probe is created, each function that the probe is
attached to, a "callback" method is called. On release of the probe,
each function entry calls the "free" method.
First, "callback" is a confusing name and does not really match what
it does. Callback sounds like it will be called when the probe
triggers. But that's not the case. This is really an "init" function,
so lets rename it as such.
Secondly, both "init" and "free" do not pass enough information back
to the handlers. Pass back the ops, ip and data for each time the
method is called. We have the information, might as well use it.
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'include/linux/ftrace.h')
-rw-r--r-- | include/linux/ftrace.h | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/include/linux/ftrace.h b/include/linux/ftrace.h index e5ca8ef50e9b..832422d706f4 100644 --- a/include/linux/ftrace.h +++ b/include/linux/ftrace.h @@ -259,8 +259,10 @@ struct ftrace_probe_ops { void (*func)(unsigned long ip, unsigned long parent_ip, void **data); - int (*callback)(unsigned long ip, void **data); - void (*free)(void **data); + int (*init)(struct ftrace_probe_ops *ops, + unsigned long ip, void **data); + void (*free)(struct ftrace_probe_ops *ops, + unsigned long ip, void **data); int (*print)(struct seq_file *m, unsigned long ip, struct ftrace_probe_ops *ops, |