Skip to content

tool lib traceevent: Fix some parsing errors

Steven Rostedt linux-mm linux-trace-devel lkml 1 version Applied

Status from this message: Thanks, applied to perf/urgent, for v5.8.

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest3 patches 3 repliesApplied

Patches in v1 Applied

#Subject and filesSizeRepliesApplied
1/3tools lib traceevent: Add append() function helper for appending stringstools/lib/traceevent/event-parse.c+40 -580 replies
2/3tools lib traceevent: Handle __attribute__((user)) in field namestools/lib/traceevent/event-parse.c+38 -10 replies
3/3tools lib traceevent: Add handler for __builtin_expect()tools/lib/traceevent/event-parse.c+35 -00 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/3] tool lib traceevent: Fix some parsing errors

Arnaldo,

Here's a few patches for lib traceevent.

The first one adds an "append()" helper function for appending to strings, which will also simplify the next patch.

The second patch handles "__attribute__((user))" in the field of a trace event. This is needed after the stack leak code added this to parameters of system call events.

The last patch adds handling of __builtin_expect(), as someone wanted to move IS_ERR_VALUE() from the fast path into the print fmt section, which breaks the parsing. This should also be useful for other macros that may evaluate down to a __builtin_expect().

Steven Rostedt (VMware) (3):
tools lib traceevent: Add append() function helper for appending strings tools lib traceevent: Handle __attribute__((user)) in field names tools lib traceevent: Add handler for __builtin_expect()

----

 tools/lib/traceevent/event-parse.c | 168 ++++++++++++++++++++++++-------------
 1 file changed, 111 insertions(+), 57 deletions(-)