* [PATCH -tip] in6.h: extern's make no sense in userspace
@ 2009-01-13 6:57 Jaswinder Singh Rajput
2009-01-13 7:15 ` Sam Ravnborg
0 siblings, 1 reply; 4+ messages in thread
From: Jaswinder Singh Rajput @ 2009-01-13 6:57 UTC (permalink / raw)
To: Ingo Molnar, sam, harvey.harrison, gorcunov, roque, LKML, netdev
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__
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 6:57 [PATCH -tip] in6.h: extern's make no sense in userspace Jaswinder Singh Rajput
@ 2009-01-13 7:15 ` Sam Ravnborg
2009-01-13 7:20 ` Jaswinder Singh Rajput
2009-01-13 7:22 ` David Miller
0 siblings, 2 replies; 4+ messages in thread
From: Sam Ravnborg @ 2009-01-13 7:15 UTC (permalink / raw)
To: Jaswinder Singh Rajput
Cc: Ingo Molnar, harvey.harrison, gorcunov, roque, LKML, netdev
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....
Sam
^ permalink raw reply [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: 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
end of thread, other threads:[~2009-01-13 7:23 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-01-13 6:57 [PATCH -tip] in6.h: extern's make no sense in userspace Jaswinder Singh Rajput
2009-01-13 7:15 ` Sam Ravnborg
2009-01-13 7:20 ` Jaswinder Singh Rajput
2009-01-13 7:22 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).