Re: [PATCH net-next] lwtunnel: replace EXPORT_SYMBOL with EXPORT_SYMBOL_GPL
Hi Roopa, [auto build test ERROR on net-next/master] url: https://github.com/0day-ci/linux/commits/Roopa-Prabhu/lwtunnel-replace-EXPORT_SYMBOL-with-EXPORT_SYMBOL_GPL/20170805-212202 config: blackfin-allyesconfig (attached as .config) compiler: bfin-uclinux-gcc (GCC) 6.2.0 reproduce: wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree make.cross ARCH=blackfin All error/warnings (new ones prefixed by >>): >> net//core/lwtunnel.c:68:1: warning: data definition has no type or storage >> class EXPORT_SYMBOL_GPL_GPL(lwtunnel_state_alloc); ^ >> net//core/lwtunnel.c:68:1: error: type defaults to 'int' in declaration of >> 'EXPORT_SYMBOL_GPL_GPL' [-Werror=implicit-int] >> net//core/lwtunnel.c:68:1: warning: parameter names (without types) in >> function declaration cc1: some warnings being treated as errors vim +68 net//core/lwtunnel.c 59 60 struct lwtunnel_state *lwtunnel_state_alloc(int encap_len) 61 { 62 struct lwtunnel_state *lws; 63 64 lws = kzalloc(sizeof(*lws) + encap_len, GFP_ATOMIC); 65 66 return lws; 67 } > 68 EXPORT_SYMBOL_GPL_GPL(lwtunnel_state_alloc); 69 --- 0-DAY kernel test infrastructureOpen Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation .config.gz Description: application/gzip
Re: [PATCH net-next] lwtunnel: replace EXPORT_SYMBOL with EXPORT_SYMBOL_GPL
On Fri, Aug 4, 2017 at 3:25 PM, Andrew Lunnwrote: > On Fri, Aug 04, 2017 at 03:23:37PM -0700, Roopa Prabhu wrote: >> From: Roopa Prabhu >> >> Signed-off-by: Roopa Prabhu >> --- >> net/core/lwtunnel.c | 26 +- >> 1 file changed, 13 insertions(+), 13 deletions(-) >> >> diff --git a/net/core/lwtunnel.c b/net/core/lwtunnel.c >> index d9cb353..8693ff8 100644 >> --- a/net/core/lwtunnel.c >> +++ b/net/core/lwtunnel.c >> @@ -65,7 +65,7 @@ struct lwtunnel_state *lwtunnel_state_alloc(int encap_len) >> >> return lws; >> } >> -EXPORT_SYMBOL(lwtunnel_state_alloc); >> +EXPORT_SYMBOL_GPL_GPL(lwtunnel_state_alloc); > > Hi Roopa > > GPL_GPL? > oops, i had fixed it but sent the wrong version. thx
Re: [PATCH net-next] lwtunnel: replace EXPORT_SYMBOL with EXPORT_SYMBOL_GPL
On Fri, Aug 04, 2017 at 03:23:37PM -0700, Roopa Prabhu wrote: > From: Roopa Prabhu> > Signed-off-by: Roopa Prabhu > --- > net/core/lwtunnel.c | 26 +- > 1 file changed, 13 insertions(+), 13 deletions(-) > > diff --git a/net/core/lwtunnel.c b/net/core/lwtunnel.c > index d9cb353..8693ff8 100644 > --- a/net/core/lwtunnel.c > +++ b/net/core/lwtunnel.c > @@ -65,7 +65,7 @@ struct lwtunnel_state *lwtunnel_state_alloc(int encap_len) > > return lws; > } > -EXPORT_SYMBOL(lwtunnel_state_alloc); > +EXPORT_SYMBOL_GPL_GPL(lwtunnel_state_alloc); Hi Roopa GPL_GPL? Andrew