perf tools: Account entry stats when it's added to the output tree
authorNamhyung Kim <namhyung@kernel.org>
Tue, 22 Apr 2014 02:44:21 +0000 (11:44 +0900)
committerJiri Olsa <jolsa@kernel.org>
Thu, 24 Apr 2014 14:32:15 +0000 (16:32 +0200)
commit820bc81f4cdaac09a8f25040d3a20d86f3da292b
treeafb1ff42a151339c55782af3b626078a81b21fd1
parent87e90f43285f4096e9ba5fc18b05c2e04caf3fab
perf tools: Account entry stats when it's added to the output tree

Currently, accounting each sample is done in multiple places - once
when adding them to the input tree, other when adding them to the
output tree.  It's not only confusing but also can cause a subtle
problem since concurrent processing like in perf top might see the
updated stats before adding entries into the output tree - like seeing
more (blank) lines at the end and/or slight inaccurate percentage.

To fix this, only account the entries when it's moved into the output
tree so that they cannot be seen prematurely.  There're some
exceptional cases here and there - they should be addressed separately
with comments.

Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Link: http://lkml.kernel.org/r/1398327843-31845-7-git-send-email-namhyung@kernel.org
Signed-off-by: Jiri Olsa <jolsa@kernel.org>
tools/perf/builtin-annotate.c
tools/perf/builtin-diff.c
tools/perf/builtin-report.c
tools/perf/util/hist.c