From owner-cvs-all Fri Nov 15 14:56:34 2002 Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id C9B3337B401; Fri, 15 Nov 2002 14:56:32 -0800 (PST) Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115]) by mx1.FreeBSD.org (Postfix) with ESMTP id 6C1A543E42; Fri, 15 Nov 2002 14:56:32 -0800 (PST) (envelope-from luigi@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.12.6/8.12.6) with ESMTP id gAFMrrmV051325; Fri, 15 Nov 2002 14:53:53 -0800 (PST) (envelope-from luigi@repoman.freebsd.org) Received: (from luigi@localhost) by repoman.freebsd.org (8.12.6/8.12.6/Submit) id gAFMrrRq051324; Fri, 15 Nov 2002 14:53:53 -0800 (PST) Message-Id: <200211152253.gAFMrrRq051324@repoman.freebsd.org> From: Luigi Rizzo Date: Fri, 15 Nov 2002 14:53:53 -0800 (PST) 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 X-FreeBSD-CVS-Branch: HEAD Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG 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