Commit 85fa41bb authored by Steven Rostedt (Red Hat)'s avatar Steven Rostedt (Red Hat) Committed by Luis Henriques

tools lib traceevent kbuffer: Remove extra update to data pointer in PADDING

commit c5e69192 upstream.

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: default avatarSteven Rostedt <rostedt@goodmis.org>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Link: http://lkml.kernel.org/r/20150324135923.461431960@goodmis.orgSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: default avatarLuis Henriques <luis.henriques@canonical.com>
parent 4d552c79
...@@ -372,7 +372,6 @@ translate_data(struct kbuffer *kbuf, void *data, void **rptr, ...@@ -372,7 +372,6 @@ translate_data(struct kbuffer *kbuf, void *data, void **rptr,
switch (type_len) { switch (type_len) {
case KBUFFER_TYPE_PADDING: case KBUFFER_TYPE_PADDING:
*length = read_4(kbuf, data); *length = read_4(kbuf, data);
data += *length;
break; break;
case KBUFFER_TYPE_TIME_EXTEND: case KBUFFER_TYPE_TIME_EXTEND:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment