perf tools: Don't display ignored entries on stdio ui
authorFrederic Weisbecker <fweisbec@gmail.com>
Wed, 29 Jun 2011 20:23:03 +0000 (22:23 +0200)
committerFrederic Weisbecker <fweisbec@gmail.com>
Wed, 29 Jun 2011 22:26:33 +0000 (00:26 +0200)
As for newt ui, don't display entries that have been marked
as ignored.

The practical current effect of this is to make parent
filtering really working. Before, entries that were ignored
were given a null parent but were still displayed. This
resulted in some weird effects:

 # Overhead      Command      Shared Object        Symbol
 # ........  ...........  .................  ............
 #
^A
                   |
                   --- __lock_acquire
                      |
                      |--95.97%-- lock_acquire
                      |          |
                      |          |--30.75%-- _raw_spin_lock

Discard these from the stdio display.

Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Stephane Eranian <eranian@google.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Sam Liao <phyomh@gmail.com>
tools/perf/util/hist.c

index dae4202fa65ae7bcfc6ae600c09dcf849611e316..677e1da6bb3eae6d688cd7e1afc51070e55810ea 100644 (file)
@@ -847,6 +847,9 @@ print_entries:
        for (nd = rb_first(&self->entries); nd; nd = rb_next(nd)) {
                struct hist_entry *h = rb_entry(nd, struct hist_entry, rb_node);
 
+               if (h->filtered)
+                       continue;
+
                if (show_displacement) {
                        if (h->pair != NULL)
                                displacement = ((long)h->pair->position -