From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752462Ab3LXI0O (ORCPT ); Tue, 24 Dec 2013 03:26:14 -0500 Received: from LGEMRELSE6Q.lge.com ([156.147.1.121]:50238 "EHLO LGEMRELSE6Q.lge.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751739Ab3LXIWf (ORCPT ); Tue, 24 Dec 2013 03:22:35 -0500 X-AuditID: 9c930179-b7c89ae000006438-07-52b94448a804 From: Namhyung Kim To: Arnaldo Carvalho de Melo Cc: Peter Zijlstra , Paul Mackerras , Ingo Molnar , Namhyung Kim , LKML , Arun Sharma , Frederic Weisbecker , Jiri Olsa , Rodrigo Campos Subject: [PATCH 12/21] perf tools: Apply percent-limit to cumulative percentage Date: Tue, 24 Dec 2013 17:22:18 +0900 Message-Id: <1387873347-28838-13-git-send-email-namhyung@kernel.org> X-Mailer: git-send-email 1.7.11.7 In-Reply-To: <1387873347-28838-1-git-send-email-namhyung@kernel.org> References: <1387873347-28838-1-git-send-email-namhyung@kernel.org> X-Brightmail-Tracker: AAAAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Namhyung Kim If -g cumulative option is given, it needs to show entries which don't have self overhead. So apply percent-limit to accumulated overhead percentage in this case. Cc: Arun Sharma Cc: Frederic Weisbecker Signed-off-by: Namhyung Kim --- tools/perf/ui/browsers/hists.c | 34 ++++++++++++++++++++++++++-------- tools/perf/ui/gtk/hists.c | 11 +++++++++-- tools/perf/ui/stdio/hist.c | 11 +++++++++-- 3 files changed, 44 insertions(+), 12 deletions(-) diff --git a/tools/perf/ui/browsers/hists.c b/tools/perf/ui/browsers/hists.c index efa78894f70d..b02e71ecc5fe 100644 --- a/tools/perf/ui/browsers/hists.c +++ b/tools/perf/ui/browsers/hists.c @@ -828,12 +828,19 @@ static unsigned int hist_browser__refresh(struct ui_browser *browser) for (nd = browser->top; nd; nd = rb_next(nd)) { struct hist_entry *h = rb_entry(nd, struct hist_entry, rb_node); - float percent = h->stat.period * 100.0 / - hb->hists->stats.total_period; + float percent; if (h->filtered) continue; + if (symbol_conf.cumulate_callchain) { + percent = h->stat_acc->period * 100.0 / + hb->hists->stats.total_period; + } else { + percent = h->stat.period * 100.0 / + hb->hists->stats.total_period; + } + if (percent < hb->min_pcnt) continue; @@ -851,13 +858,17 @@ static struct rb_node *hists__filter_entries(struct rb_node *nd, { while (nd != NULL) { struct hist_entry *h = rb_entry(nd, struct hist_entry, rb_node); - float percent = h->stat.period * 100.0 / - hists->stats.total_period; + float percent; - if (percent < min_pcnt) - return NULL; + if (symbol_conf.cumulate_callchain) { + percent = h->stat_acc->period * 100.0 / + hists->stats.total_period; + } else { + percent = h->stat.period * 100.0 / + hists->stats.total_period; + } - if (!h->filtered) + if (!h->filtered && percent >= min_pcnt) return nd; nd = rb_next(nd); @@ -872,8 +883,15 @@ static struct rb_node *hists__filter_prev_entries(struct rb_node *nd, { while (nd != NULL) { struct hist_entry *h = rb_entry(nd, struct hist_entry, rb_node); - float percent = h->stat.period * 100.0 / + float percent; + + if (symbol_conf.cumulate_callchain) { + percent = h->stat_acc->period * 100.0 / + hists->stats.total_period; + } else { + percent = h->stat.period * 100.0 / hists->stats.total_period; + } if (!h->filtered && percent >= min_pcnt) return nd; diff --git a/tools/perf/ui/gtk/hists.c b/tools/perf/ui/gtk/hists.c index 70ed0d5e1b94..06ae3342e14f 100644 --- a/tools/perf/ui/gtk/hists.c +++ b/tools/perf/ui/gtk/hists.c @@ -296,12 +296,19 @@ static void perf_gtk__show_hists(GtkWidget *window, struct hists *hists, for (nd = rb_first(&hists->entries); nd; nd = rb_next(nd)) { struct hist_entry *h = rb_entry(nd, struct hist_entry, rb_node); GtkTreeIter iter; - float percent = h->stat.period * 100.0 / - hists->stats.total_period; + float percent; if (h->filtered) continue; + if (symbol_conf.cumulate_callchain) { + percent = h->stat_acc->period * 100.0 / + hists->stats.total_period; + } else { + percent = h->stat.period * 100.0 / + hists->stats.total_period; + } + if (percent < min_pcnt) continue; diff --git a/tools/perf/ui/stdio/hist.c b/tools/perf/ui/stdio/hist.c index 4c4986e809d8..7ea8502192b0 100644 --- a/tools/perf/ui/stdio/hist.c +++ b/tools/perf/ui/stdio/hist.c @@ -487,12 +487,19 @@ print_entries: for (nd = rb_first(&hists->entries); nd; nd = rb_next(nd)) { struct hist_entry *h = rb_entry(nd, struct hist_entry, rb_node); - float percent = h->stat.period * 100.0 / - hists->stats.total_period; + float percent; if (h->filtered) continue; + if (symbol_conf.cumulate_callchain) { + percent = h->stat_acc->period * 100.0 / + hists->stats.total_period; + } else { + percent = h->stat.period * 100.0 / + hists->stats.total_period; + } + if (percent < min_pcnt) continue; -- 1.7.11.7