Date: Fri, 15 Nov 2002 14:53:53 -0800 (PST) From: Luigi Rizzo <luigi@FreeBSD.org> To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/netinet ip_input.c ip_mroute.c ip_mroute.h ip_output.c ip_var.h raw_ip.c src/sys/conf files src/sys/net route.c Message-ID: <200211152253.gAFMrrRq051324@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
luigi 2002/11/15 14:53:53 PST
Modified files:
sys/netinet ip_mroute.c ip_input.c ip_output.c
ip_var.h raw_ip.c ip_mroute.h
sys/conf files
sys/net route.c
Log:
Massive cleanup of the ip_mroute code.
No functional changes, but:
+ the mrouting module now should behave the same as the compiled-in
version (it did not before, some of the rsvp code was not loaded
properly);
+ netinet/ip_mroute.c is now truly optional;
+ removed some redundant/unused code;
+ changed many instances of '0' to NULL and INADDR_ANY as appropriate;
+ removed several static variables to make the code more SMP-friendly;
+ fixed some minor bugs in the mrouting code (mostly, incorrect return
values from functions).
This commit is also a prerequisite to the addition of support for PIM,
which i would like to put in before DP2 (it does not change any of
the existing APIs, anyways).
Note, in the process we found out that some device drivers fail to
properly handle changes in IFF_ALLMULTI, leading to interesting
behaviour when a multicast router is started. This bug is not
corrected by this commit, and will be fixed with a separate commit.
Detailed changes:
--------------------
netinet/ip_mroute.c all the above.
conf/files make ip_mroute.c optional
net/route.c fix mrt_ioctl hook
netinet/ip_input.c fix ip_mforward hook, move rsvp_input() here
together with other rsvp code, and a couple
of indentation fixes.
netinet/ip_output.c fix ip_mforward and ip_mcast_src hooks
netinet/ip_var.h rsvp function hooks
netinet/raw_ip.c hooks for mrouting and rsvp functions, plus
interface cleanup.
netinet/ip_mroute.h remove an unused and optional field from a struct
Most of the code is from Pavlin Radoslavov and the XORP project
Reviewed by: sam
MFC after: 1 week
Revision Changes Path
1.735 +1 -1 src/sys/conf/files
1.71 +1 -1 src/sys/net/route.c
1.216 +30 -3 src/sys/netinet/ip_input.c
1.83 +527 -869 src/sys/netinet/ip_mroute.c
1.20 +0 -3 src/sys/netinet/ip_mroute.h
1.170 +7 -4 src/sys/netinet/ip_output.c
1.68 +4 -3 src/sys/netinet/ip_var.h
1.105 +31 -9 src/sys/netinet/raw_ip.c
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200211152253.gAFMrrRq051324>