tracing: Pushdown the bkl tracepoints calls
authorFrederic Weisbecker <fweisbec@gmail.com>
Mon, 28 Sep 2009 15:12:49 +0000 (17:12 +0200)
committerFrederic Weisbecker <fweisbec@gmail.com>
Mon, 28 Sep 2009 16:00:48 +0000 (18:00 +0200)
commit925936ebf35a95c290e010b784c962164e6728f3
treee0bae3a2655f84dcf238c9acacd158e5ce422b32
parent3f6fe06dbf67b46d36fedec502300e04dffeb67a
tracing: Pushdown the bkl tracepoints calls

Currently we are calling the bkl tracepoint callbacks just before the
bkl lock/unlock operations, ie the tracepoint call is not inside a
lock_kernel() function but inside a lock_kernel() macro. Hence the
bkl trace event header must be included from smp_lock.h. This raises
some nasty circular header dependencies:

linux/smp_lock.h -> trace/events/bkl.h -> trace/define_trace.h
-> trace/ftrace.h -> linux/ftrace_event.h -> linux/hardirq.h
-> linux/smp_lock.h

This results in incomplete event declarations, spurious event
definitions and other kind of funny behaviours.

This is hardly fixable without ugly workarounds. So instead, we push
the file name, line number and function name as lock_kernel()
parameters, so that we only deal with the trace event header from
lib/kernel_lock.c

This adds two parameters to lock_kernel() and unlock_kernel() but
it should be fine wrt to performances because this pair dos not seem
to be called in fast paths.

Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Steven Rostedt <rostedt@goodmis.org>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Li Zefan <lizf@cn.fujitsu.com>
include/linux/smp_lock.h
lib/kernel_lock.c