diff options
| author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2010-05-09 13:02:23 -0300 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2010-05-09 13:10:39 -0300 |
| commit | 28e2a106d16046ca792722795f809e3f80a5af80 (patch) | |
| tree | c84149ddf45d02044187fe4511cead93d009b6ee /tools/perf/util/hist.h | |
| parent | 39d1e1b1e26dc84d40bf2792287d0d61e44b57df (diff) | |
| download | kernel_samsung_crespo-28e2a106d16046ca792722795f809e3f80a5af80.zip kernel_samsung_crespo-28e2a106d16046ca792722795f809e3f80a5af80.tar.gz kernel_samsung_crespo-28e2a106d16046ca792722795f809e3f80a5af80.tar.bz2 | |
perf hist: Simplify the insertion of new hist_entry instances
And with that fix at least one bug:
The first hit for an entry, the one that calls malloc to create a new
instance in __perf_session__add_hist_entry, wasn't adding the count to
the per cpumode (PERF_RECORD_MISC_USER, etc) total variable.
Cc: Frédéric Weisbecker <fweisbec@gmail.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Tom Zanussi <tzanussi@gmail.com>
LKML-Reference: <new-submission>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/util/hist.h')
| -rw-r--r-- | tools/perf/util/hist.h | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/tools/perf/util/hist.h b/tools/perf/util/hist.h index 9df1c34..b49013a 100644 --- a/tools/perf/util/hist.h +++ b/tools/perf/util/hist.h @@ -12,13 +12,10 @@ struct addr_location; struct symbol; struct rb_root; -void __perf_session__add_count(struct hist_entry *he, - struct addr_location *al, - u64 count); struct hist_entry *__perf_session__add_hist_entry(struct rb_root *hists, struct addr_location *al, struct symbol *parent, - u64 count, bool *hit); + u64 count); extern int64_t hist_entry__cmp(struct hist_entry *, struct hist_entry *); extern int64_t hist_entry__collapse(struct hist_entry *, struct hist_entry *); int hist_entry__fprintf(struct hist_entry *self, |