tracing: Skip printing "OK" if failed to disable event
authorYuanhan Liu <yuanhan.liu@linux.intel.com>
Mon, 27 Aug 2012 07:13:45 +0000 (15:13 +0800)
committerSteven Rostedt <rostedt@goodmis.org>
Fri, 14 Sep 2012 02:52:07 +0000 (22:52 -0400)
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>
kernel/trace/trace_events.c

index 6825d83..bbb0e63 100644 (file)
@@ -1646,9 +1646,11 @@ static __init void event_trace_self_tests(void)
                event_test_stuff();
 
                ret = __ftrace_set_clr_event(NULL, system->name, NULL, 0);
-               if (WARN_ON_ONCE(ret))
+               if (WARN_ON_ONCE(ret)) {
                        pr_warning("error disabling system %s\n",
                                   system->name);
+                       continue;
+               }
 
                pr_cont("OK\n");
        }