On Tue, 30 Jul 2019 15:50:25 +0300 "Tzvetomir Stoyanov (VMware)" <tz.stoyanov@xxxxxxxxx> wrote: > Before releasing libtraceevent as a standalone library, few > APIs must be changed: > - Event printing interface is redesigned, to be more simple > and user friendly. Now the tep_print_event() API can be used > to print various event information. All other event print APIs > are removed. > - Changed the user's plugin directory to > ~/.local/lib/traceevent/plugins/ to be XDG compliant. BTW, the other two patches look good. Let's send a v2 of all the patches with the comments in the first patch addressed. Thanks Ceco! -- Steve > > [ > v2 canges: > - the user's plugin directory changed from > ~/.config/traceevent/plugins/ to ~/.local/lib/traceevent/plugins/, > as suggested by Patrick McLean. > ] > > Tzvetomir Stoyanov (3): > tools/lib/traceevent, tools/perf: Changes in tep_print_event_* APIs > tools/lib/traceevent: Remove tep_register_trace_clock() > tools/lib/traceevent: Change user's plugin directory > > tools/lib/traceevent/Makefile | 6 +- > tools/lib/traceevent/event-parse-api.c | 40 --- > tools/lib/traceevent/event-parse-local.h | 6 - > tools/lib/traceevent/event-parse.c | 308 +++++++++++++---------- > tools/lib/traceevent/event-parse.h | 29 +-- > tools/lib/traceevent/event-plugin.c | 2 +- > tools/perf/builtin-kmem.c | 3 +- > tools/perf/util/sort.c | 3 +- > tools/perf/util/trace-event-parse.c | 2 +- > 9 files changed, 197 insertions(+), 202 deletions(-) >
|