libperf: Move fd array from perf's evsel to lobperf's perf_evsel class
Move the fd array from perf's evsel to libperf's perf_evsel class. Signed-off-by: Jiri Olsa <jolsa@kernel.org> Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com> Cc: Alexey Budankov <alexey.budankov@linux.intel.com> Cc: Andi Kleen <ak@linux.intel.com> Cc: Michael Petlan <mpetlan@redhat.com> Cc: Namhyung Kim <namhyung@kernel.org> Cc: Peter Zijlstra <peterz@infradead.org> Link: http://lkml.kernel.org/r/20190721112506.12306-59-jolsa@kernel.org Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
This commit is contained in:
parent
4b247fa731
commit
9dfcb75990
@ -14,6 +14,7 @@ struct perf_evsel {
|
||||
struct perf_cpu_map *cpus;
|
||||
struct perf_cpu_map *own_cpus;
|
||||
struct perf_thread_map *threads;
|
||||
struct xyarray *fd;
|
||||
};
|
||||
|
||||
#endif /* __LIBPERF_INTERNAL_EVSEL_H */
|
||||
|
@ -1403,7 +1403,7 @@ static int
|
||||
apply_config_evsel_for_key(const char *name, int map_fd, void *pkey,
|
||||
struct evsel *evsel)
|
||||
{
|
||||
struct xyarray *xy = evsel->fd;
|
||||
struct xyarray *xy = evsel->core.fd;
|
||||
struct perf_event_attr *attr;
|
||||
unsigned int key, events;
|
||||
bool check_pass = false;
|
||||
|
@ -40,7 +40,7 @@
|
||||
int sigqueue(pid_t pid, int sig, const union sigval value);
|
||||
#endif
|
||||
|
||||
#define FD(e, x, y) (*(int *)xyarray__entry(e->fd, x, y))
|
||||
#define FD(e, x, y) (*(int *)xyarray__entry(e->core.fd, x, y))
|
||||
#define SID(e, x, y) xyarray__entry(e->sample_id, x, y)
|
||||
|
||||
void evlist__init(struct evlist *evlist, struct perf_cpu_map *cpus,
|
||||
@ -321,7 +321,7 @@ void evlist__disable(struct evlist *evlist)
|
||||
struct evsel *pos;
|
||||
|
||||
evlist__for_each_entry(evlist, pos) {
|
||||
if (pos->disabled || !perf_evsel__is_group_leader(pos) || !pos->fd)
|
||||
if (pos->disabled || !perf_evsel__is_group_leader(pos) || !pos->core.fd)
|
||||
continue;
|
||||
evsel__disable(pos);
|
||||
}
|
||||
@ -334,7 +334,7 @@ void evlist__enable(struct evlist *evlist)
|
||||
struct evsel *pos;
|
||||
|
||||
evlist__for_each_entry(evlist, pos) {
|
||||
if (!perf_evsel__is_group_leader(pos) || !pos->fd)
|
||||
if (!perf_evsel__is_group_leader(pos) || !pos->core.fd)
|
||||
continue;
|
||||
evsel__enable(pos);
|
||||
}
|
||||
@ -353,7 +353,7 @@ static int perf_evlist__enable_event_cpu(struct evlist *evlist,
|
||||
int thread;
|
||||
int nr_threads = perf_evlist__nr_threads(evlist, evsel);
|
||||
|
||||
if (!evsel->fd)
|
||||
if (!evsel->core.fd)
|
||||
return -EINVAL;
|
||||
|
||||
for (thread = 0; thread < nr_threads; thread++) {
|
||||
@ -371,7 +371,7 @@ static int perf_evlist__enable_event_thread(struct evlist *evlist,
|
||||
int cpu;
|
||||
int nr_cpus = cpu_map__nr(evlist->core.cpus);
|
||||
|
||||
if (!evsel->fd)
|
||||
if (!evsel->core.fd)
|
||||
return -EINVAL;
|
||||
|
||||
for (cpu = 0; cpu < nr_cpus; cpu++) {
|
||||
|
@ -90,7 +90,7 @@ set_methods:
|
||||
return 0;
|
||||
}
|
||||
|
||||
#define FD(e, x, y) (*(int *)xyarray__entry(e->fd, x, y))
|
||||
#define FD(e, x, y) (*(int *)xyarray__entry(e->core.fd, x, y))
|
||||
|
||||
int __perf_evsel__sample_size(u64 sample_type)
|
||||
{
|
||||
@ -1155,9 +1155,9 @@ void perf_evsel__config(struct evsel *evsel, struct record_opts *opts,
|
||||
|
||||
static int perf_evsel__alloc_fd(struct evsel *evsel, int ncpus, int nthreads)
|
||||
{
|
||||
evsel->fd = xyarray__new(ncpus, nthreads, sizeof(int));
|
||||
evsel->core.fd = xyarray__new(ncpus, nthreads, sizeof(int));
|
||||
|
||||
if (evsel->fd) {
|
||||
if (evsel->core.fd) {
|
||||
int cpu, thread;
|
||||
for (cpu = 0; cpu < ncpus; cpu++) {
|
||||
for (thread = 0; thread < nthreads; thread++) {
|
||||
@ -1166,7 +1166,7 @@ static int perf_evsel__alloc_fd(struct evsel *evsel, int ncpus, int nthreads)
|
||||
}
|
||||
}
|
||||
|
||||
return evsel->fd != NULL ? 0 : -ENOMEM;
|
||||
return evsel->core.fd != NULL ? 0 : -ENOMEM;
|
||||
}
|
||||
|
||||
static int perf_evsel__run_ioctl(struct evsel *evsel,
|
||||
@ -1174,8 +1174,8 @@ static int perf_evsel__run_ioctl(struct evsel *evsel,
|
||||
{
|
||||
int cpu, thread;
|
||||
|
||||
for (cpu = 0; cpu < xyarray__max_x(evsel->fd); cpu++) {
|
||||
for (thread = 0; thread < xyarray__max_y(evsel->fd); thread++) {
|
||||
for (cpu = 0; cpu < xyarray__max_x(evsel->core.fd); cpu++) {
|
||||
for (thread = 0; thread < xyarray__max_y(evsel->core.fd); thread++) {
|
||||
int fd = FD(evsel, cpu, thread),
|
||||
err = ioctl(fd, ioc, arg);
|
||||
|
||||
@ -1283,8 +1283,8 @@ int perf_evsel__alloc_id(struct evsel *evsel, int ncpus, int nthreads)
|
||||
|
||||
static void perf_evsel__free_fd(struct evsel *evsel)
|
||||
{
|
||||
xyarray__delete(evsel->fd);
|
||||
evsel->fd = NULL;
|
||||
xyarray__delete(evsel->core.fd);
|
||||
evsel->core.fd = NULL;
|
||||
}
|
||||
|
||||
static void perf_evsel__free_id(struct evsel *evsel)
|
||||
@ -1309,8 +1309,8 @@ void perf_evsel__close_fd(struct evsel *evsel)
|
||||
{
|
||||
int cpu, thread;
|
||||
|
||||
for (cpu = 0; cpu < xyarray__max_x(evsel->fd); cpu++)
|
||||
for (thread = 0; thread < xyarray__max_y(evsel->fd); ++thread) {
|
||||
for (cpu = 0; cpu < xyarray__max_x(evsel->core.fd); cpu++)
|
||||
for (thread = 0; thread < xyarray__max_y(evsel->core.fd); ++thread) {
|
||||
close(FD(evsel, cpu, thread));
|
||||
FD(evsel, cpu, thread) = -1;
|
||||
}
|
||||
@ -1555,7 +1555,7 @@ static int get_group_fd(struct evsel *evsel, int cpu, int thread)
|
||||
* Leader must be already processed/open,
|
||||
* if not it's a bug.
|
||||
*/
|
||||
BUG_ON(!leader->fd);
|
||||
BUG_ON(!leader->core.fd);
|
||||
|
||||
fd = FD(leader, cpu, thread);
|
||||
BUG_ON(fd == -1);
|
||||
@ -1865,7 +1865,7 @@ int evsel__open(struct evsel *evsel, struct perf_cpu_map *cpus,
|
||||
else
|
||||
nthreads = threads->nr;
|
||||
|
||||
if (evsel->fd == NULL &&
|
||||
if (evsel->core.fd == NULL &&
|
||||
perf_evsel__alloc_fd(evsel, cpus->nr, nthreads) < 0)
|
||||
return -ENOMEM;
|
||||
|
||||
@ -2075,7 +2075,7 @@ out_close:
|
||||
|
||||
void perf_evsel__close(struct evsel *evsel)
|
||||
{
|
||||
if (evsel->fd == NULL)
|
||||
if (evsel->core.fd == NULL)
|
||||
return;
|
||||
|
||||
perf_evsel__close_fd(evsel);
|
||||
@ -3048,8 +3048,8 @@ static int store_evsel_ids(struct evsel *evsel, struct evlist *evlist)
|
||||
{
|
||||
int cpu, thread;
|
||||
|
||||
for (cpu = 0; cpu < xyarray__max_x(evsel->fd); cpu++) {
|
||||
for (thread = 0; thread < xyarray__max_y(evsel->fd);
|
||||
for (cpu = 0; cpu < xyarray__max_x(evsel->core.fd); cpu++) {
|
||||
for (thread = 0; thread < xyarray__max_y(evsel->core.fd);
|
||||
thread++) {
|
||||
int fd = FD(evsel, cpu, thread);
|
||||
|
||||
|
@ -104,7 +104,6 @@ struct evsel {
|
||||
struct perf_evsel core;
|
||||
struct evlist *evlist;
|
||||
char *filter;
|
||||
struct xyarray *fd;
|
||||
struct xyarray *sample_id;
|
||||
u64 *id;
|
||||
struct perf_counts *counts;
|
||||
|
Loading…
Reference in New Issue
Block a user