Re: [PATCH] tracing: code cleanup

[Steven Rostedt]

On Fri, 2013-05-24 at 12:52 +0530, Harsh Prateek Bora wrote:
> Signed-off-by: Harsh Prateek Bora <ha...@linux.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(&param, ":");


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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/