From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755225AbbDNMFZ (ORCPT ); Tue, 14 Apr 2015 08:05:25 -0400 Received: from mail-wg0-f53.google.com ([74.125.82.53]:33208 "EHLO mail-wg0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754359AbbDNMFV (ORCPT ); Tue, 14 Apr 2015 08:05:21 -0400 From: Steve Capper To: linux-kernel@vger.kernel.org Cc: Steve Capper , Peter Zijlstra , Paul Mackerras , Ingo Molnar , Arnaldo Carvalho de Melo Subject: [RFC PATCH] perf report: Don't add filtered events to histogram Date: Tue, 14 Apr 2015 13:05:03 +0100 Message-Id: <1429013103-22718-1-git-send-email-steve.capper@linaro.org> X-Mailer: git-send-email 1.7.10.4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org If one filters events by a single comm in perf report via: perf report --no-children -c program-name perf will elide the comm column as only one comm is filtered in. This lack of distinction by comm can lead to events being merged in add_hist_entry that have different comm's, including events that have been filtered out and filtered in. Ultimately, this can lead to events disappearing from the report. If one instead invokes perf via: perf report --no-children -c program-name,random-made-up-name Then the comm column remains and all the expected events are present. This patch prevents filtered out events from being considered by hist_entry_iter__add. Cc: Peter Zijlstra Cc: Paul Mackerras Cc: Ingo Molnar Cc: Arnaldo Carvalho de Melo Signed-off-by: Steve Capper --- Hi, This patch fixes my problem, but I don't know enough perf to confidently state that this will work everywhere (hence this being marked as an RFC). Please let me know if I've missed something and I'll be happy to re-spin/test/submit. Cheers, -- Steve --- tools/perf/builtin-report.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c index 2f91094..3eaf46e 100644 --- a/tools/perf/builtin-report.c +++ b/tools/perf/builtin-report.c @@ -166,6 +166,9 @@ static int process_sample_event(struct perf_tool *tool, if (al.map != NULL) al.map->dso->hit = 1; + if (al.filtered) + return 0; + ret = hist_entry_iter__add(&iter, &al, evsel, sample, rep->max_stack, rep); if (ret < 0) -- 2.1.0