* Re: [PATCH -tip] in6.h: extern's make no sense in userspace
2009-01-13 7:15 ` Sam Ravnborg
@ 2009-01-13 7:20 ` Jaswinder Singh Rajput
2009-01-13 7:22 ` David Miller
1 sibling, 0 replies; 4+ messages in thread
From: Jaswinder Singh Rajput @ 2009-01-13 7:20 UTC (permalink / raw)
To: Sam Ravnborg; +Cc: Ingo Molnar, harvey.harrison, gorcunov, roque, LKML, netdev
On Tue, 2009-01-13 at 08:15 +0100, Sam Ravnborg wrote:
> On Tue, Jan 13, 2009 at 12:27:39PM +0530, Jaswinder Singh Rajput wrote:
> > Impact: fix 2 make headers_check warnings:
> >
> > usr/include/linux/in6.h:47: extern's make no sense in userspace
> > usr/include/linux/in6.h:49: extern's make no sense in userspace
> >
> > Signed-off-by: Jaswinder Singh Rajput <jaswinderrajput@gmail.com>
> > ---
> > include/linux/in6.h | 2 +-
> > 1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/include/linux/in6.h b/include/linux/in6.h
> > index bc49204..718bf21 100644
> > --- a/include/linux/in6.h
> > +++ b/include/linux/in6.h
> > @@ -44,11 +44,11 @@ struct in6_addr
> > * NOTE: Be aware the IN6ADDR_* constants and in6addr_* externals are defined
> > * in network byte order, not in host byte order as are the IPv4 equivalents
> > */
> > +#ifdef __KERNEL__
> > extern const struct in6_addr in6addr_any;
> > #define IN6ADDR_ANY_INIT { { { 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0 } } }
> > extern const struct in6_addr in6addr_loopback;
> > #define IN6ADDR_LOOPBACK_INIT { { { 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,1 } } }
> > -#ifdef __KERNEL__
>
> It looks wrong what IN6ADDR_ANY_INIT should be a kernel only symbol.
> At least google had some hits on that symbol.
>
> netdev people should know....
>
OK, I do not want any conflicts and I want to stay in safe side ;-)
Subject: [PATCH] in6.h: extern's make no sense in userspace
Impact: fix 2 make headers_check warnings:
usr/include/linux/in6.h:47: extern's make no sense in userspace
usr/include/linux/in6.h:49: extern's make no sense in userspace
Signed-off-by: Jaswinder Singh Rajput <jaswinderrajput@gmail.com>
---
include/linux/in6.h | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/linux/in6.h b/include/linux/in6.h
index bc49204..44d921c 100644
--- a/include/linux/in6.h
+++ b/include/linux/in6.h
@@ -44,11 +44,11 @@ struct in6_addr
* NOTE: Be aware the IN6ADDR_* constants and in6addr_* externals are defined
* in network byte order, not in host byte order as are the IPv4 equivalents
*/
-extern const struct in6_addr in6addr_any;
#define IN6ADDR_ANY_INIT { { { 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0 } } }
-extern const struct in6_addr in6addr_loopback;
#define IN6ADDR_LOOPBACK_INIT { { { 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,1 } } }
#ifdef __KERNEL__
+extern const struct in6_addr in6addr_any;
+extern const struct in6_addr in6addr_loopback;
extern const struct in6_addr in6addr_linklocal_allnodes;
#define IN6ADDR_LINKLOCAL_ALLNODES_INIT \
{ { { 0xff,2,0,0,0,0,0,0,0,0,0,0,0,0,0,1 } } }
--
1.5.6.6
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH -tip] in6.h: extern's make no sense in userspace
2009-01-13 7:15 ` Sam Ravnborg
2009-01-13 7:20 ` Jaswinder Singh Rajput
@ 2009-01-13 7:22 ` David Miller
1 sibling, 0 replies; 4+ messages in thread
From: David Miller @ 2009-01-13 7:22 UTC (permalink / raw)
To: sam
Cc: jaswinder, mingo, harvey.harrison, gorcunov, roque, linux-kernel,
netdev
From: Sam Ravnborg <sam@ravnborg.org>
Date: Tue, 13 Jan 2009 08:15:13 +0100
> On Tue, Jan 13, 2009 at 12:27:39PM +0530, Jaswinder Singh Rajput wrote:
> > +#ifdef __KERNEL__
> > extern const struct in6_addr in6addr_any;
> > #define IN6ADDR_ANY_INIT { { { 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0 } } }
> > extern const struct in6_addr in6addr_loopback;
> > #define IN6ADDR_LOOPBACK_INIT { { { 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,1 } } }
> > -#ifdef __KERNEL__
>
> It looks wrong what IN6ADDR_ANY_INIT should be a kernel only symbol.
> At least google had some hits on that symbol.
>
> netdev people should know....
It is defined by libc's /usr/include/netinet/in.h
^ permalink raw reply [flat|nested] 4+ messages in thread