Commit ce57c15c authored by Steven Rostedt's avatar Steven Rostedt Committed by Greg Kroah-Hartman

tools lib traceevent: Fix output of %llu for 64 bit values read on 32 bit machines

commit 32abc2ed upstream.

When a long value is read on 32 bit machines for 64 bit output, the
parsing needs to change "%lu" into "%llu", as the value is read
natively.

Unfortunately, if "%llu" is already there, the code will add another "l"
to it and fail to parse it properly.
Signed-off-by: default avatarSteven Rostedt <rostedt@goodmis.org>
Acked-by: default avatarNamhyung Kim <namhyung@kernel.org>
Link: http://lkml.kernel.org/r/20151116172516.4b79b109@gandalf.local.homeSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 8a558d9a
...@@ -4249,13 +4249,12 @@ static void pretty_print(struct trace_seq *s, void *data, int size, struct event ...@@ -4249,13 +4249,12 @@ static void pretty_print(struct trace_seq *s, void *data, int size, struct event
sizeof(long) != 8) { sizeof(long) != 8) {
char *p; char *p;
ls = 2;
/* make %l into %ll */ /* make %l into %ll */
p = strchr(format, 'l'); if (ls == 1 && (p = strchr(format, 'l')))
if (p)
memmove(p+1, p, strlen(p)+1); memmove(p+1, p, strlen(p)+1);
else if (strcmp(format, "%p") == 0) else if (strcmp(format, "%p") == 0)
strcpy(format, "0x%llx"); strcpy(format, "0x%llx");
ls = 2;
} }
switch (ls) { switch (ls) {
case -2: case -2:
......
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