Re: [PATCH 4/4] (RFC) X86: add IPI tracepoints

From: Steven Rostedt
Date: Mon Jul 21 2014 - 18:55:15 EST


On Fri, 18 Jul 2014 16:59:50 -0400 (EDT)
Nicolas Pitre <nicolas.pitre@xxxxxxxxxx> wrote:

> On Fri, 18 Jul 2014, Steven Rostedt wrote:
>
> > On Fri, 18 Jul 2014 01:18:55 -0400
> > Nicolas Pitre <nicolas.pitre@xxxxxxxxxx> wrote:
> >
> >
> > > diff --git a/arch/x86/kernel/smp.c b/arch/x86/kernel/smp.c
> > > index be8e1bde07..e154d176cf 100644
> > > --- a/arch/x86/kernel/smp.c
> > > +++ b/arch/x86/kernel/smp.c
> > > @@ -31,6 +31,12 @@
> > > #include <asm/apic.h>
> > > #include <asm/nmi.h>
> > > #include <asm/trace/irq_vectors.h>
> > > +
> > > +#define CREATE_TRACE_POINTS
> > > +#undef TRACE_INCLUDE_PATH
> > > +#undef TRACE_INCLUDE_FILE
> >
> > I'm curious to why you added the #undefs. I wouldn't think they were
> > needed.
>
> They are needed because asm/trace/irq_vectors.h included prior that
> point defines them for itself and that makes the inclusion of
> trace/events/ipi.h fail.
>

Bah, I tried to get rid of the need for those, but it seems that the
macro magic is getting a bit out of hand. I need a new macro magic
hat :-p

What you got here will have to do.


-- Steve
--
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/