Skip to content

Commit

Permalink
perf trace: Move syscall table id <-> name routines to separate class
Browse files Browse the repository at this point in the history
We're using libaudit for doing name to id and id to syscall name
translations, but that makes 'perf trace' to have to wait for newer
libaudit versions supporting recently added syscalls, such as
"userfaultfd" at the time of this changeset.

We have all the information right there, in the kernel sources, so move
this code to a separate place, wrapped behind functions that will
progressively use the kernel source files to extract the syscall table
for use in 'perf trace'.

Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Wang Nan <wangnan0@huawei.com>
Link: http://lkml.kernel.org/n/tip-i38opd09ow25mmyrvfwnbvkj@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
  • Loading branch information
Arnaldo Carvalho de Melo committed Apr 8, 2016
1 parent ba2f22c commit fd0db10
Show file tree
Hide file tree
Showing 4 changed files with 71 additions and 13 deletions.
24 changes: 11 additions & 13 deletions tools/perf/builtin-trace.c
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,9 @@
#include "trace-event.h"
#include "util/parse-events.h"
#include "util/bpf-loader.h"
#include "syscalltbl.h"

#include <libaudit.h>
#include <libaudit.h> /* FIXME: Still needed for audit_errno_to_name */
#include <stdlib.h>
#include <sys/mman.h>
#include <linux/futex.h>
Expand Down Expand Up @@ -114,10 +115,7 @@

struct trace {
struct perf_tool tool;
struct {
int machine;
int open_id;
} audit;
struct syscalltbl *sctbl;
struct {
int max;
struct syscall *table;
Expand Down Expand Up @@ -163,6 +161,7 @@ struct trace {
bool force;
bool vfs_getname;
int trace_pgfaults;
int open_id;
};

struct tp_field {
Expand Down Expand Up @@ -1780,7 +1779,7 @@ static int trace__read_syscall_info(struct trace *trace, int id)
{
char tp_name[128];
struct syscall *sc;
const char *name = audit_syscall_to_name(id, trace->audit.machine);
const char *name = syscalltbl__name(trace->sctbl, id);

if (name == NULL)
return -1;
Expand Down Expand Up @@ -1855,7 +1854,7 @@ static int trace__validate_ev_qualifier(struct trace *trace)

strlist__for_each(pos, trace->ev_qualifier) {
const char *sc = pos->s;
int id = audit_name_to_syscall(sc, trace->audit.machine);
int id = syscalltbl__id(trace->sctbl, sc);

if (id < 0) {
if (err == 0) {
Expand Down Expand Up @@ -2137,7 +2136,7 @@ static int trace__sys_exit(struct trace *trace, struct perf_evsel *evsel,

ret = perf_evsel__sc_tp_uint(evsel, ret, sample);

if (id == trace->audit.open_id && ret >= 0 && ttrace->filename.pending_open) {
if (id == trace->open_id && ret >= 0 && ttrace->filename.pending_open) {
trace__set_fd_pathname(thread, ret, ttrace->filename.name);
ttrace->filename.pending_open = false;
++trace->stats.vfs_getname;
Expand Down Expand Up @@ -3189,10 +3188,6 @@ int cmd_trace(int argc, const char **argv, const char *prefix __maybe_unused)
NULL
};
struct trace trace = {
.audit = {
.machine = audit_detect_machine(),
.open_id = audit_name_to_syscall("open", trace.audit.machine),
},
.syscalls = {
. max = -1,
},
Expand Down Expand Up @@ -3267,8 +3262,9 @@ int cmd_trace(int argc, const char **argv, const char *prefix __maybe_unused)
signal(SIGFPE, sighandler_dump_stack);

trace.evlist = perf_evlist__new();
trace.sctbl = syscalltbl__new();

if (trace.evlist == NULL) {
if (trace.evlist == NULL || trace.sctbl == NULL) {
pr_err("Not enough memory to run!\n");
err = -ENOMEM;
goto out;
Expand Down Expand Up @@ -3306,6 +3302,8 @@ int cmd_trace(int argc, const char **argv, const char *prefix __maybe_unused)
}
}

trace.open_id = syscalltbl__id(trace.sctbl, "open");

if (ev_qualifier_str != NULL) {
const char *s = ev_qualifier_str;
struct strlist_config slist_config = {
Expand Down
1 change: 1 addition & 0 deletions tools/perf/util/Build
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ libperf-y += machine.o
libperf-y += map.o
libperf-y += pstack.o
libperf-y += session.o
libperf-$(CONFIG_AUDIT) += syscalltbl.o
libperf-y += ordered-events.o
libperf-y += comm.o
libperf-y += thread.o
Expand Down
43 changes: 43 additions & 0 deletions tools/perf/util/syscalltbl.c
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
/*
* System call table mapper
*
* (C) 2016 Arnaldo Carvalho de Melo <acme@redhat.com>
*
* This program is free software; you can redistribute it and/or modify it
* under the terms and conditions of the GNU General Public License,
* version 2, as published by the Free Software Foundation.
*
* This program is distributed in the hope it will be useful, but WITHOUT
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for
* more details.
*/

#include "syscalltbl.h"
#include <string.h>
#include <libaudit.h>


struct syscalltbl *syscalltbl__new(void)
{
struct syscalltbl *tbl = malloc(sizeof(*tbl));
if (tbl) {
tbl->audit_machine = audit_detect_machine();
}
return tbl;
}

void syscalltbl__delete(struct syscalltbl *tbl)
{
free(tbl);
}

const char *syscalltbl__name(const struct syscalltbl *tbl, int id)
{
return audit_syscall_to_name(id, tbl->audit_machine);
}

int syscalltbl__id(struct syscalltbl *tbl, const char *name)
{
return audit_name_to_syscall(name, tbl->audit_machine);
}
16 changes: 16 additions & 0 deletions tools/perf/util/syscalltbl.h
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
#ifndef __PERF_SYSCALLTBL_H
#define __PERF_SYSCALLTBL_H

struct syscalltbl {
union {
int audit_machine;
};
};

struct syscalltbl *syscalltbl__new(void);
void syscalltbl__delete(struct syscalltbl *tbl);

const char *syscalltbl__name(const struct syscalltbl *tbl, int id);
int syscalltbl__id(struct syscalltbl *tbl, const char *name);

#endif /* __PERF_SYSCALLTBL_H */

0 comments on commit fd0db10

Please sign in to comment.