[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <1369401986.6828.221.camel@gandalf.local.home>
Date: Fri, 24 May 2013 09:26:26 -0400
From: Steven Rostedt <rostedt@...dmis.org>
To: Harsh Prateek Bora <harsh@...ux.vnet.ibm.com>
Cc: linux-kernel@...r.kernel.org, srikar@...ux.vnet.ibm.com
Subject: Re: [PATCH] tracing: code cleanup
On Fri, 2013-05-24 at 12:52 +0530, Harsh Prateek Bora wrote:
> Signed-off-by: Harsh Prateek Bora <harsh@...ux.vnet.ibm.com>
> ---
> kernel/trace/trace_events.c | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c
> index 7a0cf68..371c7b0 100644
> --- a/kernel/trace/trace_events.c
> +++ b/kernel/trace/trace_events.c
> @@ -2011,10 +2011,7 @@ event_enable_func(struct ftrace_hash *hash,
> int ret;
>
> /* hash funcs only work with set_ftrace_filter */
> - if (!enabled)
> - return -EINVAL;
> -
> - if (!param)
> + if (!enabled || !param)
> return -EINVAL;
>
> system = strsep(¶m, ":");
Thanks, I added it to my 3.11 queue.
-- Steve
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists