Re: [PATCH 19/20 -v5] Trace irq disabled critical timings

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Frank Ch. Eigler <fche@...>
Cc: Peter Zijlstra <a.p.zijlstra@...>, LKML <linux-kernel@...>, Ingo Molnar <mingo@...>, Linus Torvalds <torvalds@...>, Andrew Morton <akpm@...>, Christoph Hellwig <hch@...>, Mathieu Desnoyers <mathieu.desnoyers@...>, Gregory Haskins <ghaskins@...>, Arnaldo Carvalho de Melo <acme@...>, Thomas Gleixner <tglx@...>, Tim Bird <tim.bird@...>, Sam Ravnborg <sam@...>, Jan Kiszka <jan.kiszka@...>, John Stultz <johnstul@...>, Arjan van de Ven <arjan@...>, Steven Rostedt <srostedt@...>
Date: Wednesday, January 23, 2008 - 2:48 pm

On Wed, 23 Jan 2008, Frank Ch. Eigler wrote:

So basically just having the marker implementation there to let all
tracers access to irqs enabled/disabled.

One problem is that the hooks are alse in entry.S, so the tracing ends up
being architecture dependent. One solution is to have a "generic" hook for
trace_hardirqs_on/off (kind of like what I did here) and add a marker
inside that. The hooks in entry.S can call the generic function and have
the marker call the registered probes, be that lockdep, latencytap or
latency-trace.

Have the generic hook only be compiled in when one of the tracers or
lockdep is configured on and they select it.

-- Steve

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 19/20 -v5] Trace irq disabled critical timings, Steven Rostedt, (Wed Jan 23, 12:02 pm)
Re: [PATCH 19/20 -v5] Trace irq disabled critical timings, Steven Rostedt, (Wed Jan 23, 12:53 pm)
Re: [PATCH 19/20 -v5] Trace irq disabled critical timings, Steven Rostedt, (Wed Jan 23, 1:11 pm)
Re: [PATCH 19/20 -v5] Trace irq disabled critical timings, Frank Ch. Eigler, (Wed Jan 23, 1:27 pm)
Re: [PATCH 19/20 -v5] Trace irq disabled critical timings, Peter Zijlstra, (Wed Jan 23, 1:58 pm)
Re: [PATCH 19/20 -v5] Trace irq disabled critical timings, Mathieu Desnoyers, (Wed Jan 23, 5:29 pm)
Re: [PATCH 19/20 -v5] Trace irq disabled critical timings, Frank Ch. Eigler, (Wed Jan 23, 2:25 pm)
Re: [PATCH 19/20 -v5] Trace irq disabled critical timings, Steven Rostedt, (Wed Jan 23, 2:48 pm)
Re: [PATCH 19/20 -v5] Trace irq disabled critical timings, Steven Rostedt, (Wed Jan 23, 1:45 pm)
Re: [PATCH 19/20 -v5] Trace irq disabled critical timings, Mathieu Desnoyers, (Wed Jan 23, 1:54 pm)