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