Date: Thu, 22 Dec 2016 10:49:56 -0800 From: hiren panchasara <hiren@strugglingcoder.info> To: Gleb Smirnoff <glebius@FreeBSD.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r309745 - in head: share/man/man9 sys/kern sys/sys Message-ID: <20161222184956.GD82166@strugglingcoder.info> In-Reply-To: <20161222183334.GP9389@FreeBSD.org> References: <201612091758.uB9HwYC0087384@repo.freebsd.org> <20161222001928.GZ82166@strugglingcoder.info> <20161222183334.GP9389@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--Zljh9u/ceMLi+8mf Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On 12/22/16 at 10:33P, Gleb Smirnoff wrote: > Hi! >=20 > On Wed, Dec 21, 2016 at 04:19:28PM -0800, hiren panchasara wrote: > h> On 12/09/16 at 05:58P, Gleb Smirnoff wrote: > h> > Author: glebius > h> > Date: Fri Dec 9 17:58:34 2016 > h> > New Revision: 309745 > h> > URL: https://svnweb.freebsd.org/changeset/base/309745 > h> >=20 > h> > Log: > h> > Provide counter_ratecheck(), a MP-friendly substitution to ppsrate= check(). > h> > When rated event happens at a very quick rate, the ppsratecheck() = is not > h> > only racy, but also becomes a performance bottleneck. > h> > =20 > h> > Together with: rrs, jtl > h>=20 > h> Is it possible for you to MFC this and the following fix to it to 11? >=20 > Have you tried it and found useful? If yes, you are welcome to merge it. As you stated yourself in the commit message, this is a clear bug and an opportunity for performance enhancement. Do you feel this bug won't be in 11 and systems running on 11 won't have similar performance gains as those running on -head? Specially when 11 is the latest released version that many (like myself) would be running in the production. >=20 > My current work involves only FreeBSD head, I don't have any STABLE produ= ction > boxes, that's why I am very conservative with my MFCs: I do only obvious > conservative bugfixes, not performance improvements. The obvious reason we encourage original submitter to do MFCs is that (s)he is intimately familiar with the changes.=20 I can understand your point of not having any 11 based systems to test. I'll test this on our $work prod and do the MFC myself. Thanks for the original fix. Cheers, Hiren --Zljh9u/ceMLi+8mf Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQF8BAABCgBmBQJYXCBGXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXRBNEUyMEZBMUQ4Nzg4RjNGMTdFNjZGMDI4 QjkyNTBFMTU2M0VERkU1AAoJEIuSUOFWPt/lp0oH/3ghdQFFPUoBGLPGajuwAxpl YimyXcya96wh14xr+k6F5Gm/77WIPl0CwWHnr5A7zLpgJVpewH09A2CP4U9j6hAL rFo3Hu7CoKMwt23cVft5mVNh+pObmLZC9h/82xOyJqOA6sumKhOGwwjJjXn8S5+q xZqQ4Oc2zeD0KxbJU8CL+gUhAt4IMNVMAWKxV4KikK8X5snEM1QFZ/ns50gtnlQi TDnjNXpg69IflINufLO00+/kC/lNls1QQMhSTsY6SElJZJ6X+eH+sCragf6bHgvM Gw/qiwh3ni/12auEAhfQ0AVCPrgU27dWSHoPJ305FpwBdTZ6eY1m/VtL10ZRQgw= =Yxuq -----END PGP SIGNATURE----- --Zljh9u/ceMLi+8mf--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20161222184956.GD82166>