Commit df5a5f3c authored by John Garry's avatar John Garry Committed by Arnaldo Carvalho de Melo

perf tools: Add arm64 version of get_cpuid()

Add an arm64 version of get_cpuid(), which is used for various annotation
and headers - for example, I now get the CPUID in "perf report --header",
as shown in this snippet:

  # hostname : ubuntu
  # os release : 5.5.0-rc1-dirty
  # perf version : 5.5.rc1.gbf8a13dc9851
  # arch : aarch64
  # nrcpus online : 96
  # nrcpus avail : 96
  # cpuid : 0x00000000480fd010

Since much of the code to read the MIDR is already in get_cpuid_str(),
factor out this code.

Tester notes:

I tested this patch on my new ARM64 Kunpeng 920 server.
[root@node1 zsk]# ./perf --version
perf version 5.6.rc1.g2cdb955b7252

Both perf list and perf stat can work.
Signed-off-by: default avatarJohn Garry <john.garry@huawei.com>
Tested-by: default avatarShaokun Zhang <zhangshaokun@hisilicon.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Will Deacon <will@kernel.org>
Cc: linux-arm-kernel@lists.infradead.org
Cc: linuxarm@huawei.com
Link: http://lore.kernel.org/lkml/1576245255-210926-1-git-send-email-john.garry@huawei.comSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 365f9cc1
#include <stdio.h> #include <stdio.h>
#include <stdlib.h> #include <stdlib.h>
#include <perf/cpumap.h> #include <perf/cpumap.h>
#include <util/cpumap.h>
#include <internal/cpumap.h> #include <internal/cpumap.h>
#include <api/fs/fs.h> #include <api/fs/fs.h>
#include <errno.h>
#include "debug.h" #include "debug.h"
#include "header.h" #include "header.h"
...@@ -12,26 +14,21 @@ ...@@ -12,26 +14,21 @@
#define MIDR_VARIANT_SHIFT 20 #define MIDR_VARIANT_SHIFT 20
#define MIDR_VARIANT_MASK (0xf << MIDR_VARIANT_SHIFT) #define MIDR_VARIANT_MASK (0xf << MIDR_VARIANT_SHIFT)
char *get_cpuid_str(struct perf_pmu *pmu) static int _get_cpuid(char *buf, size_t sz, struct perf_cpu_map *cpus)
{ {
char *buf = NULL;
char path[PATH_MAX];
const char *sysfs = sysfs__mountpoint(); const char *sysfs = sysfs__mountpoint();
int cpu;
u64 midr = 0; u64 midr = 0;
struct perf_cpu_map *cpus; int cpu;
FILE *file;
if (!sysfs || !pmu || !pmu->cpus) if (!sysfs || sz < MIDR_SIZE)
return NULL; return EINVAL;
buf = malloc(MIDR_SIZE); cpus = perf_cpu_map__get(cpus);
if (!buf)
return NULL;
/* read midr from list of cpus mapped to this pmu */
cpus = perf_cpu_map__get(pmu->cpus);
for (cpu = 0; cpu < perf_cpu_map__nr(cpus); cpu++) { for (cpu = 0; cpu < perf_cpu_map__nr(cpus); cpu++) {
char path[PATH_MAX];
FILE *file;
scnprintf(path, PATH_MAX, "%s/devices/system/cpu/cpu%d"MIDR, scnprintf(path, PATH_MAX, "%s/devices/system/cpu/cpu%d"MIDR,
sysfs, cpus->map[cpu]); sysfs, cpus->map[cpu]);
...@@ -57,12 +54,48 @@ char *get_cpuid_str(struct perf_pmu *pmu) ...@@ -57,12 +54,48 @@ char *get_cpuid_str(struct perf_pmu *pmu)
break; break;
} }
if (!midr) { perf_cpu_map__put(cpus);
if (!midr)
return EINVAL;
return 0;
}
int get_cpuid(char *buf, size_t sz)
{
struct perf_cpu_map *cpus = perf_cpu_map__new(NULL);
int ret;
if (!cpus)
return EINVAL;
ret = _get_cpuid(buf, sz, cpus);
perf_cpu_map__put(cpus);
return ret;
}
char *get_cpuid_str(struct perf_pmu *pmu)
{
char *buf = NULL;
int res;
if (!pmu || !pmu->cpus)
return NULL;
buf = malloc(MIDR_SIZE);
if (!buf)
return NULL;
/* read midr from list of cpus mapped to this pmu */
res = _get_cpuid(buf, MIDR_SIZE, pmu->cpus);
if (res) {
pr_err("failed to get cpuid string for PMU %s\n", pmu->name); pr_err("failed to get cpuid string for PMU %s\n", pmu->name);
free(buf); free(buf);
buf = NULL; buf = NULL;
} }
perf_cpu_map__put(cpus);
return buf; return buf;
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment