diff options
author | Harsh Prateek Bora <harsh@linux.vnet.ibm.com> | 2013-05-24 09:22:17 +0200 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2013-06-12 00:38:49 +0200 |
commit | 8092e808a31839c502a52d391b15f31c1d8764f5 (patch) | |
tree | 1ff012dac3fa01d07b6e9e51b026b877856bb176 /kernel/trace/trace_events.c | |
parent | tracing: Add function probe to trigger a ftrace dump of current CPU trace (diff) | |
download | linux-8092e808a31839c502a52d391b15f31c1d8764f5.tar.xz linux-8092e808a31839c502a52d391b15f31c1d8764f5.zip |
tracing/trivial: Consolidate error return condition
Consolidate the checks for !enabled and !param to return -EINVAL
in event_enable_func().
Link: http://lkml.kernel.org/r/1369380137-12452-1-git-send-email-harsh@linux.vnet.ibm.com
Signed-off-by: Harsh Prateek Bora <harsh@linux.vnet.ibm.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace/trace_events.c')
-rw-r--r-- | kernel/trace/trace_events.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c index 27963e2bf4bf..db086f172cf5 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, ":"); |