perf hists: Mark entries filtered by parent

And don't consider them in hists__inc_nr_entries.

Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 3a01736e
...@@ -5,6 +5,12 @@ ...@@ -5,6 +5,12 @@
#include "sort.h" #include "sort.h"
#include <math.h> #include <math.h>
enum hist_filter {
HIST_FILTER__DSO,
HIST_FILTER__THREAD,
HIST_FILTER__PARENT,
};
struct callchain_param callchain_param = { struct callchain_param callchain_param = {
.mode = CHAIN_GRAPH_REL, .mode = CHAIN_GRAPH_REL,
.min_percent = 0.5 .min_percent = 0.5
...@@ -52,11 +58,20 @@ static struct hist_entry *hist_entry__new(struct hist_entry *template) ...@@ -52,11 +58,20 @@ static struct hist_entry *hist_entry__new(struct hist_entry *template)
static void hists__inc_nr_entries(struct hists *self, struct hist_entry *entry) static void hists__inc_nr_entries(struct hists *self, struct hist_entry *entry)
{ {
if (entry->filtered)
return;
if (entry->ms.sym && self->max_sym_namelen < entry->ms.sym->namelen) if (entry->ms.sym && self->max_sym_namelen < entry->ms.sym->namelen)
self->max_sym_namelen = entry->ms.sym->namelen; self->max_sym_namelen = entry->ms.sym->namelen;
++self->nr_entries; ++self->nr_entries;
} }
static u8 symbol__parent_filter(const struct symbol *parent)
{
if (symbol_conf.exclude_other && parent == NULL)
return 1 << HIST_FILTER__PARENT;
return 0;
}
struct hist_entry *__hists__add_entry(struct hists *self, struct hist_entry *__hists__add_entry(struct hists *self,
struct addr_location *al, struct addr_location *al,
struct symbol *sym_parent, u64 period) struct symbol *sym_parent, u64 period)
...@@ -75,6 +90,7 @@ struct hist_entry *__hists__add_entry(struct hists *self, ...@@ -75,6 +90,7 @@ struct hist_entry *__hists__add_entry(struct hists *self,
.level = al->level, .level = al->level,
.period = period, .period = period,
.parent = sym_parent, .parent = sym_parent,
.filtered = symbol__parent_filter(sym_parent),
}; };
int cmp; int cmp;
...@@ -790,11 +806,6 @@ size_t hists__fprintf(struct hists *self, struct hists *pair, ...@@ -790,11 +806,6 @@ size_t hists__fprintf(struct hists *self, struct hists *pair,
return ret; return ret;
} }
enum hist_filter {
HIST_FILTER__DSO,
HIST_FILTER__THREAD,
};
static void hists__remove_entry_filter(struct hists *self, struct hist_entry *h, static void hists__remove_entry_filter(struct hists *self, struct hist_entry *h,
enum hist_filter filter) enum hist_filter filter)
{ {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment