Re: git: 1922eb3e9c23 - main - protosw: retire pr_slowtimo and pr_fasttimo
- In reply to: Charlie Li : "Re: git: 1922eb3e9c23 - main - protosw: retire pr_slowtimo and pr_fasttimo"
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Sat, 20 Aug 2022 20:47:05 UTC
On Sat, Aug 20, 2022 at 01:35:38PM -0400, Charlie Li wrote: C> > -#define PR_SLOWHZ 2 /* 2 slow timeouts per second */ C> > -#define PR_FASTHZ 5 /* 5 fast timeouts per second */ C> > - C> > /* C> > * This number should be defined again within each protocol family to avoid C> > * confusion. C> > C> > C> This broke net-mgmt/net-snmp, which uses PR_SLOWHZ directly. Upstream C> issue filed at https://github.com/net-snmp/net-snmp/issues/445 Thanks for report! Looking into mibII/tcp.c code, I'm 99% sure it wasn't correct on FreeBSD for at least 10-15 years if not always. The next failure is: mibgroup/ip-mib/data_access/scalars_sysctl.c:285:38: error: use of undeclared identifier 'PR_SLOWHZ' ipReasmTimeout_val = IPFRAGTTL / PR_SLOWHZ; And this code was actually constructing correct value. I created pull requests upstream. -- Gleb Smirnoff