Re: [RFC][PATCH] Add EXPORT_MACRO_SYMBOL() for asm

2016-10-24 Thread Steven Rostedt
On Mon, 24 Oct 2016 12:25:45 +1100 Nicholas Piggin wrote: > 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? Unfortunately, that's

Re: [RFC][PATCH] Add EXPORT_MACRO_SYMBOL() for asm

2016-10-23 Thread Nicholas Piggin
On Sat, 22 Oct 2016 15:08:52 -0400 Steven Rostedt wrote: > On Sat, 22 Oct 2016 10:44:41 +1100 > Nicholas Piggin wrote: > > > > #ifdef CONFIG_FUNCTION_GRAPH_TRACER > > > diff --git a/include/asm-generic/export.h b/include/asm-generic/export.h > > > index 43199a049da5..cb86e746865e 100644 > > >

Re: [RFC][PATCH] Add EXPORT_MACRO_SYMBOL() for asm

2016-10-22 Thread Steven Rostedt
On Sat, 22 Oct 2016 10:44:41 +1100 Nicholas Piggin 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

Re: [RFC][PATCH] Add EXPORT_MACRO_SYMBOL() for asm

2016-10-21 Thread Nicholas Piggin
On Fri, 21 Oct 2016 12:17:59 -0400 Steven Rostedt wrote: > Commit 784d5699eddc5 ("x86: move exports to actual definitions") removed the > EXPORT_SYMBOL(__fentry__) and EXPORT_SYMBOL(mcount) from x8664_ksyms_64.c, > and added EXPORT_SYMBOL(function_hook) in mcount_64.S instead. The problem > is th

[RFC][PATCH] Add EXPORT_MACRO_SYMBOL() for asm

2016-10-21 Thread Steven Rostedt
Commit 784d5699eddc5 ("x86: move exports to actual definitions") removed the EXPORT_SYMBOL(__fentry__) and EXPORT_SYMBOL(mcount) from x8664_ksyms_64.c, and added EXPORT_SYMBOL(function_hook) in mcount_64.S instead. The problem is that function_hook isn't a function at all, but a macro that is defi