diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2013-09-20 11:19:41 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2013-10-09 16:41:02 -0400 |
commit | 4aa5823225c70efba758750029504807cd30b925 (patch) | |
tree | 83cf014a17838a152c19c9654cb3491bf45d24d9 /tools/perf | |
parent | 453350dd0f0245b91b1e43310f5966fb1c51e7bd (diff) |
perf trace: Don't supress zeroed args when there is an strarray entry for it
Case in hand:
9.682 ( 0.001 ms): Xorg/13079 setitimer(which: REAL, value: 0x7fffede42470) = 0
ITIMER_REAL is zero.
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/n/tip-6hnoqsjh99t4hxi3xu2nlwep@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf')
-rw-r--r-- | tools/perf/builtin-trace.c | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c index 8855207deae8..27b0ec88cb23 100644 --- a/tools/perf/builtin-trace.c +++ b/tools/perf/builtin-trace.c | |||
@@ -940,8 +940,15 @@ static size_t syscall__scnprintf_args(struct syscall *sc, char *bf, size_t size, | |||
940 | field = field->next, ++arg.idx, bit <<= 1) { | 940 | field = field->next, ++arg.idx, bit <<= 1) { |
941 | if (arg.mask & bit) | 941 | if (arg.mask & bit) |
942 | continue; | 942 | continue; |
943 | 943 | /* | |
944 | if (args[arg.idx] == 0) | 944 | * Suppress this argument if its value is zero and |
945 | * and we don't have a string associated in an | ||
946 | * strarray for it. | ||
947 | */ | ||
948 | if (args[arg.idx] == 0 && | ||
949 | !(sc->arg_scnprintf && | ||
950 | sc->arg_scnprintf[arg.idx] == SCA_STRARRAY && | ||
951 | sc->arg_parm[arg.idx])) | ||
945 | continue; | 952 | continue; |
946 | 953 | ||
947 | printed += scnprintf(bf + printed, size - printed, | 954 | printed += scnprintf(bf + printed, size - printed, |