From: "Steven Rostedt (Red Hat)" The irqsoff tracer can be called during some of the RCU unsafe regions. The proble is that some of the internal calls that it makes may also be traced. For example, it uses spin locks. But if the spin lock gets traced and the RCU unsafe checker runs, it will trigger that RCU is not safe to use. But the only reason a spin lock is being used in an RCU unsafe region is because the irqsoff trace uses it, and causes a false positive. Disable the unsafe RCU checker when irqsoff is enabled. Signed-off-by: Steven Rostedt --- kernel/trace/trace_irqsoff.c | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff --git a/kernel/trace/trace_irqsoff.c b/kernel/trace/trace_irqsoff.c index 2aefbee..62d603c 100644 --- a/kernel/trace/trace_irqsoff.c +++ b/kernel/trace/trace_irqsoff.c @@ -584,9 +584,14 @@ static int start_irqsoff_tracer(struct trace_array *tr, int graph) ret = register_irqsoff_function(graph, 0); - if (!ret && tracing_is_enabled()) + if (!ret && tracing_is_enabled()) { + /* + * irqsoff tracer can cause unsafe rcu checker + * to have false positives. + */ + ftrace_unsafe_rcu_checker_disable(); tracer_enabled = 1; - else + } else tracer_enabled = 0; return ret; @@ -594,6 +599,9 @@ static int start_irqsoff_tracer(struct trace_array *tr, int graph) static void stop_irqsoff_tracer(struct trace_array *tr, int graph) { + if (tracer_enabled) + ftrace_unsafe_rcu_checker_enable(); + tracer_enabled = 0; unregister_irqsoff_function(graph); @@ -630,11 +638,15 @@ static void irqsoff_tracer_reset(struct trace_array *tr) static void irqsoff_tracer_start(struct trace_array *tr) { + if (!tracer_enabled) + ftrace_unsafe_rcu_checker_disable(); tracer_enabled = 1; } static void irqsoff_tracer_stop(struct trace_array *tr) { + if (tracer_enabled) + ftrace_unsafe_rcu_checker_enable(); tracer_enabled = 0; } -- 1.7.10.4 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/