Messages in this thread | | | From | Joel Fernandes <> | Date | Fri, 10 Aug 2018 05:20:19 -0700 | Subject | Re: [for-next][PATCH 1/6] tracing: Partial revert of "tracing: Centralize preemptirq tracepoints and unify their usage" |
| |
On Thu, Aug 9, 2018 at 2:03 PM, Steven Rostedt <rostedt@goodmis.org> wrote: > From: "Steven Rostedt (VMware)" <rostedt@goodmis.org> > > Joel Fernandes created a nice patch that cleaned up the duplicate hooks used > by lockdep and irqsoff latency tracer. It made both use tracepoints. But it > caused lockdep to trigger several false positives. We have not figured out > why yet, but removing lockdep from using the trace event hooks and just call > its helper functions directly (like it use to), makes the problem go away. > > This is a partial revert of the clean up patch c3bc8fd637a9 ("tracing: > Centralize preemptirq tracepoints and unify their usage") that adds direct > calls for lockdep, but also keeps most of the clean up done to get rid of > the horrible preprocessor if statements. > > Link: http://lkml.kernel.org/r/20180806155058.5ee875f4@gandalf.local.home > > Cc: Joel Fernandes <joelaf@google.com> > Cc: Peter Zijlstra <peterz@infradead.org> > Fixes: c3bc8fd637a9 ("tracing: Centralize preemptirq tracepoints and unify their usage")
Reviewed-by: Joel Fernandes (Google) <joel@joelfernandes.org>
thanks,
- Joel
> Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org> > --- > include/linux/irqflags.h | 8 ++++++-- > include/linux/lockdep.h | 2 -- > init/main.c | 2 -- > kernel/locking/lockdep.c | 14 ++----------- > kernel/trace/trace_preemptirq.c | 36 ++++++++++++++++++--------------- > 5 files changed, 28 insertions(+), 34 deletions(-) > > diff --git a/include/linux/irqflags.h b/include/linux/irqflags.h
| |