On Sat, 22 Oct 2016 15:08:52 -0400 Steven Rostedt <rost...@goodmis.org> wrote:
> On Sat, 22 Oct 2016 10:44:41 +1100 > Nicholas Piggin <npig...@gmail.com> wrote: > > > > #ifdef CONFIG_FUNCTION_GRAPH_TRACER > > > diff --git a/include/asm-generic/export.h b/include/asm-generic/export.h > > > index 43199a049da5..cb86e746865e 100644 > > > --- a/include/asm-generic/export.h > > > +++ b/include/asm-generic/export.h > > > @@ -90,5 +90,10 @@ > > > __EXPORT_SYMBOL(name, KSYM(name),) > > > #define EXPORT_DATA_SYMBOL_GPL(name) \ > > > __EXPORT_SYMBOL(name, KSYM(name),_gpl) > > > +/* > > > + * If "name" is a macro of a function and not a function itself, > > > + * it needs a second pass. > > > + */ > > > +#define EXPORT_MACRO_SYMBOL(x) EXPORT_SYMBOL(x) > > > > Seems okay, but what about just calling it EXPORT_SYMBOL? > > > Actually, this doesn't work. There's some magic going on it > Makefile.build in the scripts directory that causes this to fail. > > I have another patch I'll be sending on Monday that fixes this. Yes it's grepping for EXPORT_SYMBOL_* I think. If you need to create a new name, EXPORT_SYMBOL prefix would be preferred. But yeah if you make the standard macro do macro expansion, it should just work, no?