Messages in this thread | | | Subject | Re: [patch 01/15] Kernel Tracepoints | From | Peter Zijlstra <> | Date | Tue, 15 Jul 2008 17:13:49 +0200 |
| |
On Tue, 2008-07-15 at 10:46 -0400, Mathieu Desnoyers wrote:
> Talking about headers, I have noticed that placing headers with the code > may not be as clean as I would hope. For instance, the kernel/irq-trace.h > header, when included from kernel/irq/handle.c, has to be included with: > > #include "../irq-trace.h" > > Which is not _that_ bad, but we we want to instrument the irq handler > found in arch/x86/kernel/cpu/mcheck/mce_intel_64.c, including > #include "../../../../../kernel/irq-trace.h" makes me go "yeeeek!" > > How about creating include/trace/irq.h and friends ?
Might as well.. anybody else got opinions?
| |