mirror of
https://github.com/torvalds/linux
synced 2024-10-01 00:39:03 +00:00
3e43d79da1
- auxtrace_record__init() is called only once, so there is no point in using a static variable to cache the results of find_all_arm_spe_pmus(), make it local and free the results after use. - Another reason is, even though SPE is micro-architecture dependent, but so far it only supports "statistical-profiling-extension-v1" and we have no chance to use multiple SPE's PMU events in Perf command. So remove the useless check code to make it clear. Signed-off-by: Wei Li <liwei391@huawei.com> Reviewed-by: Mathieu Poirier <mathieu.poirier@linaro.org> Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com> Cc: Hanjun Guo <guohanjun@huawei.com> Cc: Jiri Olsa <jolsa@redhat.com> Cc: Kim Phillips <kim.phillips@arm.com> Cc: Leo Yan <leo.yan@linaro.org> Cc: Mark Rutland <mark.rutland@arm.com> Cc: Mike Leach <mike.leach@linaro.org> Cc: Namhyung Kim <namhyung@kernel.org> Cc: Peter Zijlstra <peterz@infradead.org> Cc: Suzuki Poulouse <suzuki.poulose@arm.com> Cc: linux-arm-kernel@lists.infradead.org Link: http://lore.kernel.org/lkml/20200724071111.35593-3-liwei391@huawei.com Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
110 lines
2.6 KiB
C
110 lines
2.6 KiB
C
// SPDX-License-Identifier: GPL-2.0
|
|
/*
|
|
* Copyright(C) 2015 Linaro Limited. All rights reserved.
|
|
* Author: Mathieu Poirier <mathieu.poirier@linaro.org>
|
|
*/
|
|
|
|
#include <stdbool.h>
|
|
#include <linux/coresight-pmu.h>
|
|
#include <linux/zalloc.h>
|
|
|
|
#include "../../util/auxtrace.h"
|
|
#include "../../util/debug.h"
|
|
#include "../../util/evlist.h"
|
|
#include "../../util/pmu.h"
|
|
#include "cs-etm.h"
|
|
#include "arm-spe.h"
|
|
|
|
static struct perf_pmu **find_all_arm_spe_pmus(int *nr_spes, int *err)
|
|
{
|
|
struct perf_pmu **arm_spe_pmus = NULL;
|
|
int ret, i, nr_cpus = sysconf(_SC_NPROCESSORS_CONF);
|
|
/* arm_spe_xxxxxxxxx\0 */
|
|
char arm_spe_pmu_name[sizeof(ARM_SPE_PMU_NAME) + 10];
|
|
|
|
arm_spe_pmus = zalloc(sizeof(struct perf_pmu *) * nr_cpus);
|
|
if (!arm_spe_pmus) {
|
|
pr_err("spes alloc failed\n");
|
|
*err = -ENOMEM;
|
|
return NULL;
|
|
}
|
|
|
|
for (i = 0; i < nr_cpus; i++) {
|
|
ret = sprintf(arm_spe_pmu_name, "%s%d", ARM_SPE_PMU_NAME, i);
|
|
if (ret < 0) {
|
|
pr_err("sprintf failed\n");
|
|
*err = -ENOMEM;
|
|
return NULL;
|
|
}
|
|
|
|
arm_spe_pmus[*nr_spes] = perf_pmu__find(arm_spe_pmu_name);
|
|
if (arm_spe_pmus[*nr_spes]) {
|
|
pr_debug2("%s %d: arm_spe_pmu %d type %d name %s\n",
|
|
__func__, __LINE__, *nr_spes,
|
|
arm_spe_pmus[*nr_spes]->type,
|
|
arm_spe_pmus[*nr_spes]->name);
|
|
(*nr_spes)++;
|
|
}
|
|
}
|
|
|
|
return arm_spe_pmus;
|
|
}
|
|
|
|
struct auxtrace_record
|
|
*auxtrace_record__init(struct evlist *evlist, int *err)
|
|
{
|
|
struct perf_pmu *cs_etm_pmu;
|
|
struct evsel *evsel;
|
|
bool found_etm = false;
|
|
struct perf_pmu *found_spe = NULL;
|
|
struct perf_pmu **arm_spe_pmus = NULL;
|
|
int nr_spes = 0;
|
|
int i = 0;
|
|
|
|
if (!evlist)
|
|
return NULL;
|
|
|
|
cs_etm_pmu = perf_pmu__find(CORESIGHT_ETM_PMU_NAME);
|
|
arm_spe_pmus = find_all_arm_spe_pmus(&nr_spes, err);
|
|
|
|
evlist__for_each_entry(evlist, evsel) {
|
|
if (cs_etm_pmu &&
|
|
evsel->core.attr.type == cs_etm_pmu->type)
|
|
found_etm = true;
|
|
|
|
if (!nr_spes || found_spe)
|
|
continue;
|
|
|
|
for (i = 0; i < nr_spes; i++) {
|
|
if (evsel->core.attr.type == arm_spe_pmus[i]->type) {
|
|
found_spe = arm_spe_pmus[i];
|
|
break;
|
|
}
|
|
}
|
|
}
|
|
free(arm_spe_pmus);
|
|
|
|
if (found_etm && found_spe) {
|
|
pr_err("Concurrent ARM Coresight ETM and SPE operation not currently supported\n");
|
|
*err = -EOPNOTSUPP;
|
|
return NULL;
|
|
}
|
|
|
|
if (found_etm)
|
|
return cs_etm_record_init(err);
|
|
|
|
#if defined(__aarch64__)
|
|
if (found_spe)
|
|
return arm_spe_recording_init(err, found_spe);
|
|
#endif
|
|
|
|
/*
|
|
* Clear 'err' even if we haven't found an event - that way perf
|
|
* record can still be used even if tracers aren't present. The NULL
|
|
* return value will take care of telling the infrastructure HW tracing
|
|
* isn't available.
|
|
*/
|
|
*err = 0;
|
|
return NULL;
|
|
}
|