diff options
| author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2009-08-12 18:19:57 -0300 |
|---|---|---|
| committer | Ingo Molnar <mingo@elte.hu> | 2009-08-13 12:05:52 +0200 |
| commit | 8fd101f20bdf771949a8f3a5a779877d09b2fb56 (patch) | |
| tree | 3addf01571a9a6a59cf1590a74188b288aec4ea3 /tools | |
| parent | daac07b2e6b77f1bd44104aa2f0593e5505f27d4 (diff) | |
| download | lwn-8fd101f20bdf771949a8f3a5a779877d09b2fb56.tar.gz lwn-8fd101f20bdf771949a8f3a5a779877d09b2fb56.zip | |
perf report: Don't show unresolved DSOs and symbols when -S/-d is used
We're interested in just those symbols/DSOs, so filter out the
unresolved ones.
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Peter Zijlstra <peterz@infradead.org>
LKML-Reference: <20090812211957.GE3495@ghostprotocols.net>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/perf/builtin-report.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c index 23e1457f2409..b53a60fc12de 100644 --- a/tools/perf/builtin-report.c +++ b/tools/perf/builtin-report.c @@ -1590,10 +1590,11 @@ process_sample_event(event_t *event, unsigned long offset, unsigned long head) if (show & show_mask) { struct symbol *sym = resolve_symbol(thread, &map, &dso, &ip); - if (dso_list && dso && dso->name && !strlist__has_entry(dso_list, dso->name)) + if (dso_list && (!dso || !dso->name || + !strlist__has_entry(dso_list, dso->name))) return 0; - if (sym_list && sym && !strlist__has_entry(sym_list, sym->name)) + if (sym_list && (!sym || !strlist__has_entry(sym_list, sym->name))) return 0; if (hist_entry__add(thread, map, dso, sym, ip, chain, level, period)) { |