diff options
author | Yuanhan Liu <yuanhan.liu@linux.intel.com> | 2012-08-27 03:13:45 -0400 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2012-09-13 22:52:07 -0400 |
commit | 76bab1b78ab6f25d5f74165f94526c25fc93d984 (patch) | |
tree | d6640f7bffa1fdeea0403b1c793a82fc880b9faa /kernel/trace | |
parent | d57c5d51a30152f3175d2344cb6395f08bf8ee0c (diff) |
tracing: Skip printing "OK" if failed to disable event
No acutal case found. But logically, we should skip "OK" in case any
error met.
Link: http://lkml.kernel.org/r/1346051625-25231-1-git-send-email-yuanhan.liu@linux.intel.com
Signed-off-by: Yuanhan Liu <yuanhan.liu@linux.intel.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace')
-rw-r--r-- | kernel/trace/trace_events.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c index 6825d833a257..bbb0e63d78e9 100644 --- a/kernel/trace/trace_events.c +++ b/kernel/trace/trace_events.c | |||
@@ -1646,9 +1646,11 @@ static __init void event_trace_self_tests(void) | |||
1646 | event_test_stuff(); | 1646 | event_test_stuff(); |
1647 | 1647 | ||
1648 | ret = __ftrace_set_clr_event(NULL, system->name, NULL, 0); | 1648 | ret = __ftrace_set_clr_event(NULL, system->name, NULL, 0); |
1649 | if (WARN_ON_ONCE(ret)) | 1649 | if (WARN_ON_ONCE(ret)) { |
1650 | pr_warning("error disabling system %s\n", | 1650 | pr_warning("error disabling system %s\n", |
1651 | system->name); | 1651 | system->name); |
1652 | continue; | ||
1653 | } | ||
1652 | 1654 | ||
1653 | pr_cont("OK\n"); | 1655 | pr_cont("OK\n"); |
1654 | } | 1656 | } |