diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2009-12-14 08:37:11 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-12-14 10:57:17 -0500 |
commit | b9bf089212d95746ce66482bcdbc7e77a0651088 (patch) | |
tree | f6a5d219d100498a2c16c1fb3a555f518c2c528d /tools/perf/builtin-report.c | |
parent | 4aa65636411ccb12f006a6ad593930655c445ff6 (diff) |
perf tools: No need for three rb_trees for sorting hist entries
All hist entries are in only one of them, so use just one and a
temporary rb_root while sorting/collapsing.
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Frédéric Weisbecker <fweisbec@gmail.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Paul Mackerras <paulus@samba.org>
LKML-Reference: <1260797831-11220-1-git-send-email-acme@infradead.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'tools/perf/builtin-report.c')
-rw-r--r-- | tools/perf/builtin-report.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c index 3487224670a8..9cbdcbc4cd56 100644 --- a/tools/perf/builtin-report.c +++ b/tools/perf/builtin-report.c | |||
@@ -567,7 +567,7 @@ static size_t output__fprintf(FILE *fp, u64 total_samples) | |||
567 | fprintf(fp, "#\n"); | 567 | fprintf(fp, "#\n"); |
568 | 568 | ||
569 | print_entries: | 569 | print_entries: |
570 | for (nd = rb_first(&output_hists); nd; nd = rb_next(nd)) { | 570 | for (nd = rb_first(&hist); nd; nd = rb_next(nd)) { |
571 | pos = rb_entry(nd, struct hist_entry, rb_node); | 571 | pos = rb_entry(nd, struct hist_entry, rb_node); |
572 | ret += hist_entry__fprintf(fp, pos, total_samples); | 572 | ret += hist_entry__fprintf(fp, pos, total_samples); |
573 | } | 573 | } |