diff options
author | Steven Rostedt <srostedt@redhat.com> | 2010-04-22 11:46:44 -0400 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2010-05-14 14:20:30 -0400 |
commit | 0405ab80aa94afb13bf9ac4a6fc9f2923d4b9114 (patch) | |
tree | 43eedc09bb15ecfda26adb661d59badeacea0de6 /include/linux/ftrace_event.h | |
parent | 2e33af029556cb8bd22bf4f86f42d540249177ea (diff) |
tracing: Move raw_init from events to class
The raw_init function pointer in the event is used to initialize
various kinds of events. The type of initialization needed is usually
classed to the kind of event it is.
Two events with the same class will always have the same initialization
function, so it makes sense to move this to the class structure.
Perhaps even making a special system structure would work since
the initialization is the same for all events within a system.
But since there's no system structure (yet), this will just move it
to the class.
text data bss dec hex filename
4913961 1088356 861512 6863829 68bbd5 vmlinux.orig
4900375 1053380 861512 6815267 67fe23 vmlinux.fields
4900382 1048964 861512 6810858 67ecea vmlinux.init
The text grew very slightly, but this is a constant growth that happened
with the changing of the C files that call the init code.
The bigger savings is the data which will be saved the more events share
a class.
Acked-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Acked-by: Masami Hiramatsu <mhiramat@redhat.com>
Acked-by: Frederic Weisbecker <fweisbec@gmail.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'include/linux/ftrace_event.h')
-rw-r--r-- | include/linux/ftrace_event.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/ftrace_event.h b/include/linux/ftrace_event.h index 479c3c1876e6..393a8394df0e 100644 --- a/include/linux/ftrace_event.h +++ b/include/linux/ftrace_event.h | |||
@@ -133,6 +133,7 @@ struct ftrace_event_class { | |||
133 | int (*define_fields)(struct ftrace_event_call *); | 133 | int (*define_fields)(struct ftrace_event_call *); |
134 | struct list_head *(*get_fields)(struct ftrace_event_call *); | 134 | struct list_head *(*get_fields)(struct ftrace_event_call *); |
135 | struct list_head fields; | 135 | struct list_head fields; |
136 | int (*raw_init)(struct ftrace_event_call *); | ||
136 | }; | 137 | }; |
137 | 138 | ||
138 | struct ftrace_event_call { | 139 | struct ftrace_event_call { |
@@ -144,7 +145,6 @@ struct ftrace_event_call { | |||
144 | int enabled; | 145 | int enabled; |
145 | int id; | 146 | int id; |
146 | const char *print_fmt; | 147 | const char *print_fmt; |
147 | int (*raw_init)(struct ftrace_event_call *); | ||
148 | int filter_active; | 148 | int filter_active; |
149 | struct event_filter *filter; | 149 | struct event_filter *filter; |
150 | void *mod; | 150 | void *mod; |