From owner-freebsd-rc@FreeBSD.ORG Tue Aug 25 14:55:23 2009 Return-Path: Delivered-To: freebsd-rc@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 21A73106568D; Tue, 25 Aug 2009 14:55:23 +0000 (UTC) (envelope-from ady@ady.ro) Received: from mail-ew0-f209.google.com (mail-ew0-f209.google.com [209.85.219.209]) by mx1.freebsd.org (Postfix) with ESMTP id 5F2558FC26; Tue, 25 Aug 2009 14:55:22 +0000 (UTC) Received: by ewy5 with SMTP id 5so1875712ewy.36 for ; Tue, 25 Aug 2009 07:55:21 -0700 (PDT) MIME-Version: 1.0 Sender: ady@ady.ro Received: by 10.210.37.19 with SMTP id k19mr3713414ebk.52.1251211701604; Tue, 25 Aug 2009 07:48:21 -0700 (PDT) In-Reply-To: <4A933B77.7030108@delphij.net> References: <4A7B2792.4090803@delphij.net> <4A7F948A.2040602@delphij.net> <4A7F9EC6.9090605@FreeBSD.org> <4A7FAA42.4030607@delphij.net> <78cb3d3f0908101113s412021adk8956dd083a8fed72@mail.gmail.com> <4A806D17.9020404@delphij.net> <4A933294.8000906@delphij.net> <4A93351E.2020107@FreeBSD.org> <4A933B77.7030108@delphij.net> Date: Tue, 25 Aug 2009 16:48:21 +0200 X-Google-Sender-Auth: 09ed31d0db89436a Message-ID: <78cb3d3f0908250748j16ff3879y4ea28fd83b0780c0@mail.gmail.com> From: Adrian Penisoara To: d@delphij.net Content-Type: text/plain; charset=ISO-8859-1 X-Content-Filtered-By: Mailman/MimeDel 2.1.5 Cc: Doug Barton , freebsd-rc@freebsd.org Subject: Re: [RFC] script for binding ARP <-> IP pairs X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussion related to /etc/rc.d design and implementation." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 25 Aug 2009 14:55:23 -0000 Hi, On Tue, Aug 25, 2009 at 3:16 AM, Xin LI wrote: > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 > > Doug Barton wrote: > > Xin LI wrote: > >> Any objections for the attached patch? I'll commit this version if > >> everybody is happy with it and then see if re@ would approve it for > >> 8.0-RELEASE. > > > > Sorry if I wasn't clear before, but all of the rc.conf variables > > should be prepended with static_arp_ (as opposed to just arp_). So > > rather than arp_args you'd want static_arp_args. I realize you're not > > using set_rcvar() but it's still a good idea to do that for > > consistency's sake, as well as to make it easier for admins to sort > > stuff, etc. Please make sure you change that in the man page as well. > For consistency we should leave it as "arp_xxx" like it's currently done for static_routes/route_xxx and ipv6_static_routes/ipv6_route_xxx ... If we want to keep consistency then either leave it as "static_arp_pairs/arp_xxx" or change it to something like "static_arps/arp_xxx > > > I see. I have corrected arp_ prefixes to static_arp_ prefixes. Note > that "arp_args" are not changed because they are intermediate variables > that are intended as "arp(1) arguments". I have marked these as local. > > > Other than that it looks fine to me. > > > > Silly question that I'm sorry just occurred to me right now, have you > > tested this with IPv6? :) > > I think IPv6 would need to use "ndp" instead (cross referenced from > arp(8)) but am not sure. Because the script rely on space or tab as > separator I think it would just work after a simple s/arp/ndp/g for ndp > but I don't have a local production IPv6 network for the test... > > Cheers, > - -- > Xin LI http://www.delphij.net/ > FreeBSD - The Power to Serve! > -----BEGIN PGP SIGNATURE----- > Version: GnuPG v2.0.12 (FreeBSD) > > iEYEARECAAYFAkqTO3cACgkQi+vbBBjt66CtWwCZARUOgOP3AoZEEEeCCXl9ziVT > ZwsAoIC1cmv6d0QY7z1RjEoFcfz0Sey/ > =py3J > -----END PGP SIGNATURE----- > > Index: etc/defaults/rc.conf > =================================================================== > --- etc/defaults/rc.conf (revision 196516) > +++ etc/defaults/rc.conf (working copy) > @@ -357,6 +357,7 @@ > > ### Network routing options: ### > defaultrouter="NO" # Set to default gateway (or NO). > +static_arp_pairs="" # Set to static ARP list (or leave empty). > static_routes="" # Set to static route list (or leave > empty). > natm_static_routes="" # Set to static route list for NATM (or > leave empty). > gateway_enable="NO" # Set to YES if this host will be a > gateway. > Please add commented line(s) as an example on how to use this, like for ipv6_static_routes: #static_arp_pairs="xxx" # An example to set an ARP static entry #arp_xxx="10.20.30.40 00:11:22:33:44:55" Regards, Adrian Penisoara EnterpriseBSD