From nobody Sun Aug 21 02:18:56 2022 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4M9K1H5hhcz4ZvXk; Sun, 21 Aug 2022 02:18:59 +0000 (UTC) (envelope-from glebius@freebsd.org) Received: from cell.glebi.us (glebi.us [162.251.186.162]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "cell.glebi.us", Issuer "cell.glebi.us" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id 4M9K1H4LbBz3LbF; Sun, 21 Aug 2022 02:18:59 +0000 (UTC) (envelope-from glebius@freebsd.org) Received: from cell.glebi.us (localhost [127.0.0.1]) by cell.glebi.us (8.16.1/8.16.1) with ESMTPS id 27L2IubH006280 (version=TLSv1.3 cipher=TLS_AES_256_GCM_SHA384 bits=256 verify=NO); Sat, 20 Aug 2022 19:18:56 -0700 (PDT) (envelope-from glebius@freebsd.org) Received: (from glebius@localhost) by cell.glebi.us (8.16.1/8.16.1/Submit) id 27L2Iuhg006279; Sat, 20 Aug 2022 19:18:56 -0700 (PDT) (envelope-from glebius@freebsd.org) X-Authentication-Warning: cell.glebi.us: glebius set sender to glebius@freebsd.org using -f Date: Sat, 20 Aug 2022 19:18:56 -0700 From: Gleb Smirnoff To: Michael Tuexen Cc: "src-committers@freebsd.org" , "dev-commits-src-all@freebsd.org" , "dev-commits-src-main@freebsd.org" Subject: Re: git: 8338690a0a43 - main - ip_reass: provide sysctl MIB returning IP fragment TTL Message-ID: References: <202208202041.27KKfe9e061428@gitrepo.freebsd.org> List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; none X-Rspamd-Queue-Id: 4M9K1H4LbBz3LbF X-Rspamd-Pre-Result: action=no action; module=replies; Message is reply to one we originated X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[] X-ThisMailContainsUnwantedMimeParts: N On Sun, Aug 21, 2022 at 12:07:39AM +0200, Michael Tuexen wrote: M> > commit 8338690a0a43c8b11a382e2160f8eedb9aeecf91 M> > Author: Gleb Smirnoff M> > AuthorDate: 2022-08-20 20:33:39 +0000 M> > Commit: Gleb Smirnoff M> > CommitDate: 2022-08-20 20:39:12 +0000 M> > M> > ip_reass: provide sysctl MIB returning IP fragment TTL M> > M> > For now it is read-only, but eventually the cycle that goes over M> > all fragments should be refactored and this MIB should also become M> > read/write. M> > M> > This MIB will allow SNMP daemons to implement MIB-II ipReasmTimeout MIB M> > straightfoward. Right now net-snmp compilation is broken by 1922eb3e9c2. M> > The base system bsnmpd is not broken just because it ignored PR_SLOWTIMO, M> > and thus always returned incorrectly doubled value for ipReasmTimeout. M> > --- M> > sys/netinet/ip_reass.c | 4 ++++ M> > 1 file changed, 4 insertions(+) M> > M> > diff --git a/sys/netinet/ip_reass.c b/sys/netinet/ip_reass.c M> > index 0420091e0a2a..e581e2e622c7 100644 M> > --- a/sys/netinet/ip_reass.c M> > +++ b/sys/netinet/ip_reass.c M> > @@ -167,6 +167,10 @@ SYSCTL_PROC(_net_inet_ip, OID_AUTO, maxfragbucketsize, M> > sysctl_maxfragbucketsize, "I", M> > "Maximum number of IPv4 fragment reassembly queue entries per bucket"); M> > M> > +static u_int ipfragttl = IPFRAGTTL / 2; M> > +SYSCTL_UINT(_net_inet_ip, OID_AUTO, fragttl, CTLFLAG_RD, &ipfragttl, M> > + IPFRAGTTL / 2, "IP fragment life time on reassembly queue"); M> What unit is used? seconds? milliseconds? You might want to state that. M> What us the semantic of IPFRAGTTL? What is the unit of that constant? I'll add '(seconds)' to the sysctl description. The semantic of IPFRAGTTL is ugly. It is number of timeouts, which are 0.5 second. A legacy of PR_SLOWTIMO. This will go away, so is not worth documenting. -- Gleb Smirnoff