From owner-freebsd-net@FreeBSD.ORG Sat Dec 1 10:06:05 2012 Return-Path: Delivered-To: freebsd-net@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 815319CD for ; Sat, 1 Dec 2012 10:06:05 +0000 (UTC) (envelope-from yanegomi@gmail.com) Received: from mail-oa0-f54.google.com (mail-oa0-f54.google.com [209.85.219.54]) by mx1.freebsd.org (Postfix) with ESMTP id 3DB898FC0C for ; Sat, 1 Dec 2012 10:06:04 +0000 (UTC) Received: by mail-oa0-f54.google.com with SMTP id n9so1692758oag.13 for ; Sat, 01 Dec 2012 02:06:04 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=rq0ibiB3dY9X/qNOZOdG7/cZJv55bIxpK5+ovHeqans=; b=oboSrL+7weKQs18BiRdL/i8DrMpkrasXKh/8NsnNs1DnKtSsGn2qS5N7I/x9+k5tS1 uTpTuC9+Rq6fiE2/A+Oztvi070tKjo4VLx6kyBq1wPAZjhWIQinqPwW+WANI2vD24or6 3nGqHrNdx3H4RIL6AM+hCEiOCIiUFgm2BcVzilYT0N8dFoJpQ60CyFit8VurziYkT/KV Ux4/xUBemBcgb/TJNOG6r3LBaSD4U2dvbQO258TJrMranvDlP6vfOG+AeDT758xoxa78 aLvh0uOrnLmMd5DiRNcu+tPm+pDh96rKEDtdroA6bl5xE/jIUbp3yhi59OfK4DtYJBTY jquQ== MIME-Version: 1.0 Received: by 10.60.10.133 with SMTP id i5mr3368162oeb.24.1354356364385; Sat, 01 Dec 2012 02:06:04 -0800 (PST) Received: by 10.76.143.33 with HTTP; Sat, 1 Dec 2012 02:06:04 -0800 (PST) In-Reply-To: <20121201202138.J808@besplex.bde.org> References: <20121201202138.J808@besplex.bde.org> Date: Sat, 1 Dec 2012 02:06:04 -0800 Message-ID: Subject: Re: [RFC] Better document net.inet6 sysctls and prune dead sysctls (fwd) From: Garrett Cooper To: Bruce Evans Content-Type: text/plain; charset=ISO-8859-1 Cc: freebsd-net@freebsd.org X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Networking and TCP/IP with FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 01 Dec 2012 10:06:05 -0000 On Sat, Dec 1, 2012 at 1:55 AM, Bruce Evans wrote: > On Fri, 30 Nov 2012, Garrett Cooper wrote: > >> On Fri, 30 Nov 2012, Garrett Cooper wrote: > > I got the previous one but was busy. This one seems to only add freebsd-net > to the Cc list > >>>> It would be good to fix the style bugs when changing lots. >>>> ... >>> >>> Is this better? I tried to be consistent about using v6 properly >>> when dealing with protocols in order to match what the IETF did and I >>> believe I properly integrated in your comments. > > > I'm (almost) happy with this. Noticed a couple more details. We're > stuck with sysctls misspelled by name (without a v)... > >>> Index: sys/netinet6/in6_proto.c >>> =================================================================== >>> --- sys/netinet6/in6_proto.c (revision 243557) >>> +++ sys/netinet6/in6_proto.c (working copy) >>> ... >>> >>> @@ -382,9 +382,9 @@ >>> */ >>> #ifndef IPV6FORWARDING > > Stray tab here. Fixed. >>> #ifdef GATEWAY6 > > ... and options spelled without a V. "Fixing" this will break backwards compatibility -_-. Trying to avoid changing interfaces/#defines. >>> ... >>> >>> @@ -443,13 +441,12 @@ >>> >>> /* ICMPV6 parameters */ >>> VNET_DEFINE(int, icmp6_rediraccept) = 1;/* accept and process redirects >>> */ >>> -VNET_DEFINE(int, icmp6_redirtimeout) = 10 * 60; /* 10 minutes */ > > Lost this? This is intentional. This and ip6_rr_prune need to be pruned because they're unused in the kernel proper. ... >>> +SYSCTL_NODE(_net_inet6, IPPROTO_SCTP, sctp6, CTLFLAG_RW, 0, >>> "SCTPv6"); >>> #endif >>> #ifdef IPSEC >>> SYSCTL_NODE(_net_inet6, IPPROTO_ESP, ipsec6, CTLFLAG_RW, 0, "IPSEC6"); > > I don't like this fancy formatting. It is hard to maintain, and it is only > possible to line up all the fields and fit on 1 line when all are short. > Some mailer already mangled the lines by splitting them and quoting the > split. alpine with ^j and gmail probably again (I really need to fix that or get a less "intelligent" mailer) :/... > Descriptions like this that used to less than echo the leaf of the > sysctl name are less than useful. They seem to be bug for bug > compatible with ipv4, however: net.inet.ip is now described as IP (no > v or 4 in sight). net.inet6.ip6 is now described as IP6. I also > don't like duplicating the 6 at every lower level in name. I'll just pull the descriptions because it's useless bloat in the kernel in text form. >>> ... >>> SYSCTL_VNET_INT(_net_inet6_ip6, IPV6CTL_V6ONLY, v6only, CTLFLAG_RW, > > Stray tab before CTLFLAG* (left over from the fancy formatting?). I think so. Either way, I quashed what was there :). >>> - &VNET_NAME(ip6_v6only), 0, ""); >>> + &VNET_NAME(ip6_v6only), 0, >>> + "Allow IPv4-mapped ip6 addresses per RFC 3493"); Thanks again! -Garrett