Commit 64c12e04 authored by Jason Baron's avatar Jason Baron Committed by Frederic Weisbecker

tracing: Add individual syscalls tracepoint id support

The current state of syscalls tracepoints generates only one event id
for every syscall events.

This patch associates an id with each syscall trace event, so that we
can identify each syscall trace event using the 'perf' tool.
Signed-off-by: default avatarJason Baron <jbaron@redhat.com>
Cc: Lai Jiangshan <laijs@cn.fujitsu.com>
Cc: Steven Rostedt <rostedt@goodmis.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca>
Cc: Jiaying Zhang <jiayingz@google.com>
Cc: Martin Bligh <mbligh@google.com>
Cc: Li Zefan <lizf@cn.fujitsu.com>
Cc: Masami Hiramatsu <mhiramat@redhat.com>
Signed-off-by: default avatarFrederic Weisbecker <fweisbec@gmail.com>
parent fb34a08c
...@@ -516,6 +516,16 @@ int syscall_name_to_nr(char *name) ...@@ -516,6 +516,16 @@ int syscall_name_to_nr(char *name)
return -1; return -1;
} }
void set_syscall_enter_id(int num, int id)
{
syscalls_metadata[num]->enter_id = id;
}
void set_syscall_exit_id(int num, int id)
{
syscalls_metadata[num]->exit_id = id;
}
static int __init arch_init_ftrace_syscalls(void) static int __init arch_init_ftrace_syscalls(void)
{ {
int i; int i;
......
...@@ -116,13 +116,20 @@ struct perf_counter_attr; ...@@ -116,13 +116,20 @@ struct perf_counter_attr;
#define SYSCALL_TRACE_ENTER_EVENT(sname) \ #define SYSCALL_TRACE_ENTER_EVENT(sname) \
static struct ftrace_event_call event_enter_##sname; \ static struct ftrace_event_call event_enter_##sname; \
struct trace_event enter_syscall_print_##sname = { \
.trace = print_syscall_enter, \
}; \
static int init_enter_##sname(void) \ static int init_enter_##sname(void) \
{ \ { \
int num; \ int num, id; \
num = syscall_name_to_nr("sys"#sname); \ num = syscall_name_to_nr("sys"#sname); \
if (num < 0) \ if (num < 0) \
return -ENOSYS; \ return -ENOSYS; \
register_ftrace_event(&event_syscall_enter); \ id = register_ftrace_event(&enter_syscall_print_##sname);\
if (!id) \
return -ENODEV; \
event_enter_##sname.id = id; \
set_syscall_enter_id(num, id); \
INIT_LIST_HEAD(&event_enter_##sname.fields); \ INIT_LIST_HEAD(&event_enter_##sname.fields); \
init_preds(&event_enter_##sname); \ init_preds(&event_enter_##sname); \
return 0; \ return 0; \
...@@ -142,13 +149,20 @@ struct perf_counter_attr; ...@@ -142,13 +149,20 @@ struct perf_counter_attr;
#define SYSCALL_TRACE_EXIT_EVENT(sname) \ #define SYSCALL_TRACE_EXIT_EVENT(sname) \
static struct ftrace_event_call event_exit_##sname; \ static struct ftrace_event_call event_exit_##sname; \
struct trace_event exit_syscall_print_##sname = { \
.trace = print_syscall_exit, \
}; \
static int init_exit_##sname(void) \ static int init_exit_##sname(void) \
{ \ { \
int num; \ int num, id; \
num = syscall_name_to_nr("sys"#sname); \ num = syscall_name_to_nr("sys"#sname); \
if (num < 0) \ if (num < 0) \
return -ENOSYS; \ return -ENOSYS; \
register_ftrace_event(&event_syscall_exit); \ id = register_ftrace_event(&exit_syscall_print_##sname);\
if (!id) \
return -ENODEV; \
event_exit_##sname.id = id; \
set_syscall_exit_id(num, id); \
INIT_LIST_HEAD(&event_exit_##sname.fields); \ INIT_LIST_HEAD(&event_exit_##sname.fields); \
init_preds(&event_exit_##sname); \ init_preds(&event_exit_##sname); \
return 0; \ return 0; \
......
...@@ -32,23 +32,31 @@ DECLARE_TRACE_WITH_CALLBACK(syscall_exit, ...@@ -32,23 +32,31 @@ DECLARE_TRACE_WITH_CALLBACK(syscall_exit,
* @nb_args: number of parameters it takes * @nb_args: number of parameters it takes
* @types: list of types as strings * @types: list of types as strings
* @args: list of args as strings (args[i] matches types[i]) * @args: list of args as strings (args[i] matches types[i])
* @enter_id: associated ftrace enter event id
* @exit_id: associated ftrace exit event id
*/ */
struct syscall_metadata { struct syscall_metadata {
const char *name; const char *name;
int nb_args; int nb_args;
const char **types; const char **types;
const char **args; const char **args;
int enter_id;
int exit_id;
}; };
#ifdef CONFIG_FTRACE_SYSCALLS #ifdef CONFIG_FTRACE_SYSCALLS
extern struct syscall_metadata *syscall_nr_to_meta(int nr); extern struct syscall_metadata *syscall_nr_to_meta(int nr);
extern int syscall_name_to_nr(char *name); extern int syscall_name_to_nr(char *name);
void set_syscall_enter_id(int num, int id);
void set_syscall_exit_id(int num, int id);
extern struct trace_event event_syscall_enter; extern struct trace_event event_syscall_enter;
extern struct trace_event event_syscall_exit; extern struct trace_event event_syscall_exit;
extern int reg_event_syscall_enter(void *ptr); extern int reg_event_syscall_enter(void *ptr);
extern void unreg_event_syscall_enter(void *ptr); extern void unreg_event_syscall_enter(void *ptr);
extern int reg_event_syscall_exit(void *ptr); extern int reg_event_syscall_exit(void *ptr);
extern void unreg_event_syscall_exit(void *ptr); extern void unreg_event_syscall_exit(void *ptr);
enum print_line_t print_syscall_enter(struct trace_iterator *iter, int flags);
enum print_line_t print_syscall_exit(struct trace_iterator *iter, int flags);
#endif #endif
#endif /* _TRACE_SYSCALL_H */ #endif /* _TRACE_SYSCALL_H */
...@@ -34,8 +34,6 @@ enum trace_type { ...@@ -34,8 +34,6 @@ enum trace_type {
TRACE_GRAPH_ENT, TRACE_GRAPH_ENT,
TRACE_USER_STACK, TRACE_USER_STACK,
TRACE_HW_BRANCHES, TRACE_HW_BRANCHES,
TRACE_SYSCALL_ENTER,
TRACE_SYSCALL_EXIT,
TRACE_KMEM_ALLOC, TRACE_KMEM_ALLOC,
TRACE_KMEM_FREE, TRACE_KMEM_FREE,
TRACE_POWER, TRACE_POWER,
...@@ -319,10 +317,6 @@ extern void __ftrace_bad_type(void); ...@@ -319,10 +317,6 @@ extern void __ftrace_bad_type(void);
TRACE_KMEM_ALLOC); \ TRACE_KMEM_ALLOC); \
IF_ASSIGN(var, ent, struct kmemtrace_free_entry, \ IF_ASSIGN(var, ent, struct kmemtrace_free_entry, \
TRACE_KMEM_FREE); \ TRACE_KMEM_FREE); \
IF_ASSIGN(var, ent, struct syscall_trace_enter, \
TRACE_SYSCALL_ENTER); \
IF_ASSIGN(var, ent, struct syscall_trace_exit, \
TRACE_SYSCALL_EXIT); \
__ftrace_bad_type(); \ __ftrace_bad_type(); \
} while (0) } while (0)
......
...@@ -36,14 +36,18 @@ print_syscall_enter(struct trace_iterator *iter, int flags) ...@@ -36,14 +36,18 @@ print_syscall_enter(struct trace_iterator *iter, int flags)
struct syscall_metadata *entry; struct syscall_metadata *entry;
int i, ret, syscall; int i, ret, syscall;
trace_assign_type(trace, ent); trace = (typeof(trace))ent;
syscall = trace->nr; syscall = trace->nr;
entry = syscall_nr_to_meta(syscall); entry = syscall_nr_to_meta(syscall);
if (!entry) if (!entry)
goto end; goto end;
if (entry->enter_id != ent->type) {
WARN_ON_ONCE(1);
goto end;
}
ret = trace_seq_printf(s, "%s(", entry->name); ret = trace_seq_printf(s, "%s(", entry->name);
if (!ret) if (!ret)
return TRACE_TYPE_PARTIAL_LINE; return TRACE_TYPE_PARTIAL_LINE;
...@@ -78,16 +82,20 @@ print_syscall_exit(struct trace_iterator *iter, int flags) ...@@ -78,16 +82,20 @@ print_syscall_exit(struct trace_iterator *iter, int flags)
struct syscall_metadata *entry; struct syscall_metadata *entry;
int ret; int ret;
trace_assign_type(trace, ent); trace = (typeof(trace))ent;
syscall = trace->nr; syscall = trace->nr;
entry = syscall_nr_to_meta(syscall); entry = syscall_nr_to_meta(syscall);
if (!entry) { if (!entry) {
trace_seq_printf(s, "\n"); trace_seq_printf(s, "\n");
return TRACE_TYPE_HANDLED; return TRACE_TYPE_HANDLED;
} }
if (entry->exit_id != ent->type) {
WARN_ON_ONCE(1);
return TRACE_TYPE_UNHANDLED;
}
ret = trace_seq_printf(s, "%s -> 0x%lx\n", entry->name, ret = trace_seq_printf(s, "%s -> 0x%lx\n", entry->name,
trace->ret); trace->ret);
if (!ret) if (!ret)
...@@ -114,7 +122,7 @@ void ftrace_syscall_enter(struct pt_regs *regs, long id) ...@@ -114,7 +122,7 @@ void ftrace_syscall_enter(struct pt_regs *regs, long id)
size = sizeof(*entry) + sizeof(unsigned long) * sys_data->nb_args; size = sizeof(*entry) + sizeof(unsigned long) * sys_data->nb_args;
event = trace_current_buffer_lock_reserve(TRACE_SYSCALL_ENTER, size, event = trace_current_buffer_lock_reserve(sys_data->enter_id, size,
0, 0); 0, 0);
if (!event) if (!event)
return; return;
...@@ -142,7 +150,7 @@ void ftrace_syscall_exit(struct pt_regs *regs, long ret) ...@@ -142,7 +150,7 @@ void ftrace_syscall_exit(struct pt_regs *regs, long ret)
if (!sys_data) if (!sys_data)
return; return;
event = trace_current_buffer_lock_reserve(TRACE_SYSCALL_EXIT, event = trace_current_buffer_lock_reserve(sys_data->exit_id,
sizeof(*entry), 0, 0); sizeof(*entry), 0, 0);
if (!event) if (!event)
return; return;
...@@ -239,10 +247,8 @@ void unreg_event_syscall_exit(void *ptr) ...@@ -239,10 +247,8 @@ void unreg_event_syscall_exit(void *ptr)
struct trace_event event_syscall_enter = { struct trace_event event_syscall_enter = {
.trace = print_syscall_enter, .trace = print_syscall_enter,
.type = TRACE_SYSCALL_ENTER
}; };
struct trace_event event_syscall_exit = { struct trace_event event_syscall_exit = {
.trace = print_syscall_exit, .trace = print_syscall_exit,
.type = TRACE_SYSCALL_EXIT
}; };
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