Re: [PATCH 2/3] perf trace: Update eval_flag() flags array tomatch interrupt.h

From: Frederic Weisbecker
Date: Tue Oct 06 2009 - 11:28:02 EST


On Tue, Oct 06, 2009 at 05:22:58PM +0200, Frederic Weisbecker wrote:
> On Tue, Oct 06, 2009 at 11:21:06AM -0400, Christoph Hellwig wrote:
> > On Tue, Oct 06, 2009 at 05:19:00PM +0200, Frederic Weisbecker wrote:
> > > Yeah. We may want to do that by including trace/events/irq.h
> > > and then use the show_softirq_name() macro defined there.
> > >
> > > The rest of the header can be wrapped through no-op macros and
> > > stub includes.
> >
> > No, not at all. Performance tracing tools really should not be
> > dependent on the kernel source. This kind of creep is exactly what I
> > feared from putting the perf source in the kernel tree.
> >
>
> I see...
> Then the only solution I can imagine is to export a debugfs file
> in the tracing directory that provides this name resolution.
>

I mean, not something that we would check from an offline client,
but something we could include in the traces info while recording the
traces.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/