diff options
| author | Steven Rostedt (Red Hat) <[email protected]> | 2015-03-24 09:57:55 -0400 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <[email protected]> | 2015-03-24 12:22:04 -0300 |
| commit | c5e691928bf166ac03430e957038b60adba3cf6c (patch) | |
| tree | adb7dc882e539a3027be6aeb8e7d7fe5834af17a | |
| parent | 5dbcfd930e430b33fb4dfded9c1af0cf432772b9 (diff) | |
tools lib traceevent kbuffer: Remove extra update to data pointer in PADDING
When a event PADDING is hit (a deleted event that is still in the ring
buffer), translate_data() sets the length of the padding and also updates
the data pointer which is passed back to the caller.
This is unneeded because the caller also updates the data pointer with
the passed back length. translate_data() should not update the pointer,
only set the length.
Signed-off-by: Steven Rostedt <[email protected]>
Cc: Andrew Morton <[email protected]>
Cc: Jiri Olsa <[email protected]>
Cc: Namhyung Kim <[email protected]>
Cc: [email protected] # 3.12+
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
| -rw-r--r-- | tools/lib/traceevent/kbuffer-parse.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/tools/lib/traceevent/kbuffer-parse.c b/tools/lib/traceevent/kbuffer-parse.c index dcc665228c71..deb3569ab004 100644 --- a/tools/lib/traceevent/kbuffer-parse.c +++ b/tools/lib/traceevent/kbuffer-parse.c @@ -372,7 +372,6 @@ translate_data(struct kbuffer *kbuf, void *data, void **rptr, switch (type_len) { case KBUFFER_TYPE_PADDING: *length = read_4(kbuf, data); - data += *length; break; case KBUFFER_TYPE_TIME_EXTEND: |