Re: [mpls-linux-devel] Hi all

[Ramon C. <cas...@in...>]

On 23 Feb 2004, Jamal Hadi Salim wrote:

> didnt mean to desert you - things have been extremely hectic. Still are
> (both at work and home). So i hope you wont mind me being intermittent.
> I will try my best to respond when i believe its high priority like in
> this case.

I understand... I will also have some other projects that may impact my
productivity :)


>
> Scary - It compiles just fine for me.

Uhm...


> Substitute that nhl to be nhp - its what it should be.

Well, I _know_ that :)



> ------
> [root@jzny 261-mod]# gcc -v

gandalf casellas$ gcc -v
Reading specs from /usr/lib/gcc-lib/i686-pc-linux-gnu/3.3.3/specs
Configured with: /var/tmp/portage/gcc-3.3.3/work/gcc-3.3.3/configure
--prefix=/usr --bindir=/usr/i686-pc-linux-gnu/gcc-bin/3.3
--includedir=/usr/lib/gcc-lib/i686-pc-linux-gnu/3.3.3/include
--datadir=/usr/share/gcc-data/i686-pc-linux-gnu/3.3
--mandir=/usr/share/gcc-data/i686-pc-linux-gnu/3.3/man
--infodir=/usr/share/gcc-data/i686-pc-linux-gnu/3.3/info --enable-shared
--host=i686-pc-linux-gnu --target=i686-pc-linux-gnu --with-system-zlib
--enable-languages=c,c++,f77,objc,java --enable-threads=posix
--enable-long-long --disable-checking --enable-cstdio=stdio
--enable-clocale=generic --enable-__cxa_atexit
--enable-version-specific-runtime-libs
--with-gxx-include-dir=/usr/lib/gcc-lib/i686-pc-linux-gnu/3.3.3/include/g++-v3
--with-local-prefix=/usr/local --enable-shared --enable-nls
--without-included-gettext --x-includes=/usr/X11R6/include
--x-libraries=/usr/X11R6/lib --enable-interpreter --enable-java-awt=xlib
--with-x --disable-multilib
Thread model: posix
gcc version 3.3.3 20040217


>
> I susbtituted that nhl with nhx and it still compiled.

gandalf mpls-kernel-davem# grep -bn MPLS .config
583:12615:CONFIG_IP_MPLS=y
603:12997:CONFIG_INET6_MPLS=m
706:15439:CONFIG_NET_MPLS=y


> ...
> Why did yours work?

What happens in your tree if you put a #error directive in other parts of
the file/other files? I'm afraid that you just did not compile MPLS
support in...




> Can you replace with the above #error and see if still compiles?

  CC      net/ipv4/fib_semantics.o
net/ipv4/fib_semantics.c:351:2: #error "I bet this month salary that this
message is displayed"
make[2]: *** [net/ipv4/fib_semantics.o] Error 1
make[1]: *** [net/ipv4] Error 2
make: *** [net] Error 2


Well, what did you expect ? :))



Update:
******
Nahhh, I bet 'today salary' that you do not have CONFIG_IP_ROUTE_MULTIPATH
.that would explain it... :)


Other things that I will cleanup (as soon as James server is online)
net/mpls/mpls_fib.c: In function `mpls_ilm_lookup':
net/mpls/mpls_fib.c:212: warning: ISO C90 forbids mixed declarations and
code
net/mpls/mpls_fib.c: In function `lt_fill_ilm':
net/mpls/mpls_fib.c:340: warning: implicit declaration of function
`gen_copy_stats'
net/mpls/mpls_fib.c: In function `mpls_get_ilm':
net/mpls/mpls_fib.c:446: warning: unused variable `lt'
net/mpls/mpls_fib.c:447: warning: unused variable `i'
net/netlink/af_netlink.c: In function `netlink_proto_init':
net/netlink/af_netlink.c:1136: warning: implicit declaration of function
`l2cnetlink_init'



Regards,
R.