* [PATCH] tracing: remove syscall_exit_fields
@ 2011-01-14 9:49 Lai Jiangshan
2011-01-14 17:49 ` Frederic Weisbecker
2011-01-15 14:55 ` [tip:perf/urgent] tracing: Remove syscall_exit_fields tip-bot for Lai Jiangshan
0 siblings, 2 replies; 4+ messages in thread
From: Lai Jiangshan @ 2011-01-14 9:49 UTC (permalink / raw)
To: Steven Rostedt, Ingo Molnar, Frederic Weisbecker, LKML
Use event_class_syscall_exit.fields instead.
Remove syscall_exit_fields.
Remove syscall_get_enter_fields()
Add event_class_syscall_exit.fields initialization.
Indentation has to be changed also, otherwise the line
".fields = LIST_HEAD_INIT(event_class_syscall_exit.fields),"
I added will exceed 80 characters.
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
---
diff --git a/kernel/trace/trace_syscalls.c b/kernel/trace/trace_syscalls.c
index bac752f..b706529 100644
--- a/kernel/trace/trace_syscalls.c
+++ b/kernel/trace/trace_syscalls.c
@@ -23,9 +23,6 @@ static int syscall_exit_register(struct ftrace_event_call *event,
static int syscall_enter_define_fields(struct ftrace_event_call *call);
static int syscall_exit_define_fields(struct ftrace_event_call *call);
-/* All syscall exit events have the same fields */
-static LIST_HEAD(syscall_exit_fields);
-
static struct list_head *
syscall_get_enter_fields(struct ftrace_event_call *call)
{
@@ -34,34 +31,28 @@ syscall_get_enter_fields(struct ftrace_event_call *call)
return &entry->enter_fields;
}
-static struct list_head *
-syscall_get_exit_fields(struct ftrace_event_call *call)
-{
- return &syscall_exit_fields;
-}
-
struct trace_event_functions enter_syscall_print_funcs = {
- .trace = print_syscall_enter,
+ .trace = print_syscall_enter,
};
struct trace_event_functions exit_syscall_print_funcs = {
- .trace = print_syscall_exit,
+ .trace = print_syscall_exit,
};
struct ftrace_event_class event_class_syscall_enter = {
- .system = "syscalls",
- .reg = syscall_enter_register,
- .define_fields = syscall_enter_define_fields,
- .get_fields = syscall_get_enter_fields,
- .raw_init = init_syscall_trace,
+ .system = "syscalls",
+ .reg = syscall_enter_register,
+ .define_fields = syscall_enter_define_fields,
+ .get_fields = syscall_get_enter_fields,
+ .raw_init = init_syscall_trace,
};
struct ftrace_event_class event_class_syscall_exit = {
- .system = "syscalls",
- .reg = syscall_exit_register,
- .define_fields = syscall_exit_define_fields,
- .get_fields = syscall_get_exit_fields,
- .raw_init = init_syscall_trace,
+ .system = "syscalls",
+ .reg = syscall_exit_register,
+ .define_fields = syscall_exit_define_fields,
+ .fields = LIST_HEAD_INIT(event_class_syscall_exit.fields),
+ .raw_init = init_syscall_trace,
};
extern unsigned long __start_syscalls_metadata[];
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] tracing: remove syscall_exit_fields
2011-01-14 9:49 [PATCH] tracing: remove syscall_exit_fields Lai Jiangshan
@ 2011-01-14 17:49 ` Frederic Weisbecker
2011-01-14 20:47 ` Steven Rostedt
2011-01-15 14:55 ` [tip:perf/urgent] tracing: Remove syscall_exit_fields tip-bot for Lai Jiangshan
1 sibling, 1 reply; 4+ messages in thread
From: Frederic Weisbecker @ 2011-01-14 17:49 UTC (permalink / raw)
To: Lai Jiangshan; +Cc: Steven Rostedt, Ingo Molnar, LKML
On Fri, Jan 14, 2011 at 05:49:02PM +0800, Lai Jiangshan wrote:
>
> Use event_class_syscall_exit.fields instead.
> Remove syscall_exit_fields.
> Remove syscall_get_enter_fields()
I assume you meant syscall_get_exit_fields()
> Add event_class_syscall_exit.fields initialization.
>
> Indentation has to be changed also, otherwise the line
> ".fields = LIST_HEAD_INIT(event_class_syscall_exit.fields),"
> I added will exceed 80 characters.
>
> Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
Nice!
Steve, if you don't mind I'll handle that one as I have another
patch for syscalls tracing.
Thanks.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] tracing: remove syscall_exit_fields
2011-01-14 17:49 ` Frederic Weisbecker
@ 2011-01-14 20:47 ` Steven Rostedt
0 siblings, 0 replies; 4+ messages in thread
From: Steven Rostedt @ 2011-01-14 20:47 UTC (permalink / raw)
To: Frederic Weisbecker; +Cc: Lai Jiangshan, Ingo Molnar, LKML
On Fri, 2011-01-14 at 18:49 +0100, Frederic Weisbecker wrote:
> On Fri, Jan 14, 2011 at 05:49:02PM +0800, Lai Jiangshan wrote:
> >
> > Use event_class_syscall_exit.fields instead.
> > Remove syscall_exit_fields.
> > Remove syscall_get_enter_fields()
>
> I assume you meant syscall_get_exit_fields()
>
> > Add event_class_syscall_exit.fields initialization.
> >
> > Indentation has to be changed also, otherwise the line
> > ".fields = LIST_HEAD_INIT(event_class_syscall_exit.fields),"
> > I added will exceed 80 characters.
> >
> > Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
>
> Nice!
>
> Steve, if you don't mind I'll handle that one as I have another
> patch for syscalls tracing.
Sure, go ahead.
-- Steve
^ permalink raw reply [flat|nested] 4+ messages in thread
* [tip:perf/urgent] tracing: Remove syscall_exit_fields
2011-01-14 9:49 [PATCH] tracing: remove syscall_exit_fields Lai Jiangshan
2011-01-14 17:49 ` Frederic Weisbecker
@ 2011-01-15 14:55 ` tip-bot for Lai Jiangshan
1 sibling, 0 replies; 4+ messages in thread
From: tip-bot for Lai Jiangshan @ 2011-01-15 14:55 UTC (permalink / raw)
To: linux-tip-commits
Cc: linux-kernel, hpa, mingo, fweisbec, rostedt, tglx, laijs
Commit-ID: 7f85803a26f304e698c673838aab06cc6d4d6e59
Gitweb: http://git.kernel.org/tip/7f85803a26f304e698c673838aab06cc6d4d6e59
Author: Lai Jiangshan <laijs@cn.fujitsu.com>
AuthorDate: Fri, 14 Jan 2011 17:49:02 +0800
Committer: Steven Rostedt <rostedt@goodmis.org>
CommitDate: Fri, 14 Jan 2011 18:06:35 -0500
tracing: Remove syscall_exit_fields
There is no need for syscall_exit_fields as the syscall
exit event class can already host the fields in its structure,
like most other trace events do by default. Use that
default behavior instead.
Following this scheme, we don't need anymore to override the
get_fields() callback of the syscall exit event class either.
Hence both syscall_exit_fields and syscall_get_exit_fields() can
be removed.
Also changed some indentation to keep the following under 80
characters:
".fields = LIST_HEAD_INIT(event_class_syscall_exit.fields),"
Acked-by: Frederic Weisbecker <fweisbec@gmail.com>
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D301C0E.8090408@cn.fujitsu.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
---
kernel/trace/trace_syscalls.c | 33 ++++++++++++---------------------
1 files changed, 12 insertions(+), 21 deletions(-)
diff --git a/kernel/trace/trace_syscalls.c b/kernel/trace/trace_syscalls.c
index bac752f..b706529 100644
--- a/kernel/trace/trace_syscalls.c
+++ b/kernel/trace/trace_syscalls.c
@@ -23,9 +23,6 @@ static int syscall_exit_register(struct ftrace_event_call *event,
static int syscall_enter_define_fields(struct ftrace_event_call *call);
static int syscall_exit_define_fields(struct ftrace_event_call *call);
-/* All syscall exit events have the same fields */
-static LIST_HEAD(syscall_exit_fields);
-
static struct list_head *
syscall_get_enter_fields(struct ftrace_event_call *call)
{
@@ -34,34 +31,28 @@ syscall_get_enter_fields(struct ftrace_event_call *call)
return &entry->enter_fields;
}
-static struct list_head *
-syscall_get_exit_fields(struct ftrace_event_call *call)
-{
- return &syscall_exit_fields;
-}
-
struct trace_event_functions enter_syscall_print_funcs = {
- .trace = print_syscall_enter,
+ .trace = print_syscall_enter,
};
struct trace_event_functions exit_syscall_print_funcs = {
- .trace = print_syscall_exit,
+ .trace = print_syscall_exit,
};
struct ftrace_event_class event_class_syscall_enter = {
- .system = "syscalls",
- .reg = syscall_enter_register,
- .define_fields = syscall_enter_define_fields,
- .get_fields = syscall_get_enter_fields,
- .raw_init = init_syscall_trace,
+ .system = "syscalls",
+ .reg = syscall_enter_register,
+ .define_fields = syscall_enter_define_fields,
+ .get_fields = syscall_get_enter_fields,
+ .raw_init = init_syscall_trace,
};
struct ftrace_event_class event_class_syscall_exit = {
- .system = "syscalls",
- .reg = syscall_exit_register,
- .define_fields = syscall_exit_define_fields,
- .get_fields = syscall_get_exit_fields,
- .raw_init = init_syscall_trace,
+ .system = "syscalls",
+ .reg = syscall_exit_register,
+ .define_fields = syscall_exit_define_fields,
+ .fields = LIST_HEAD_INIT(event_class_syscall_exit.fields),
+ .raw_init = init_syscall_trace,
};
extern unsigned long __start_syscalls_metadata[];
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2011-01-15 14:55 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-01-14 9:49 [PATCH] tracing: remove syscall_exit_fields Lai Jiangshan
2011-01-14 17:49 ` Frederic Weisbecker
2011-01-14 20:47 ` Steven Rostedt
2011-01-15 14:55 ` [tip:perf/urgent] tracing: Remove syscall_exit_fields tip-bot for Lai Jiangshan
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.