Re: [PATCH 01/11] [BUGFIX] tracing: Return 0 if event_enable_func succeeded

2013-05-09 Thread Steven Rostedt
On Thu, 2013-05-09 at 14:44 +0900, Masami Hiramatsu wrote: > Return 0 instead of the number of activated ftrace if > event_enable_func succeeded and return an error code if failed, > beacuse 0 is success code at caller (ftrace_regex_write). > > Without this fix, writing enable_event trigger on

Re: [PATCH 01/11] [BUGFIX] tracing: Return 0 if event_enable_func succeeded

2013-05-09 Thread Steven Rostedt
On Thu, 2013-05-09 at 10:31 -0400, Steven Rostedt wrote: > > diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c > > index 53582e9..44ac836 100644 > > --- a/kernel/trace/trace_events.c > > +++ b/kernel/trace/trace_events.c > > @@ -2061,8 +2061,11 @@ event_enable_func(struct

Re: [PATCH 01/11] [BUGFIX] tracing: Return 0 if event_enable_func succeeded

2013-05-09 Thread Steven Rostedt
On Thu, 2013-05-09 at 14:44 +0900, Masami Hiramatsu wrote: > Return 0 instead of the number of activated ftrace if > event_enable_func succeeded and return an error code if failed, > beacuse 0 is success code at caller (ftrace_regex_write). > > Without this fix, writing enable_event trigger on

Re: [PATCH 01/11] [BUGFIX] tracing: Return 0 if event_enable_func succeeded

2013-05-09 Thread Steven Rostedt
On Thu, 2013-05-09 at 14:44 +0900, Masami Hiramatsu wrote: Return 0 instead of the number of activated ftrace if event_enable_func succeeded and return an error code if failed, beacuse 0 is success code at caller (ftrace_regex_write). Without this fix, writing enable_event trigger on

Re: [PATCH 01/11] [BUGFIX] tracing: Return 0 if event_enable_func succeeded

2013-05-09 Thread Steven Rostedt
On Thu, 2013-05-09 at 10:31 -0400, Steven Rostedt wrote: diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c index 53582e9..44ac836 100644 --- a/kernel/trace/trace_events.c +++ b/kernel/trace/trace_events.c @@ -2061,8 +2061,11 @@ event_enable_func(struct ftrace_hash

Re: [PATCH 01/11] [BUGFIX] tracing: Return 0 if event_enable_func succeeded

2013-05-09 Thread Steven Rostedt
On Thu, 2013-05-09 at 14:44 +0900, Masami Hiramatsu wrote: Return 0 instead of the number of activated ftrace if event_enable_func succeeded and return an error code if failed, beacuse 0 is success code at caller (ftrace_regex_write). Without this fix, writing enable_event trigger on