diff options
author | Sage Weil <sage@inktank.com> | 2013-08-15 14:11:45 -0400 |
---|---|---|
committer | Sage Weil <sage@inktank.com> | 2013-08-15 14:11:45 -0400 |
commit | ee3e542fec6e69bc9fb668698889a37d93950ddf (patch) | |
tree | e74ee766a4764769ef1d3d45d266b4dea64101d3 /tools/perf/builtin-diff.c | |
parent | fe2a801b50c0bb8039d627e5ae1fec249d10ff39 (diff) | |
parent | f1d6e17f540af37bb1891480143669ba7636c4cf (diff) |
Merge remote-tracking branch 'linus/master' into testing
Diffstat (limited to 'tools/perf/builtin-diff.c')
-rw-r--r-- | tools/perf/builtin-diff.c | 20 |
1 files changed, 12 insertions, 8 deletions
diff --git a/tools/perf/builtin-diff.c b/tools/perf/builtin-diff.c index 2d0462d89a97..0aac5f3e594d 100644 --- a/tools/perf/builtin-diff.c +++ b/tools/perf/builtin-diff.c | |||
@@ -323,13 +323,20 @@ static void hists__baseline_only(struct hists *hists) | |||
323 | 323 | ||
324 | static void hists__precompute(struct hists *hists) | 324 | static void hists__precompute(struct hists *hists) |
325 | { | 325 | { |
326 | struct rb_node *next = rb_first(&hists->entries); | 326 | struct rb_root *root; |
327 | struct rb_node *next; | ||
328 | |||
329 | if (sort__need_collapse) | ||
330 | root = &hists->entries_collapsed; | ||
331 | else | ||
332 | root = hists->entries_in; | ||
327 | 333 | ||
334 | next = rb_first(root); | ||
328 | while (next != NULL) { | 335 | while (next != NULL) { |
329 | struct hist_entry *he = rb_entry(next, struct hist_entry, rb_node); | 336 | struct hist_entry *he = rb_entry(next, struct hist_entry, rb_node_in); |
330 | struct hist_entry *pair = hist_entry__next_pair(he); | 337 | struct hist_entry *pair = hist_entry__next_pair(he); |
331 | 338 | ||
332 | next = rb_next(&he->rb_node); | 339 | next = rb_next(&he->rb_node_in); |
333 | if (!pair) | 340 | if (!pair) |
334 | continue; | 341 | continue; |
335 | 342 | ||
@@ -457,7 +464,7 @@ static void hists__process(struct hists *old, struct hists *new) | |||
457 | hists__output_resort(new); | 464 | hists__output_resort(new); |
458 | } | 465 | } |
459 | 466 | ||
460 | hists__fprintf(new, true, 0, 0, stdout); | 467 | hists__fprintf(new, true, 0, 0, 0, stdout); |
461 | } | 468 | } |
462 | 469 | ||
463 | static int __cmd_diff(void) | 470 | static int __cmd_diff(void) |
@@ -600,7 +607,6 @@ int cmd_diff(int argc, const char **argv, const char *prefix __maybe_unused) | |||
600 | input_new = "perf.data.guest"; | 607 | input_new = "perf.data.guest"; |
601 | } | 608 | } |
602 | 609 | ||
603 | symbol_conf.exclude_other = false; | ||
604 | if (symbol__init() < 0) | 610 | if (symbol__init() < 0) |
605 | return -1; | 611 | return -1; |
606 | 612 | ||
@@ -611,9 +617,7 @@ int cmd_diff(int argc, const char **argv, const char *prefix __maybe_unused) | |||
611 | 617 | ||
612 | setup_pager(); | 618 | setup_pager(); |
613 | 619 | ||
614 | sort_entry__setup_elide(&sort_dso, symbol_conf.dso_list, "dso", NULL); | 620 | sort__setup_elide(NULL); |
615 | sort_entry__setup_elide(&sort_comm, symbol_conf.comm_list, "comm", NULL); | ||
616 | sort_entry__setup_elide(&sort_sym, symbol_conf.sym_list, "symbol", NULL); | ||
617 | 621 | ||
618 | return __cmd_diff(); | 622 | return __cmd_diff(); |
619 | } | 623 | } |