Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753944AbbHRQnf (ORCPT ); Tue, 18 Aug 2015 12:43:35 -0400 Received: from mga11.intel.com ([192.55.52.93]:35956 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753709AbbHRQlX (ORCPT ); Tue, 18 Aug 2015 12:41:23 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.15,702,1432623600"; d="scan'208";a="786447412" From: kan.liang@intel.com To: acme@kernel.org Cc: a.p.zijlstra@chello.nl, mingo@redhat.com, jolsa@kernel.org, namhyung@kernel.org, ak@linux.intel.com, eranian@google.com, linux-kernel@vger.kernel.org, Kan Liang Subject: [PATCH RFC 05/10] perf,tools: Enable statistic read for perf record Date: Tue, 18 Aug 2015 05:25:41 -0400 Message-Id: <1439889946-28986-6-git-send-email-kan.liang@intel.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1439889946-28986-1-git-send-email-kan.liang@intel.com> References: <1439889946-28986-1-git-send-email-kan.liang@intel.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3748 Lines: 103 From: Kan Liang Using 'N' event/group modifier to specify the event which want to read counter statistics during sampling. For this event, the sampling will be disabled unless it's group leader. The 'N' modifier only be available for in system-wide/CPU mode. If a group is marked as 'N' modifier, only group members read counter statistics. Group leader always do sampling. The other limit is that the first event cannot be stat read event, since many tools special handle first event. Signed-off-by: Kan Liang --- tools/perf/Documentation/perf-list.txt | 5 +++++ tools/perf/builtin-record.c | 31 +++++++++++++++++++++++++++++++ tools/perf/util/evlist.c | 3 +++ 3 files changed, 39 insertions(+) diff --git a/tools/perf/Documentation/perf-list.txt b/tools/perf/Documentation/perf-list.txt index bada893..133dc2c 100644 --- a/tools/perf/Documentation/perf-list.txt +++ b/tools/perf/Documentation/perf-list.txt @@ -31,6 +31,11 @@ counted. The following modifiers exist: H - host counting (not in KVM guests) p - precise level S - read sample value (PERF_SAMPLE_READ) + N - read counter statistics during sampling (Can be used to read + counter statistics of PMU_A event when PMU_B events are sampling. + For example, getting memory bandwidth by uncore events during the + CPU PMU events run time) + (Only available for group members or non-first event in system-wide/CPU mode) D - pin the event to the PMU The 'p' modifier can be used for specifying how precise the instruction diff --git a/tools/perf/builtin-record.c b/tools/perf/builtin-record.c index a0178bf..5b09318 100644 --- a/tools/perf/builtin-record.c +++ b/tools/perf/builtin-record.c @@ -273,11 +273,42 @@ static int record__open(struct record *rec) struct perf_evlist *evlist = rec->evlist; struct perf_session *session = rec->session; struct record_opts *opts = &rec->opts; + struct perf_evsel *first = perf_evlist__first(evlist); + struct perf_event_attr *attr; int rc = 0; perf_evlist__config(evlist, opts); evlist__for_each(evlist, pos) { + + if (pos->stat_read) { + if (!target__has_cpu(&opts->target)) { + pos->stat_read = 0; + ui__warning("Statistics read only available " + "on system-wide/CPU mode. " + "Remove :N modifier for event %s\n", + pos->name); + goto out; + } + /* Don't do stat read for Group leader */ + if ((pos->leader == pos) && (pos->leader->nr_members > 1)) + pos->stat_read = 0; + else { + if (first == pos) { + pos->stat_read = 0; + ui__warning("The first event cannot " + "be stat read event\n"); + goto out; + } + attr = &pos->attr; + attr->read_format = PERF_FORMAT_TOTAL_TIME_ENABLED | + PERF_FORMAT_TOTAL_TIME_RUNNING; + attr->sample_freq = 0; + attr->sample_period = 0; + attr->sample_type = 0; + pos->sample_size = 0; + } + } try_again: if (perf_evsel__open(pos, pos->cpus, pos->threads) < 0) { if (perf_evsel__fallback(pos, errno, msg, sizeof(msg))) { diff --git a/tools/perf/util/evlist.c b/tools/perf/util/evlist.c index 373f65b..ca7bf8d 100644 --- a/tools/perf/util/evlist.c +++ b/tools/perf/util/evlist.c @@ -854,6 +854,9 @@ static int perf_evlist__mmap_per_evsel(struct perf_evlist *evlist, int idx, if (evsel->system_wide && thread) continue; + if (evsel->stat_read) + continue; + fd = FD(evsel, cpu, thread); if (*output == -1) { -- 1.8.3.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/