diff options
| author | Li Zefan <lizf@cn.fujitsu.com> | 2010-05-24 04:24:52 -0400 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2010-06-28 17:12:53 -0400 |
| commit | ffb9f99528574ab9a55d4c8fd22e9d3ca49efa86 (patch) | |
| tree | 3b5d2a5e0c33b214bd7c53d1733fe3771a68d4c7 /kernel/trace | |
| parent | c9d932cf8a1c608b676021aef0189376ba6ef151 (diff) | |
tracing: Remove redundant raw_init callbacks
raw_init callback is optional.
Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
LKML-Reference: <4BFA37D4.7070500@cn.fujitsu.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace')
| -rw-r--r-- | kernel/trace/trace_export.c | 8 | ||||
| -rw-r--r-- | kernel/trace/trace_kprobe.c | 10 |
2 files changed, 2 insertions, 16 deletions
diff --git a/kernel/trace/trace_export.c b/kernel/trace/trace_export.c index 8536e2a65969..4ba44deaac25 100644 --- a/kernel/trace/trace_export.c +++ b/kernel/trace/trace_export.c | |||
| @@ -125,12 +125,6 @@ ftrace_define_fields_##name(struct ftrace_event_call *event_call) \ | |||
| 125 | 125 | ||
| 126 | #include "trace_entries.h" | 126 | #include "trace_entries.h" |
| 127 | 127 | ||
| 128 | static int ftrace_raw_init_event(struct ftrace_event_call *call) | ||
| 129 | { | ||
| 130 | INIT_LIST_HEAD(&call->class->fields); | ||
| 131 | return 0; | ||
| 132 | } | ||
| 133 | |||
| 134 | #undef __entry | 128 | #undef __entry |
| 135 | #define __entry REC | 129 | #define __entry REC |
| 136 | 130 | ||
| @@ -158,7 +152,7 @@ static int ftrace_raw_init_event(struct ftrace_event_call *call) | |||
| 158 | struct ftrace_event_class event_class_ftrace_##call = { \ | 152 | struct ftrace_event_class event_class_ftrace_##call = { \ |
| 159 | .system = __stringify(TRACE_SYSTEM), \ | 153 | .system = __stringify(TRACE_SYSTEM), \ |
| 160 | .define_fields = ftrace_define_fields_##call, \ | 154 | .define_fields = ftrace_define_fields_##call, \ |
| 161 | .raw_init = ftrace_raw_init_event, \ | 155 | .fields = LIST_HEAD_INIT(event_class_ftrace_##call.fields),\ |
| 162 | }; \ | 156 | }; \ |
| 163 | \ | 157 | \ |
| 164 | struct ftrace_event_call __used \ | 158 | struct ftrace_event_call __used \ |
diff --git a/kernel/trace/trace_kprobe.c b/kernel/trace/trace_kprobe.c index f52b5f50299d..3b831d8e201e 100644 --- a/kernel/trace/trace_kprobe.c +++ b/kernel/trace/trace_kprobe.c | |||
| @@ -1214,11 +1214,6 @@ static void probe_event_disable(struct ftrace_event_call *call) | |||
| 1214 | } | 1214 | } |
| 1215 | } | 1215 | } |
| 1216 | 1216 | ||
| 1217 | static int probe_event_raw_init(struct ftrace_event_call *event_call) | ||
| 1218 | { | ||
| 1219 | return 0; | ||
| 1220 | } | ||
| 1221 | |||
| 1222 | #undef DEFINE_FIELD | 1217 | #undef DEFINE_FIELD |
| 1223 | #define DEFINE_FIELD(type, item, name, is_signed) \ | 1218 | #define DEFINE_FIELD(type, item, name, is_signed) \ |
| 1224 | do { \ | 1219 | do { \ |
| @@ -1486,15 +1481,12 @@ static int register_probe_event(struct trace_probe *tp) | |||
| 1486 | int ret; | 1481 | int ret; |
| 1487 | 1482 | ||
| 1488 | /* Initialize ftrace_event_call */ | 1483 | /* Initialize ftrace_event_call */ |
| 1484 | INIT_LIST_HEAD(&call->class->fields); | ||
| 1489 | if (probe_is_return(tp)) { | 1485 | if (probe_is_return(tp)) { |
| 1490 | INIT_LIST_HEAD(&call->class->fields); | ||
| 1491 | call->event.funcs = &kretprobe_funcs; | 1486 | call->event.funcs = &kretprobe_funcs; |
| 1492 | call->class->raw_init = probe_event_raw_init; | ||
| 1493 | call->class->define_fields = kretprobe_event_define_fields; | 1487 | call->class->define_fields = kretprobe_event_define_fields; |
| 1494 | } else { | 1488 | } else { |
| 1495 | INIT_LIST_HEAD(&call->class->fields); | ||
| 1496 | call->event.funcs = &kprobe_funcs; | 1489 | call->event.funcs = &kprobe_funcs; |
| 1497 | call->class->raw_init = probe_event_raw_init; | ||
| 1498 | call->class->define_fields = kprobe_event_define_fields; | 1490 | call->class->define_fields = kprobe_event_define_fields; |
| 1499 | } | 1491 | } |
| 1500 | if (set_print_fmt(tp) < 0) | 1492 | if (set_print_fmt(tp) < 0) |
