On Tue, Jul 14, 2020 at 5:28 PM Steven Rostedt <rostedt@xxxxxxxxxxx> wrote: > > On Tue, 14 Jul 2020 13:30:20 +0300 > "Tzvetomir Stoyanov (VMware)" <tz.stoyanov@xxxxxxxxx> wrote: > > > Add description of tep_load_plugins_hook() traceevent API. > > > > Should add: > > Link: https://lore.kernel.org/r/CAM9d7cgLBWCrEHwz+Lhv5x5EXGcNWB0QQoeGh3OKh2JfR=dV9Q@xxxxxxxxxxxxxx > > (you get that part after '/r/' from the message ID in the email.) > Ok, I'll send the v2 with links and updated subjects to "trace-cmd libtraceevent:" > > Suggested-by: Namhyung Kim <namhyung@xxxxxxxxxx> > > Signed-off-by: Tzvetomir Stoyanov (VMware) <tz.stoyanov@xxxxxxxxx> > > --- > > lib/traceevent/event-plugin.c | 16 ++++++++++++++++ > > 1 file changed, 16 insertions(+) > > > > diff --git a/lib/traceevent/event-plugin.c b/lib/traceevent/event-plugin.c > > index 30c1526d..c11636ce 100644 > > --- a/lib/traceevent/event-plugin.c > > +++ b/lib/traceevent/event-plugin.c > > @@ -544,6 +544,22 @@ load_plugins_dir(struct tep_handle *tep, const char *suffix, > > closedir(dir); > > } > > > > +/** > > + * tep_load_plugins_hook - call a user specified callback to load a plugin > > + * @tep: handler to traceevent context > > + * @suffix: filter only plugin files with given suffix > > + * @load_plugin: user specified callback, called for each plugin file > > + * @data: custom context, passed to @load_plugin > > + * > > + * Searches for traceevent plugin files and calls @load_plugin for each > > + * The order of plugins search is: > > + * - Directories, specified in @tep->plugins_dir and priority TEP_PLUGIN_FIRST > > + * - Directory, specified at compile time with PLUGIN_TRACEEVENT_DIR > > + * - Directory, specified by environment variable TRACEEVENT_PLUGIN_DIR > > + * - In user's home: ~/.local/lib/traceevent/plugins/ > > + * - Directories, specified in @tep->plugins_dir and priority TEP_PLUGIN_LAST > > We should probably have a man page as well. I'm planning to add a man page when backporting these patches from trace-cmd to kernel repo, as there are no libtraceevent man pages in trace-cmd, they exist only in the kernel tree. > > Thanks Tzvetomir! > > -- Steve > > > + * > > + */ > > void tep_load_plugins_hook(struct tep_handle *tep, const char *suffix, > > void (*load_plugin)(struct tep_handle *tep, > > const char *path, > -- Tzvetomir (Ceco) Stoyanov VMware Open Source Technology Center
|