[PATCH 4/7] tracing/function: Introduce persistent trace option

Steven Rostedt rostedt at goodmis.org
Wed Jun 27 12:40:12 UTC 2012


On Tue, 2012-06-26 at 16:23 -0700, Anton Vorontsov wrote:
>  
> @@ -219,6 +232,8 @@ static int func_set_flag(u32 old_flags, u32 bit, int set)
>  
>  		return 0;
>  	}
> +	if (bit == TRACE_FUNC_OPT_PSTORE)
> +		return 0;

Perhaps this should be a switch() statement, or at least put a
 } else if () {

As it can't be bit == TRACE_FUNC_OPT_STACK and bit ==
TRACE_FUNC_OPT_PSTORE.

-- Steve

>  
>  	return -EINVAL;
>  }





More information about the devel mailing list