Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751637AbaKFMZn (ORCPT ); Thu, 6 Nov 2014 07:25:43 -0500 Received: from mail-wi0-f174.google.com ([209.85.212.174]:39953 "EHLO mail-wi0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751160AbaKFMXa (ORCPT ); Thu, 6 Nov 2014 07:23:30 -0500 From: Matt Fleming To: Peter Zijlstra , Ingo Molnar , Jiri Olsa , Arnaldo Carvalho de Melo , Andi Kleen Cc: Thomas Gleixner , linux-kernel@vger.kernel.org, "H. Peter Anvin" , Kanaka Juvva , Matt Fleming , Arnaldo Carvalho de Melo Subject: [PATCH 01/11] perf tools: Parse event per-package info files Date: Thu, 6 Nov 2014 12:23:12 +0000 Message-Id: <1415276602-10337-2-git-send-email-matt@console-pimps.org> X-Mailer: git-send-email 1.9.3 In-Reply-To: <1415276602-10337-1-git-send-email-matt@console-pimps.org> References: <1415276602-10337-1-git-send-email-matt@console-pimps.org> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Matt Fleming In preparation for upcoming PMU drivers that support system-wide, per-package counters and hence report duplicate values, add support for parsing the .per-pkg file. An event can export this info file to indicate that all but one value per socket should be discarded. The discarding is much easier to do in userspace than inside the kernel because the kernel cannot infer what userspace is going to do with the reported values, what order it will read them in, etc. Cc: Jiri Olsa Cc: Arnaldo Carvalho de Melo Cc: Peter Zijlstra Signed-off-by: Matt Fleming --- tools/perf/builtin-stat.c | 88 +++++++++++++++++++++++++++++++++++++++++- tools/perf/util/evsel.c | 6 ++- tools/perf/util/evsel.h | 8 ++-- tools/perf/util/parse-events.c | 1 + tools/perf/util/pmu.c | 27 +++++++++++++ tools/perf/util/pmu.h | 2 + 6 files changed, 127 insertions(+), 5 deletions(-) diff --git a/tools/perf/builtin-stat.c b/tools/perf/builtin-stat.c index 055ce9232c9e..fa5264668187 100644 --- a/tools/perf/builtin-stat.c +++ b/tools/perf/builtin-stat.c @@ -389,17 +389,92 @@ static void update_shadow_stats(struct perf_evsel *counter, u64 *count) } /* + * If 'evsel' is a per-socket event we may get duplicate values + * reported. We need to discard all but one per-socket value. + */ +static bool counter_per_socket_skip(struct perf_evsel *evsel, int cpu, u64 val) +{ + struct cpu_map *map; + int i, ncpus; + int s1, s2; + + if (!evsel->system_wide) + return false; + + map = perf_evsel__cpus(evsel); + ncpus = map->nr; + + s1 = cpu_map__get_socket(evsel_list->cpus, map->map[cpu]); + + /* + * Read all CPUs for this socket and see if any already have + * value assigned. + */ + for (i = 0; i < ncpus; i++) { + s2 = cpu_map__get_socket(evsel_list->cpus, map->map[i]); + if (s1 != s2) + continue; + + if (evsel->counts->cpu[i].val) + return true; + } + + /* Stash the counter value in unused ->counts */ + evsel->counts->cpu[cpu].val = val; + return false; +} + +static bool aggr_per_socket_skip(struct perf_evsel *evsel, int cpu) +{ + struct cpu_map *map; + int leader_cpu = -1; + int i, ncpus; + int s1, s2; + + map = perf_evsel__cpus(evsel); + ncpus = map->nr; + + s1 = cpu_map__get_socket(evsel_list->cpus, map->map[cpu]); + + /* + * Find the first enabled counter for this socket and skip + * everything else. + */ + for (i = 0; i < ncpus; i++) { + s2 = cpu_map__get_socket(evsel_list->cpus, map->map[i]); + if (s1 != s2) + continue; + + if (!evsel->counts->cpu[i].ena) + continue; + + leader_cpu = i; + break; + } + + if (cpu == leader_cpu) + return false; + + return true; +} + +/* * Read out the results of a single counter: * aggregate counts across CPUs in system-wide mode */ static int read_counter_aggr(struct perf_evsel *counter) { struct perf_stat *ps = counter->priv; + bool (*f_skip)(struct perf_evsel *evsel, int cpu, u64 val) = NULL; u64 *count = counter->counts->aggr.values; int i; + if (counter->per_pkg) + f_skip = counter_per_socket_skip; + if (__perf_evsel__read(counter, perf_evsel__nr_cpus(counter), - thread_map__nr(evsel_list->threads), scale) < 0) + thread_map__nr(evsel_list->threads), + scale, f_skip) < 0) return -1; for (i = 0; i < 3; i++) @@ -451,12 +526,18 @@ static void print_interval(void) evlist__for_each(evsel_list, counter) { ps = counter->priv; memset(ps->res_stats, 0, sizeof(ps->res_stats)); + memset(counter->counts->cpu, 0, + sizeof(struct perf_counts_values) * + perf_evsel__nr_cpus(counter)); read_counter_aggr(counter); } } else { evlist__for_each(evsel_list, counter) { ps = counter->priv; memset(ps->res_stats, 0, sizeof(ps->res_stats)); + memset(counter->counts->cpu, 0, + sizeof(struct perf_counts_values) * + perf_evsel__nr_cpus(counter)); read_counter(counter); } } @@ -1130,6 +1211,11 @@ static void print_aggr(char *prefix) val = ena = run = 0; nr = 0; for (cpu = 0; cpu < perf_evsel__nr_cpus(counter); cpu++) { + if (counter->per_pkg) { + if (aggr_per_socket_skip(counter, cpu)) + continue; + } + cpu2 = perf_evsel__cpus(counter)->map[cpu]; s2 = aggr_get_id(evsel_list->cpus, cpu2); if (s2 != id) diff --git a/tools/perf/util/evsel.c b/tools/perf/util/evsel.c index 2f9e68025ede..a748e5e16533 100644 --- a/tools/perf/util/evsel.c +++ b/tools/perf/util/evsel.c @@ -917,7 +917,8 @@ int __perf_evsel__read_on_cpu(struct perf_evsel *evsel, } int __perf_evsel__read(struct perf_evsel *evsel, - int ncpus, int nthreads, bool scale) + int ncpus, int nthreads, bool scale, + bool (*f_skip)(struct perf_evsel *evsel, int cpu, u64 val)) { size_t nv = scale ? 3 : 1; int cpu, thread; @@ -937,6 +938,9 @@ int __perf_evsel__read(struct perf_evsel *evsel, &count, nv * sizeof(u64)) < 0) return -errno; + if (f_skip && f_skip(evsel, cpu, count.val)) + continue; + aggr->val += count.val; if (scale) { aggr->ena += count.ena; diff --git a/tools/perf/util/evsel.h b/tools/perf/util/evsel.h index 979790951bfb..13ca8a7693e4 100644 --- a/tools/perf/util/evsel.h +++ b/tools/perf/util/evsel.h @@ -91,6 +91,7 @@ struct perf_evsel { bool immediate; bool system_wide; bool tracking; + bool per_pkg; /* parse modifier helper */ int exclude_GH; int nr_members; @@ -257,7 +258,8 @@ static inline int perf_evsel__read_on_cpu_scaled(struct perf_evsel *evsel, } int __perf_evsel__read(struct perf_evsel *evsel, int ncpus, int nthreads, - bool scale); + bool scale, + bool (*f_skip)(struct perf_evsel *evsel, int cpu, u64 val)); /** * perf_evsel__read - Read the aggregate results on all CPUs @@ -269,7 +271,7 @@ int __perf_evsel__read(struct perf_evsel *evsel, int ncpus, int nthreads, static inline int perf_evsel__read(struct perf_evsel *evsel, int ncpus, int nthreads) { - return __perf_evsel__read(evsel, ncpus, nthreads, false); + return __perf_evsel__read(evsel, ncpus, nthreads, false, NULL); } /** @@ -282,7 +284,7 @@ static inline int perf_evsel__read(struct perf_evsel *evsel, static inline int perf_evsel__read_scaled(struct perf_evsel *evsel, int ncpus, int nthreads) { - return __perf_evsel__read(evsel, ncpus, nthreads, true); + return __perf_evsel__read(evsel, ncpus, nthreads, true, NULL); } int perf_evsel__parse_sample(struct perf_evsel *evsel, union perf_event *event, diff --git a/tools/perf/util/parse-events.c b/tools/perf/util/parse-events.c index c659a3ca1283..5a373483f0e4 100644 --- a/tools/perf/util/parse-events.c +++ b/tools/perf/util/parse-events.c @@ -681,6 +681,7 @@ int parse_events_add_pmu(struct list_head *list, int *idx, if (evsel) { evsel->unit = info.unit; evsel->scale = info.scale; + evsel->per_pkg = info.per_pkg; } return evsel ? 0 : -ENOMEM; diff --git a/tools/perf/util/pmu.c b/tools/perf/util/pmu.c index 881b75490533..f003b5a9e059 100644 --- a/tools/perf/util/pmu.c +++ b/tools/perf/util/pmu.c @@ -163,6 +163,24 @@ error: return -1; } +static int +perf_pmu__parse_per_pkg(struct perf_pmu_alias *alias, char *dir, char *name) +{ + char path[PATH_MAX]; + int fd; + + snprintf(path, PATH_MAX, "%s/%s.per-pkg", dir, name); + + fd = open(path, O_RDONLY); + if (fd == -1) + return -1; + + close(fd); + + alias->per_pkg = true; + return 0; +} + static int perf_pmu__new_alias(struct list_head *list, char *dir, char *name, FILE *file) { struct perf_pmu_alias *alias; @@ -181,6 +199,7 @@ static int perf_pmu__new_alias(struct list_head *list, char *dir, char *name, FI INIT_LIST_HEAD(&alias->terms); alias->scale = 1.0; alias->unit[0] = '\0'; + alias->per_pkg = false; ret = parse_events_terms(&alias->terms, buf); if (ret) { @@ -194,6 +213,7 @@ static int perf_pmu__new_alias(struct list_head *list, char *dir, char *name, FI */ perf_pmu__parse_unit(alias, dir, name); perf_pmu__parse_scale(alias, dir, name); + perf_pmu__parse_per_pkg(alias, dir, name); list_add_tail(&alias->list, list); @@ -209,6 +229,8 @@ static inline bool pmu_alias_info_file(char *name) return true; if (len > 6 && !strcmp(name + len - 6, ".scale")) return true; + if (len > 8 && !strcmp(name + len - 8, ".per-pkg")) + return true; return false; } @@ -649,6 +671,8 @@ int perf_pmu__check_alias(struct perf_pmu *pmu, struct list_head *head_terms, struct perf_pmu_alias *alias; int ret; + info->per_pkg = false; + /* * Mark unit and scale as not set * (different from default values, see below) @@ -668,6 +692,9 @@ int perf_pmu__check_alias(struct perf_pmu *pmu, struct list_head *head_terms, if (ret) return ret; + if (alias->per_pkg) + info->per_pkg = true; + list_del(&term->list); free(term); } diff --git a/tools/perf/util/pmu.h b/tools/perf/util/pmu.h index 8092de78e818..c3a74e0e17a2 100644 --- a/tools/perf/util/pmu.h +++ b/tools/perf/util/pmu.h @@ -29,6 +29,7 @@ struct perf_pmu { struct perf_pmu_info { const char *unit; double scale; + bool per_pkg; }; #define UNIT_MAX_LEN 31 /* max length for event unit name */ @@ -39,6 +40,7 @@ struct perf_pmu_alias { struct list_head list; /* ELEM */ char unit[UNIT_MAX_LEN+1]; double scale; + bool per_pkg; }; struct perf_pmu *perf_pmu__find(const char *name); -- 1.9.3 -- 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/