diff options
| author | David Ahern | 2013-12-05 03:41:41 +0100 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo | 2013-12-05 14:09:58 +0100 |
| commit | 3160565f0e005d2ec736ae25cf0a79988c0cbe71 (patch) | |
| tree | f2164ae21d3ad54e605ca91f96e111474e5123fd /tools/perf/builtin-trace.c | |
| parent | perf trace: Add support for syscalls vs raw_syscalls (diff) | |
| download | kernel-qcow2-linux-3160565f0e005d2ec736ae25cf0a79988c0cbe71.tar.gz kernel-qcow2-linux-3160565f0e005d2ec736ae25cf0a79988c0cbe71.tar.xz kernel-qcow2-linux-3160565f0e005d2ec736ae25cf0a79988c0cbe71.zip | |
perf trace: Fix summary percentage when processing files
Getting a divide by 0 when events are processed from a file:
perf trace -i perf.data -s
...
dnsmasq (1684), 10 events, inf%, 0.000 msec
The problem is that the event count is not incremented as events are
processed. With this patch:
perf trace -i perf.data -s
...
dnsmasq (1684), 10 events, 8.9%, 0.000 msec
Signed-off-by: David Ahern <dsahern@gmail.com>
Link: http://lkml.kernel.org/r/1386211302-31303-4-git-send-email-dsahern@gmail.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/builtin-trace.c')
| -rw-r--r-- | tools/perf/builtin-trace.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c index a7aa771a98e6..56bbca5bc2dc 100644 --- a/tools/perf/builtin-trace.c +++ b/tools/perf/builtin-trace.c @@ -1770,8 +1770,10 @@ static int trace__process_sample(struct perf_tool *tool, if (!trace->full_time && trace->base_time == 0) trace->base_time = sample->time; - if (handler) + if (handler) { + ++trace->nr_events; handler(trace, evsel, sample); + } return err; } |