From owner-freebsd-net@FreeBSD.ORG Sat Dec 1 09:55:41 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 D6A48934 for ; Sat, 1 Dec 2012 09:55:41 +0000 (UTC) (envelope-from brde@optusnet.com.au) Received: from mail05.syd.optusnet.com.au (mail05.syd.optusnet.com.au [211.29.132.186]) by mx1.freebsd.org (Postfix) with ESMTP id 699E68FC12 for ; Sat, 1 Dec 2012 09:55:40 +0000 (UTC) Received: from c122-106-175-26.carlnfd1.nsw.optusnet.com.au (c122-106-175-26.carlnfd1.nsw.optusnet.com.au [122.106.175.26]) by mail05.syd.optusnet.com.au (8.13.1/8.13.1) with ESMTP id qB19tVlL008715 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Sat, 1 Dec 2012 20:55:33 +1100 Date: Sat, 1 Dec 2012 20:55:31 +1100 (EST) From: Bruce Evans X-X-Sender: bde@besplex.bde.org To: Garrett Cooper Subject: Re: [RFC] Better document net.inet6 sysctls and prune dead sysctls (fwd) In-Reply-To: Message-ID: <20121201202138.J808@besplex.bde.org> References: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed X-Optus-CM-Score: 0 X-Optus-CM-Analysis: v=2.0 cv=SfSv7Ytu c=1 sm=1 a=fQ7hV5z4qHoA:10 a=kj9zAlcOel0A:10 a=PO7r1zJSAAAA:8 a=JzwRw_2MAAAA:8 a=dgaFysBZQSQA:10 a=N5sG7UAMo_YXlXBtImUA:9 a=CjuIK1q_8ugA:10 a=bxQHXO5Py4tHmhUgaywp5w==:117 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 09:55:42 -0000 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. >> #ifdef GATEWAY6 ... and options spelled without a V. >> ... >> @@ -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? >> @@ -461,12 +458,12 @@ >> "Internet6 Family"); >> >> /* net.inet6 */ >> -SYSCTL_NODE(_net_inet6, IPPROTO_IPV6, ip6, CTLFLAG_RW, 0, >> "IP6"); >> -SYSCTL_NODE(_net_inet6, IPPROTO_ICMPV6, icmp6, CTLFLAG_RW, 0, >> "ICMP6"); >> -SYSCTL_NODE(_net_inet6, IPPROTO_UDP, udp6, CTLFLAG_RW, 0, >> "UDP6"); >> -SYSCTL_NODE(_net_inet6, IPPROTO_TCP, tcp6, CTLFLAG_RW, 0, >> "TCP6"); >> +SYSCTL_NODE(_net_inet6, IPPROTO_IPV6, ip6, CTLFLAG_RW, 0, >> "IPv6"); >> +SYSCTL_NODE(_net_inet6, IPPROTO_ICMPV6, icmp6, CTLFLAG_RW, 0, >> "ICMPv6"); >> +SYSCTL_NODE(_net_inet6, IPPROTO_UDP, udp6, CTLFLAG_RW, 0, >> "UDPv6"); >> +SYSCTL_NODE(_net_inet6, IPPROTO_TCP, tcp6, CTLFLAG_RW, 0, >> "TCPv6"); >> #ifdef SCTP >> -SYSCTL_NODE(_net_inet6, IPPROTO_SCTP, sctp6, CTLFLAG_RW, 0, >> "SCTP6"); >> +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. 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. >> ... >> SYSCTL_VNET_INT(_net_inet6_ip6, IPV6CTL_V6ONLY, v6only, CTLFLAG_RW, Stray tab before CTLFLAG* (left over from the fancy formatting?). >> - &VNET_NAME(ip6_v6only), 0, ""); >> + &VNET_NAME(ip6_v6only), 0, >> + "Allow IPv4-mapped ip6 addresses per RFC 3493"); > Wrong To: address >_>. Bruce