tools lib traceevent, perf tools: Rename pevent plugin related APIs
authorTzvetomir Stoyanov (VMware) <tz.stoyanov@gmail.com>
Wed, 8 Aug 2018 18:02:48 +0000 (14:02 -0400)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Mon, 13 Aug 2018 18:21:39 +0000 (15:21 -0300)
In order to make libtraceevent into a proper library, variables, data
structures and functions require a unique prefix to prevent name space
conflicts. That prefix will be "tep_" and not "pevent_". This changes
the pevent plugin related API.

Signed-off-by: Tzvetomir Stoyanov (VMware) <tz.stoyanov@gmail.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Yordan Karadzhov (VMware) <y.karadz@gmail.com>
Cc: linux-trace-devel@vger.kernel.org
Link: http://lkml.kernel.org/r/20180808180700.005287044@goodmis.org
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
13 files changed:
tools/lib/traceevent/event-parse.c
tools/lib/traceevent/event-parse.h
tools/lib/traceevent/event-plugin.c
tools/lib/traceevent/plugin_cfg80211.c
tools/lib/traceevent/plugin_function.c
tools/lib/traceevent/plugin_hrtimer.c
tools/lib/traceevent/plugin_jbd2.c
tools/lib/traceevent/plugin_kmem.c
tools/lib/traceevent/plugin_kvm.c
tools/lib/traceevent/plugin_mac80211.c
tools/lib/traceevent/plugin_sched_switch.c
tools/lib/traceevent/plugin_scsi.c
tools/lib/traceevent/plugin_xen.c

index 90f30f9dde144a00bdae028a7adfcb58f7652d76..57ed9348d1aa511ccb78233f38d0304e7bcb30d6 100644 (file)
@@ -88,7 +88,7 @@ struct event_handler {
        int                             id;
        const char                      *sys_name;
        const char                      *event_name;
-       pevent_event_handler_func       func;
+       tep_event_handler_func          func;
        void                            *context;
 };
 
@@ -6633,7 +6633,7 @@ static struct event_format *pevent_search_event(struct tep_handle *pevent, int i
  */
 int pevent_register_event_handler(struct tep_handle *pevent, int id,
                                  const char *sys_name, const char *event_name,
-                                 pevent_event_handler_func func, void *context)
+                                 tep_event_handler_func func, void *context)
 {
        struct event_format *event;
        struct event_handler *handle;
@@ -6682,7 +6682,7 @@ int pevent_register_event_handler(struct tep_handle *pevent, int id,
 
 static int handle_matches(struct event_handler *handler, int id,
                          const char *sys_name, const char *event_name,
-                         pevent_event_handler_func func, void *context)
+                         tep_event_handler_func func, void *context)
 {
        if (id >= 0 && id != handler->id)
                return 0;
@@ -6717,7 +6717,7 @@ static int handle_matches(struct event_handler *handler, int id,
  */
 int pevent_unregister_event_handler(struct tep_handle *pevent, int id,
                                    const char *sys_name, const char *event_name,
-                                   pevent_event_handler_func func, void *context)
+                                   tep_event_handler_func func, void *context)
 {
        struct event_format *event;
        struct event_handler *handle;
index b1e4d37bedd82d83678bec994489a6923e00cbf8..b4b698c3833e34b2fca60e6c7ec4a4efc2e82eb1 100644 (file)
@@ -101,16 +101,16 @@ extern int trace_seq_do_printf(struct trace_seq *s);
 struct tep_handle;
 struct event_format;
 
-typedef int (*pevent_event_handler_func)(struct trace_seq *s,
-                                        struct tep_record *record,
-                                        struct event_format *event,
-                                        void *context);
+typedef int (*tep_event_handler_func)(struct trace_seq *s,
+                                     struct tep_record *record,
+                                     struct event_format *event,
+                                     void *context);
 
-typedef int (*pevent_plugin_load_func)(struct tep_handle *pevent);
-typedef int (*pevent_plugin_unload_func)(struct tep_handle *pevent);
+typedef int (*tep_plugin_load_func)(struct tep_handle *pevent);
+typedef int (*tep_plugin_unload_func)(struct tep_handle *pevent);
 
-struct pevent_plugin_option {
-       struct pevent_plugin_option     *next;
+struct tep_plugin_option {
+       struct tep_plugin_option        *next;
        void                            *handle;
        char                            *file;
        char                            *name;
@@ -124,20 +124,20 @@ struct pevent_plugin_option {
 /*
  * Plugin hooks that can be called:
  *
- * PEVENT_PLUGIN_LOADER:  (required)
+ * TEP_PLUGIN_LOADER:  (required)
  *   The function name to initialized the plugin.
  *
- *   int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
+ *   int TEP_PLUGIN_LOADER(struct tep_handle *pevent)
  *
- * PEVENT_PLUGIN_UNLOADER:  (optional)
+ * TEP_PLUGIN_UNLOADER:  (optional)
  *   The function called just before unloading
  *
- *   int PEVENT_PLUGIN_UNLOADER(struct tep_handle *pevent)
+ *   int TEP_PLUGIN_UNLOADER(struct tep_handle *pevent)
  *
- * PEVENT_PLUGIN_OPTIONS:  (optional)
+ * TEP_PLUGIN_OPTIONS:  (optional)
  *   Plugin options that can be set before loading
  *
- *   struct pevent_plugin_option PEVENT_PLUGIN_OPTIONS[] = {
+ *   struct tep_plugin_option TEP_PLUGIN_OPTIONS[] = {
  *     {
  *             .name = "option-name",
  *             .plugin_alias = "override-file-name", (optional)
@@ -158,19 +158,19 @@ struct pevent_plugin_option {
  *   .set will be processed. If .value is defined, then it is considered
  *   a string option and .set will be ignored.
  *
- * PEVENT_PLUGIN_ALIAS: (optional)
+ * TEP_PLUGIN_ALIAS: (optional)
  *   The name to use for finding options (uses filename if not defined)
  */
-#define PEVENT_PLUGIN_LOADER pevent_plugin_loader
-#define PEVENT_PLUGIN_UNLOADER pevent_plugin_unloader
-#define PEVENT_PLUGIN_OPTIONS pevent_plugin_options
-#define PEVENT_PLUGIN_ALIAS pevent_plugin_alias
+#define TEP_PLUGIN_LOADER tep_plugin_loader
+#define TEP_PLUGIN_UNLOADER tep_plugin_unloader
+#define TEP_PLUGIN_OPTIONS tep_plugin_options
+#define TEP_PLUGIN_ALIAS tep_plugin_alias
 #define _MAKE_STR(x)   #x
 #define MAKE_STR(x)    _MAKE_STR(x)
-#define PEVENT_PLUGIN_LOADER_NAME MAKE_STR(PEVENT_PLUGIN_LOADER)
-#define PEVENT_PLUGIN_UNLOADER_NAME MAKE_STR(PEVENT_PLUGIN_UNLOADER)
-#define PEVENT_PLUGIN_OPTIONS_NAME MAKE_STR(PEVENT_PLUGIN_OPTIONS)
-#define PEVENT_PLUGIN_ALIAS_NAME MAKE_STR(PEVENT_PLUGIN_ALIAS)
+#define TEP_PLUGIN_LOADER_NAME MAKE_STR(TEP_PLUGIN_LOADER)
+#define TEP_PLUGIN_UNLOADER_NAME MAKE_STR(TEP_PLUGIN_UNLOADER)
+#define TEP_PLUGIN_OPTIONS_NAME MAKE_STR(TEP_PLUGIN_OPTIONS)
+#define TEP_PLUGIN_ALIAS_NAME MAKE_STR(TEP_PLUGIN_ALIAS)
 
 enum format_flags {
        FIELD_IS_ARRAY          = 1,
@@ -327,7 +327,7 @@ struct event_format {
        struct format           format;
        struct print_fmt        print_fmt;
        char                    *system;
-       pevent_event_handler_func handler;
+       tep_event_handler_func  handler;
        void                    *context;
 };
 
@@ -441,8 +441,8 @@ void traceevent_unload_plugins(struct plugin_list *plugin_list,
 char **traceevent_plugin_list_options(void);
 void traceevent_plugin_free_options_list(char **list);
 int traceevent_plugin_add_options(const char *name,
-                                 struct pevent_plugin_option *options);
-void traceevent_plugin_remove_options(struct pevent_plugin_option *options);
+                                 struct tep_plugin_option *options);
+void traceevent_plugin_remove_options(struct tep_plugin_option *options);
 void traceevent_print_plugins(struct trace_seq *s,
                              const char *prefix, const char *suffix,
                              const struct plugin_list *list);
@@ -675,10 +675,10 @@ int pevent_print_func_field(struct trace_seq *s, const char *fmt,
 
 int pevent_register_event_handler(struct tep_handle *pevent, int id,
                                  const char *sys_name, const char *event_name,
-                                 pevent_event_handler_func func, void *context);
+                                 tep_event_handler_func func, void *context);
 int pevent_unregister_event_handler(struct tep_handle *pevent, int id,
                                    const char *sys_name, const char *event_name,
-                                   pevent_event_handler_func func, void *context);
+                                   tep_event_handler_func func, void *context);
 int pevent_register_print_function(struct tep_handle *pevent,
                                   pevent_func_handler func,
                                   enum pevent_func_arg_type ret_type,
index 62b011ae18c1cb77ce7b404d6023a9feb2d83ed1..8c91c8f0abd99cd9d46d605b0af0936630dea0af 100644 (file)
@@ -34,7 +34,7 @@
 
 static struct registered_plugin_options {
        struct registered_plugin_options        *next;
-       struct pevent_plugin_option             *options;
+       struct tep_plugin_option                *options;
 } *registered_options;
 
 static struct trace_plugin_options {
@@ -58,7 +58,7 @@ static void lower_case(char *str)
                *str = tolower(*str);
 }
 
-static int update_option_value(struct pevent_plugin_option *op, const char *val)
+static int update_option_value(struct tep_plugin_option *op, const char *val)
 {
        char *op_val;
 
@@ -111,7 +111,7 @@ static int update_option_value(struct pevent_plugin_option *op, const char *val)
 char **traceevent_plugin_list_options(void)
 {
        struct registered_plugin_options *reg;
-       struct pevent_plugin_option *op;
+       struct tep_plugin_option *op;
        char **list = NULL;
        char *name;
        int count = 0;
@@ -163,7 +163,7 @@ void traceevent_plugin_free_options_list(char **list)
 }
 
 static int
-update_option(const char *file, struct pevent_plugin_option *option)
+update_option(const char *file, struct tep_plugin_option *option)
 {
        struct trace_plugin_options *op;
        char *plugin;
@@ -222,7 +222,7 @@ update_option(const char *file, struct pevent_plugin_option *option)
  * Sets the options with the values that have been added by user.
  */
 int traceevent_plugin_add_options(const char *name,
-                                 struct pevent_plugin_option *options)
+                                 struct tep_plugin_option *options)
 {
        struct registered_plugin_options *reg;
 
@@ -244,7 +244,7 @@ int traceevent_plugin_add_options(const char *name,
  * traceevent_plugin_remove_options - remove plugin options that were registered
  * @options: Options to removed that were registered with traceevent_plugin_add_options
  */
-void traceevent_plugin_remove_options(struct pevent_plugin_option *options)
+void traceevent_plugin_remove_options(struct tep_plugin_option *options)
 {
        struct registered_plugin_options **last;
        struct registered_plugin_options *reg;
@@ -285,7 +285,7 @@ load_plugin(struct tep_handle *pevent, const char *path,
            const char *file, void *data)
 {
        struct plugin_list **plugin_list = data;
-       pevent_plugin_load_func func;
+       tep_plugin_load_func func;
        struct plugin_list *list;
        const char *alias;
        char *plugin;
@@ -305,14 +305,14 @@ load_plugin(struct tep_handle *pevent, const char *path,
                goto out_free;
        }
 
-       alias = dlsym(handle, PEVENT_PLUGIN_ALIAS_NAME);
+       alias = dlsym(handle, TEP_PLUGIN_ALIAS_NAME);
        if (!alias)
                alias = file;
 
-       func = dlsym(handle, PEVENT_PLUGIN_LOADER_NAME);
+       func = dlsym(handle, TEP_PLUGIN_LOADER_NAME);
        if (!func) {
                warning("could not find func '%s' in plugin '%s'\n%s\n",
-                       PEVENT_PLUGIN_LOADER_NAME, plugin, dlerror());
+                       TEP_PLUGIN_LOADER_NAME, plugin, dlerror());
                goto out_free;
        }
 
@@ -442,13 +442,13 @@ traceevent_load_plugins(struct tep_handle *pevent)
 void
 traceevent_unload_plugins(struct plugin_list *plugin_list, struct tep_handle *pevent)
 {
-       pevent_plugin_unload_func func;
+       tep_plugin_unload_func func;
        struct plugin_list *list;
 
        while (plugin_list) {
                list = plugin_list;
                plugin_list = list->next;
-               func = dlsym(list->handle, PEVENT_PLUGIN_UNLOADER_NAME);
+               func = dlsym(list->handle, TEP_PLUGIN_UNLOADER_NAME);
                if (func)
                        func(pevent);
                dlclose(list->handle);
index 959d94f1d7f5fccc10d1b662d6c0260374dc5808..c189bbc9fa18d0de63df32a4a16d02a7f9ddf89d 100644 (file)
@@ -25,7 +25,7 @@ process___le16_to_cpup(struct trace_seq *s, unsigned long long *args)
        return val ? (long long) le16toh(*val) : 0;
 }
 
-int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
+int TEP_PLUGIN_LOADER(struct tep_handle *pevent)
 {
        pevent_register_print_function(pevent,
                                       process___le16_to_cpup,
@@ -36,7 +36,7 @@ int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
        return 0;
 }
 
-void PEVENT_PLUGIN_UNLOADER(struct tep_handle *pevent)
+void TEP_PLUGIN_UNLOADER(struct tep_handle *pevent)
 {
        pevent_unregister_print_function(pevent, process___le16_to_cpup,
                                         "__le16_to_cpup");
index 86b5bb9bf22c2b395dca967bdcdd8f719ef09c6f..342be570ee8cfe0b699f4eb1b8038258ff1b55cc 100644 (file)
@@ -33,7 +33,7 @@ static int cpus = -1;
 
 #define STK_BLK 10
 
-struct pevent_plugin_option plugin_options[] =
+struct tep_plugin_option plugin_options[] =
 {
        {
                .name = "parent",
@@ -53,8 +53,8 @@ struct pevent_plugin_option plugin_options[] =
        }
 };
 
-static struct pevent_plugin_option *ftrace_parent = &plugin_options[0];
-static struct pevent_plugin_option *ftrace_indent = &plugin_options[1];
+static struct tep_plugin_option *ftrace_parent = &plugin_options[0];
+static struct tep_plugin_option *ftrace_indent = &plugin_options[1];
 
 static void add_child(struct func_stack *stack, const char *child, int pos)
 {
@@ -163,7 +163,7 @@ static int function_handler(struct trace_seq *s, struct tep_record *record,
        return 0;
 }
 
-int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
+int TEP_PLUGIN_LOADER(struct tep_handle *pevent)
 {
        pevent_register_event_handler(pevent, -1, "ftrace", "function",
                                      function_handler, NULL);
@@ -173,7 +173,7 @@ int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
        return 0;
 }
 
-void PEVENT_PLUGIN_UNLOADER(struct tep_handle *pevent)
+void TEP_PLUGIN_UNLOADER(struct tep_handle *pevent)
 {
        int i, x;
 
index 695e8e1b90e643836b1dc4c29c4abe6ce48290d3..5704d56c6f6fbfbb84213bc82f7a7fc872a716df 100644 (file)
@@ -66,7 +66,7 @@ static int timer_start_handler(struct trace_seq *s,
        return 0;
 }
 
-int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
+int TEP_PLUGIN_LOADER(struct tep_handle *pevent)
 {
        pevent_register_event_handler(pevent, -1,
                                      "timer", "hrtimer_expire_entry",
@@ -77,7 +77,7 @@ int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
        return 0;
 }
 
-void PEVENT_PLUGIN_UNLOADER(struct tep_handle *pevent)
+void TEP_PLUGIN_UNLOADER(struct tep_handle *pevent)
 {
        pevent_unregister_event_handler(pevent, -1,
                                        "timer", "hrtimer_expire_entry",
index 3909db7043c9000110d50a0cd399d2e250e783de..9b88aebb48a9290981880510f1647e9924aaef82 100644 (file)
@@ -47,7 +47,7 @@ process_jiffies_to_msecs(struct trace_seq *s, unsigned long long *args)
        return jiffies;
 }
 
-int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
+int TEP_PLUGIN_LOADER(struct tep_handle *pevent)
 {
        pevent_register_print_function(pevent,
                                       process_jbd2_dev_to_name,
@@ -65,7 +65,7 @@ int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
        return 0;
 }
 
-void PEVENT_PLUGIN_UNLOADER(struct tep_handle *pevent)
+void TEP_PLUGIN_UNLOADER(struct tep_handle *pevent)
 {
        pevent_unregister_print_function(pevent, process_jbd2_dev_to_name,
                                         "jbd2_dev_to_name");
index 46e8e959539eb3846a1f932c0ed3995454516e24..0a56562d0645608336588dff0784f42dec992b0c 100644 (file)
@@ -48,7 +48,7 @@ static int call_site_handler(struct trace_seq *s, struct tep_record *record,
        return 1;
 }
 
-int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
+int TEP_PLUGIN_LOADER(struct tep_handle *pevent)
 {
        pevent_register_event_handler(pevent, -1, "kmem", "kfree",
                                      call_site_handler, NULL);
@@ -71,7 +71,7 @@ int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
        return 0;
 }
 
-void PEVENT_PLUGIN_UNLOADER(struct tep_handle *pevent)
+void TEP_PLUGIN_UNLOADER(struct tep_handle *pevent)
 {
        pevent_unregister_event_handler(pevent, -1, "kmem", "kfree",
                                        call_site_handler, NULL);
index 63420e3a340cc8533df9e831d914ae88d4ee5a8a..ab5439e0ad87b7b953e880b24e99ee4c5379f673 100644 (file)
@@ -444,7 +444,7 @@ process_is_writable_pte(struct trace_seq *s, unsigned long long *args)
        return pte & PT_WRITABLE_MASK;
 }
 
-int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
+int TEP_PLUGIN_LOADER(struct tep_handle *pevent)
 {
        init_disassembler();
 
@@ -486,7 +486,7 @@ int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
        return 0;
 }
 
-void PEVENT_PLUGIN_UNLOADER(struct tep_handle *pevent)
+void TEP_PLUGIN_UNLOADER(struct tep_handle *pevent)
 {
        pevent_unregister_event_handler(pevent, -1, "kvm", "kvm_exit",
                                        kvm_exit_handler, NULL);
index 83d002753251280e022adad9492cf9cbcf0dbfd2..61eea3543cf16b85e825bae54927d2f9c5e5e4e5 100644 (file)
@@ -86,7 +86,7 @@ static int drv_bss_info_changed(struct trace_seq *s,
        return 0;
 }
 
-int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
+int TEP_PLUGIN_LOADER(struct tep_handle *pevent)
 {
        pevent_register_event_handler(pevent, -1, "mac80211",
                                      "drv_bss_info_changed",
@@ -94,7 +94,7 @@ int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
        return 0;
 }
 
-void PEVENT_PLUGIN_UNLOADER(struct tep_handle *pevent)
+void TEP_PLUGIN_UNLOADER(struct tep_handle *pevent)
 {
        pevent_unregister_event_handler(pevent, -1, "mac80211",
                                        "drv_bss_info_changed",
index b72a52d2fd1d2a49a3fdfb5888a00aa425a09d1c..24dc2bca8e6787b96aca36baa58b2e71c5e5d003 100644 (file)
@@ -134,7 +134,7 @@ static int sched_switch_handler(struct trace_seq *s,
        return 0;
 }
 
-int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
+int TEP_PLUGIN_LOADER(struct tep_handle *pevent)
 {
        pevent_register_event_handler(pevent, -1, "sched", "sched_switch",
                                      sched_switch_handler, NULL);
@@ -147,7 +147,7 @@ int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
        return 0;
 }
 
-void PEVENT_PLUGIN_UNLOADER(struct tep_handle *pevent)
+void TEP_PLUGIN_UNLOADER(struct tep_handle *pevent)
 {
        pevent_unregister_event_handler(pevent, -1, "sched", "sched_switch",
                                        sched_switch_handler, NULL);
index e430a49e4995adb53af18c69bcea06a4e14a88e5..bb9aea6fb000765e62841b5fdb5f085a65223cb7 100644 (file)
@@ -413,7 +413,7 @@ unsigned long long process_scsi_trace_parse_cdb(struct trace_seq *s,
        return 0;
 }
 
-int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
+int TEP_PLUGIN_LOADER(struct tep_handle *pevent)
 {
        pevent_register_print_function(pevent,
                                       process_scsi_trace_parse_cdb,
@@ -426,7 +426,7 @@ int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
        return 0;
 }
 
-void PEVENT_PLUGIN_UNLOADER(struct tep_handle *pevent)
+void TEP_PLUGIN_UNLOADER(struct tep_handle *pevent)
 {
        pevent_unregister_print_function(pevent, process_scsi_trace_parse_cdb,
                                         "scsi_trace_parse_cdb");
index 11dbddde2e6ac179acb03f4c5ad2f7f845f3f262..1973dd8abbfdd1a0660e43e608cf0b691d0983e4 100644 (file)
@@ -119,7 +119,7 @@ unsigned long long process_xen_hypercall_name(struct trace_seq *s,
        return 0;
 }
 
-int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
+int TEP_PLUGIN_LOADER(struct tep_handle *pevent)
 {
        pevent_register_print_function(pevent,
                                       process_xen_hypercall_name,
@@ -130,7 +130,7 @@ int PEVENT_PLUGIN_LOADER(struct tep_handle *pevent)
        return 0;
 }
 
-void PEVENT_PLUGIN_UNLOADER(struct tep_handle *pevent)
+void TEP_PLUGIN_UNLOADER(struct tep_handle *pevent)
 {
        pevent_unregister_print_function(pevent, process_xen_hypercall_name,
                                         "xen_hypercall_name");