From owner-svn-src-all@FreeBSD.ORG Fri Feb 7 17:14:42 2014 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 9C11CE1F; Fri, 7 Feb 2014 17:14:42 +0000 (UTC) Received: from cell.glebius.int.ru (glebius.int.ru [81.19.69.10]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id EB30613DF; Fri, 7 Feb 2014 17:14:41 +0000 (UTC) Received: from cell.glebius.int.ru (localhost [127.0.0.1]) by cell.glebius.int.ru (8.14.7/8.14.7) with ESMTP id s17HEdgO085922 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO); Fri, 7 Feb 2014 21:14:39 +0400 (MSK) (envelope-from glebius@FreeBSD.org) Received: (from glebius@localhost) by cell.glebius.int.ru (8.14.7/8.14.7/Submit) id s17HEdDs085921; Fri, 7 Feb 2014 21:14:39 +0400 (MSK) (envelope-from glebius@FreeBSD.org) X-Authentication-Warning: cell.glebius.int.ru: glebius set sender to glebius@FreeBSD.org using -f Date: Fri, 7 Feb 2014 21:14:39 +0400 From: Gleb Smirnoff To: "Bjoern A. Zeeb" Subject: Re: svn commit: r261601 - in head: sys/net sys/netinet sys/netinet6 usr.bin/netstat Message-ID: <20140207171439.GD66160@FreeBSD.org> References: <201402071518.s17FINtP074186@svn.freebsd.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.22 (2013-10-16) Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.17 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 07 Feb 2014 17:14:42 -0000 On Fri, Feb 07, 2014 at 03:34:27PM +0000, Bjoern A. Zeeb wrote: B> > * XXX This does not end up updating timeouts at runtime B> > * and only reflects the value for the last table added :-/ B> > */ B> > -SYSCTL_VNET_INT(_net_inet_flowtable, OID_AUTO, syn_expire, CTLFLAG_RW, B> > +SYSCTL_VNET_INT(_net_flowtable, OID_AUTO, syn_expire, CTLFLAG_RW, B> > &VNET_NAME(flowtable_syn_expire), 0, B> > "seconds after which to remove syn allocated flow."); B> > -SYSCTL_VNET_INT(_net_inet_flowtable, OID_AUTO, udp_expire, CTLFLAG_RW, B> > +SYSCTL_VNET_INT(_net_flowtable, OID_AUTO, udp_expire, CTLFLAG_RW, B> > &VNET_NAME(flowtable_udp_expire), 0, B> > "seconds after which to remove flow allocated to UDP."); B> > -SYSCTL_VNET_INT(_net_inet_flowtable, OID_AUTO, fin_wait_expire, CTLFLAG_RW, B> > +SYSCTL_VNET_INT(_net_flowtable, OID_AUTO, fin_wait_expire, CTLFLAG_RW, B> > &VNET_NAME(flowtable_fin_wait_expire), 0, B> > "seconds after which to remove a flow in FIN_WAIT."); B> > -SYSCTL_VNET_INT(_net_inet_flowtable, OID_AUTO, tcp_expire, CTLFLAG_RW, B> > +SYSCTL_VNET_INT(_net_flowtable, OID_AUTO, tcp_expire, CTLFLAG_RW, B> > &VNET_NAME(flowtable_tcp_expire), 0, B> > "seconds after which to remove flow allocated to a TCP connection."); B> … B> B> B> This is not what we are doing for a lot of other things in the tree which simply treat “inet” as “IP” without version number. Now I know that the floatable sits even between L3 and L2 given it caches pairs, I am still not sure I want to make net.* the “dumping ground” for these. And that's not nice that we are using "inet" as dumping ground for all kind of IP. B> Pending a better solution I don’t care apart from that sysctl.confs entries for the people using floatable after all break with this. Taking into account absence of FLOWTABLE in GENERIC and all the problems that were (are?) there, I'm pretty sure these people are empty set :) -- Totus tuus, Glebius.