Commit 8d0c2224 authored by Namhyung Kim's avatar Namhyung Kim Committed by Arnaldo Carvalho de Melo

tools lib traceevent: Make plugin unload function receive pevent

The PEVENT_PLUGIN_UNLOADER function might need some cleanup using pevent
like unregister some handlers.  So pass pevent as argument.
Signed-off-by: default avatarNamhyung Kim <namhyung@kernel.org>
Reviewed-by: default avatarJiri Olsa <jolsa@redhat.com>
Acked-by: default avatarSteven Rostedt <rostedt@goodmis.org>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung.kim@lge.com>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Steven Rostedt <rostedt@goodmis.org>
Link: http://lkml.kernel.org/r/1389750340-15965-6-git-send-email-namhyung@kernel.orgSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent e825e756
...@@ -105,7 +105,7 @@ typedef int (*pevent_event_handler_func)(struct trace_seq *s, ...@@ -105,7 +105,7 @@ typedef int (*pevent_event_handler_func)(struct trace_seq *s,
void *context); void *context);
typedef int (*pevent_plugin_load_func)(struct pevent *pevent); typedef int (*pevent_plugin_load_func)(struct pevent *pevent);
typedef int (*pevent_plugin_unload_func)(void); typedef int (*pevent_plugin_unload_func)(struct pevent *pevent);
struct plugin_option { struct plugin_option {
struct plugin_option *next; struct plugin_option *next;
...@@ -130,7 +130,7 @@ struct plugin_option { ...@@ -130,7 +130,7 @@ struct plugin_option {
* PEVENT_PLUGIN_UNLOADER: (optional) * PEVENT_PLUGIN_UNLOADER: (optional)
* The function called just before unloading * The function called just before unloading
* *
* int PEVENT_PLUGIN_UNLOADER(void) * int PEVENT_PLUGIN_UNLOADER(struct pevent *pevent)
* *
* PEVENT_PLUGIN_OPTIONS: (optional) * PEVENT_PLUGIN_OPTIONS: (optional)
* Plugin options that can be set before loading * Plugin options that can be set before loading
...@@ -411,7 +411,8 @@ enum pevent_errno { ...@@ -411,7 +411,8 @@ enum pevent_errno {
struct plugin_list; struct plugin_list;
struct plugin_list *traceevent_load_plugins(struct pevent *pevent); struct plugin_list *traceevent_load_plugins(struct pevent *pevent);
void traceevent_unload_plugins(struct plugin_list *plugin_list); void traceevent_unload_plugins(struct plugin_list *plugin_list,
struct pevent *pevent);
struct cmdline; struct cmdline;
struct cmdline_list; struct cmdline_list;
......
...@@ -197,7 +197,7 @@ traceevent_load_plugins(struct pevent *pevent) ...@@ -197,7 +197,7 @@ traceevent_load_plugins(struct pevent *pevent)
} }
void void
traceevent_unload_plugins(struct plugin_list *plugin_list) traceevent_unload_plugins(struct plugin_list *plugin_list, struct pevent *pevent)
{ {
pevent_plugin_unload_func func; pevent_plugin_unload_func func;
struct plugin_list *list; struct plugin_list *list;
...@@ -207,7 +207,7 @@ traceevent_unload_plugins(struct plugin_list *plugin_list) ...@@ -207,7 +207,7 @@ traceevent_unload_plugins(struct plugin_list *plugin_list)
plugin_list = list->next; plugin_list = list->next;
func = dlsym(list->handle, PEVENT_PLUGIN_UNLOADER_NAME); func = dlsym(list->handle, PEVENT_PLUGIN_UNLOADER_NAME);
if (func) if (func)
func(); func(pevent);
dlclose(list->handle); dlclose(list->handle);
free(list->name); free(list->name);
free(list); free(list);
......
...@@ -144,7 +144,7 @@ int PEVENT_PLUGIN_LOADER(struct pevent *pevent) ...@@ -144,7 +144,7 @@ int PEVENT_PLUGIN_LOADER(struct pevent *pevent)
return 0; return 0;
} }
void PEVENT_PLUGIN_UNLOADER(void) void PEVENT_PLUGIN_UNLOADER(struct pevent *pevent)
{ {
int i, x; int i, x;
......
...@@ -34,8 +34,8 @@ int trace_event__init(struct trace_event *t) ...@@ -34,8 +34,8 @@ int trace_event__init(struct trace_event *t)
void trace_event__cleanup(struct trace_event *t) void trace_event__cleanup(struct trace_event *t)
{ {
traceevent_unload_plugins(t->plugin_list, t->pevent);
pevent_free(t->pevent); pevent_free(t->pevent);
traceevent_unload_plugins(t->plugin_list);
} }
static struct event_format* static struct event_format*
......
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