No acutal case found. But logically, we should skip "OK" in case any error met.
Link: http://lkml.kernel.org/r/[email protected] Signed-off-by: Yuanhan Liu <[email protected]> Signed-off-by: Steven Rostedt <[email protected]> --- kernel/trace/trace_events.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c index 6825d83..bbb0e63 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) 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"); } -- 1.7.10.4
signature.asc
Description: This is a digitally signed message part
