iproute2 not compiling anymore

iproute2 not compiling anymore

[Marcelo Tosatti]

Hi Erik, 

In previous messages you said iproute used to compile on "olders" 2.4.x 
kernel but doesnt compile anymore on recent 2.4. Is that information 
correct ? 

Can you tell me in more detail what is failing?

Interfaces should not change in a stable kernel. 

Re: iproute2 not compiling anymore

[Willy TARREAU]

Hi Marcelo, all,

On Sun, Oct 05, 2003 at 09:42:30AM -0300, Marcelo Tosatti wrote:
> In previous messages you said iproute used to compile on "olders" 2.4.x 
> kernel but doesnt compile anymore on recent 2.4. Is that information 
> correct ? 
> 
> Can you tell me in more detail what is failing?

Just tested, and I confirm this too :

gcc -D_GNU_SOURCE -O2 -Wstrict-prototypes -Wall -g -I../include-glibc -I/usr/include/db3 -include ../include-glibc/glibc-bugs.h -I/
sr/src/linux/include -I../include -DRESOLVE_HOSTNAMES   -c -o ip.o ip.c
In file included from ../include-glibc/netinet/in.h:7,
                 from ip.c:23:
/usr/src/linux/include/linux/in.h:141: field `gr_group' has incomplete type
/usr/src/linux/include/linux/in.h:147: field `gsr_group' has incomplete type
/usr/src/linux/include/linux/in.h:148: field `gsr_source' has incomplete type
/usr/src/linux/include/linux/in.h:154: field `gf_group' has incomplete type
/usr/src/linux/include/linux/in.h:157: field `gf_slist' has incomplete type
make[1]: *** [ip.o] Error 1
make[1]: Leaving directory `/data/src/net/ip-routing/iproute2-2.4.7-020116/ip'
make: *** [all] Error 2

These fields are of type 'struct sockaddr_storage' :

struct group_source_req
{
        __u32                   gsr_interface;  /* interface index */
        struct sockaddr_storage gsr_group;      /* group address */
        struct sockaddr_storage gsr_source;     /* source address */
};

But sockaddr_storage is defined in socket.h within such a #if :

#if defined(__KERNEL__) || !defined(__GLIBC__) || (__GLIBC__ < 2)

So it is clear that compiling with such headers on a recent libc outside the
kernel may fail. I don't know what changed exactly, the #if or
sockaddr_storage declaration.

BTW, wouldn't it be interesting to have some sort of way to bypass these
checks when we know that we want to compile against kernel headers ? I
encountered the case for arptables a few weeks ago. I find it very dirty to
add -D__KERNEL__ to user-space makefiles, and I don't find it logical to rebuild
a libc with pre-release kernel headers either.

So perhaps something like this would be useful for a limited set of userspace
programs :
  #if defined(__KERNEL__) || defined(REALLY_USE_KHDR) || !defined(__GLIBC__)...

Then, the affected programs could simply add a '#define REALLY_USE_KHDR' line
before including particular headers.

Any ideas ?
Willy

Re: iproute2 not compiling anymore

[David S. Miller]

On Sun, 5 Oct 2003 15:00:44 +0200
Willy TARREAU <willy@w.ods.org> wrote:

> /usr/src/linux/include/linux/in.h:147: field `gsr_group' has incomplete type

I'll happily fix this, thanks for reporting this.

Can you please in the future report such things on the networking
development list netdev@oss.sgi.com?  Thanks.

Re: iproute2 not compiling anymore

[Mr. James W. Laferriere]

	Hello Dave ,

On Sun, 5 Oct 2003, David S. Miller wrote:
> On Sun, 5 Oct 2003 15:00:44 +0200
> Willy TARREAU <willy@w.ods.org> wrote:
> > /usr/src/linux/include/linux/in.h:147: field `gsr_group' has incomplete type
> I'll happily fix this, thanks for reporting this.
> Can you please in the future report such things on the networking
> development list netdev@oss.sgi.com?  Thanks.
	What has happened to linux-net ?
	One more maillist I and others have to watch for exploding
	pieces .  JimL

-- 
       +------------------------------------------------------------------+
       | James   W.   Laferriere | System    Techniques | Give me VMS     |
       | Network        Engineer |     P.O. Box 854     |  Give me Linux  |
       | babydr@baby-dragons.com | Coudersport PA 16915 |   only  on  AXP |
       +------------------------------------------------------------------+

Re: iproute2 not compiling anymore

[David S. Miller]

On Sun, 5 Oct 2003 10:30:10 -0400 (EDT)
"Mr. James W. Laferriere" <babydr@baby-dragons.com> wrote:

> On Sun, 5 Oct 2003, David S. Miller wrote:
> > Can you please in the future report such things on the networking
> > development list netdev@oss.sgi.com?  Thanks.
> 	What has happened to linux-net ?

It's where users typically discuss user side configuration issues
not kernel development.

Re: iproute2 not compiling anymore

[Mr. James W. Laferriere]

	Hello Dave ,

On Sun, 5 Oct 2003, David S. Miller wrote:
> On Sun, 5 Oct 2003 10:30:10 -0400 (EDT)
> "Mr. James W. Laferriere" <babydr@baby-dragons.com> wrote:
> > On Sun, 5 Oct 2003, David S. Miller wrote:
> > > Can you please in the future report such things on the networking
> > > development list netdev@oss.sgi.com?  Thanks.
> > 	What has happened to linux-net ?
>
> It's where users typically discuss user side configuration issues
> not kernel development.
	netdev then is for Network Developement IN the Kernel ?
	iproute is a userland tool .
	Shouldn't conversations concerning it be in linux-net ?
	Above are quisitives ,  Not disparagements .  JimL
-- 
       +------------------------------------------------------------------+
       | James   W.   Laferriere | System    Techniques | Give me VMS     |
       | Network        Engineer |     P.O. Box 854     |  Give me Linux  |
       | babydr@baby-dragons.com | Coudersport PA 16915 |   only  on  AXP |
       +------------------------------------------------------------------+

Re: iproute2 not compiling anymore

[Erik Andersen]

On Sun Oct 05, 2003 at 09:42:30AM -0300, Marcelo Tosatti wrote:
> 
> Hi Erik, 
> 
> In previous messages you said iproute used to compile on "olders" 2.4.x 
> kernel but doesnt compile anymore on recent 2.4. Is that information 
> correct ? 
> 
> Can you tell me in more detail what is failing?
> 
> Interfaces should not change in a stable kernel. 

What Willy said covers it....

 -Erik

--
Erik B. Andersen             http://codepoet-consulting.com/
--This message was written using 73% post-consumer electrons--

Re: iproute2 not compiling anymore

[Marcelo Tosatti]

On Sun, 5 Oct 2003, Willy TARREAU wrote:

> Hi Marcelo, all,
> 
> On Sun, Oct 05, 2003 at 09:42:30AM -0300, Marcelo Tosatti wrote:
> > In previous messages you said iproute used to compile on "olders" 2.4.x 
> > kernel but doesnt compile anymore on recent 2.4. Is that information 
> > correct ? 
> > 
> > Can you tell me in more detail what is failing?
> 
> Just tested, and I confirm this too :

Willy,

David seems to have fixed it. Mind trying to again with the latest BK 
tree?

Danke

Re: iproute2 not compiling anymore

[Willy TARREAU]

Hi Marcelo,

On Thu, Oct 09, 2003 at 10:52:05AM -0300, Marcelo Tosatti wrote:
 
> David seems to have fixed it. Mind trying to again with the latest BK 
> tree?

I have just checked, but I still get the errors, although the patch looks fine
at first glance. I wondered whether I was doing something wrong, so I rechecked
and it compiles cleanly with 2.4.21, and does not since 2.4.22. Unfortunately,
I'm too tired to investigate more, so I should do it tomorrow. Here is the
compilation output, just in case it helps David. Please ask for .config if
needed, but I don't think so.

Cheers,
Willy

$ make KERNEL_INCLUDE=/usr/src/linux-2.4.23-pre6-20031009/include
make[1]: Entering directory `/data/src/net/ip-routing/iproute2-2.4.7-020116/lib'
gcc -D_GNU_SOURCE -O2 -Wstrict-prototypes -Wall -g -I../include-glibc -I/usr/include/db3 -include ../include-glibc/glibc-bugs.h -I/usr/src/linux-2.4.23-pre6-20031009/include -I../include -DRESOLVE_HOSTNAMES   -c -o ll_map.o ll_map.c
In file included from ../include-glibc/netinet/in.h:7,
                 from ll_map.c:19:
/usr/src/linux-2.4.23-pre6-20031009/include/linux/in.h:141: field `gr_group' has incomplete type
/usr/src/linux-2.4.23-pre6-20031009/include/linux/in.h:147: field `gsr_group' has incomplete type
/usr/src/linux-2.4.23-pre6-20031009/include/linux/in.h:148: field `gsr_source' has incomplete type
/usr/src/linux-2.4.23-pre6-20031009/include/linux/in.h:154: field `gf_group' has incomplete type
/usr/src/linux-2.4.23-pre6-20031009/include/linux/in.h:157: field `gf_slist' has incomplete type
make[1]: *** [ll_map.o] Error 1
make[1]: Leaving directory `/data/src/net/ip-routing/iproute2-2.4.7-020116/lib'
make: *** [all] Error 2
$

Re: iproute2 not compiling anymore

[David S. Miller]

On Fri, 10 Oct 2003 02:13:52 +0200
Willy TARREAU <willy@w.ods.org> wrote:

> I'm too tired to investigate more, so I should do it tomorrow. Here is the
> compilation output, just in case it helps David. Please ask for .config if
> needed, but I don't think so.

I know what's wrong, linux/in.h needs to include linux/socket.h
Duh, I'll fix this up.

Re: iproute2 not compiling anymore

[Willy Tarreau]

Hi David,

On Thu, Oct 09, 2003 at 10:45:35PM -0700, David S. Miller wrote:
> On Fri, 10 Oct 2003 02:13:52 +0200
> Willy TARREAU <willy@w.ods.org> wrote:
> 
> > I'm too tired to investigate more, so I should do it tomorrow. Here is the
> > compilation output, just in case it helps David. Please ask for .config if
> > needed, but I don't think so.
> 
> I know what's wrong, linux/in.h needs to include linux/socket.h
> Duh, I'll fix this up.

You're right, it now compiles. Here's the patch :

diff -urN linux-2.4.23-pre7/include/linux/in.h linux-2.4.23-pre7-fix/include/linux/in.h
--- linux-2.4.23-pre7/include/linux/in.h	Fri Oct 10 08:47:15 2003
+++ linux-2.4.23-pre7-fix/include/linux/in.h	Fri Oct 10 08:49:20 2003
@@ -19,6 +19,7 @@
 #define _LINUX_IN_H
 
 #include <linux/types.h>
+#include <linux/socket.h>
 
 /* Standard well-defined IP protocols.  */
 enum {