From owner-svn-src-head@freebsd.org Tue Jul 19 12:25:46 2016 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 40C9BB9D943; Tue, 19 Jul 2016 12:25:46 +0000 (UTC) (envelope-from gjb@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) by mx1.freebsd.org (Postfix) with ESMTP id 257E216CB; Tue, 19 Jul 2016 12:25:46 +0000 (UTC) (envelope-from gjb@FreeBSD.org) Received: from FreeBSD.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) by freefall.freebsd.org (Postfix) with ESMTP id C06FC180F; Tue, 19 Jul 2016 12:25:45 +0000 (UTC) (envelope-from gjb@FreeBSD.org) Date: Tue, 19 Jul 2016 12:25:45 +0000 From: Glen Barber To: Randall Stewart Cc: Gleb Smirnoff , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r302998 - head/sys/kern Message-ID: <20160719122545.GC65494@FreeBSD.org> References: <201607180929.u6I9T9Uw063705@repo.freebsd.org> <64C1543A-3EDE-4852-88EA-5B0B78FCF016@netflix.com> <58868615-8255-4D8B-BD9E-8E19A734CB6C@netflix.com> <5E551FA8-C779-45AE-B038-D8B51B53EEAA@netflix.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="4ZLFUWh1odzi/v6L" Content-Disposition: inline In-Reply-To: <5E551FA8-C779-45AE-B038-D8B51B53EEAA@netflix.com> X-Operating-System: FreeBSD 11.0-CURRENT amd64 X-SCUD-Definition: Sudden Completely Unexpected Dataloss X-SULE-Definition: Sudden Unexpected Learning Event X-PEKBAC-Definition: Problem Exists, Keyboard Between Admin/Computer User-Agent: Mutt/1.5.24 (2015-08-30) X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 19 Jul 2016 12:25:46 -0000 --4ZLFUWh1odzi/v6L Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jul 19, 2016 at 01:43:16PM +0200, Randall Stewart wrote: > Gleb >=20 > Ok >=20 > I have now updated >=20 > https://reviews.freebsd.org/D7135 >=20 > You can take this or not=E2=80=A6 I really don=E2=80=99t care either way= =E2=80=A6 (you are welcome to > own the kern_timeout.c code I hate it) :-) >=20 > Basically when you went off and re-factored kern_timeout.c I had worked i= n parallel on fixing > the bugs you were seeing.. There were three distinct problems that I fixe= d=E2=80=A6 but then > you had refactored the stop() routine.. and I thought ok.. thats fine. I = had actually thought about > doing something similar to what you did and was too chicken to poke that = much at it.. it has > always had a nasty habit of biting back when you make a lot of changes :-D >=20 > I know my version has worked for quite some time in my testing so I broug= ht it back. > Complete with its 3 return codes (I only recently switched to your versio= n and thus > started having difficulties with leaks and crashes)=E2=80=A6. >=20 > You are welcome not to use this.. I know it works (it ran > on a number of machines at NF last night.. and we will of course continue= testing > it as we finish our dev testing for the upcoming OCA software release).. = For now > this is what will be going out into the OCA=E2=80=99s at least :-) >=20 I'm honestly done with this topic, and at the point now where I'm considering backing out all changes to callout(9) and related changes to the state they were at in stable/10. This changes the KBI, and if it needs to be done, it needs to happen now. We cannot wait for RC1 phase for this, and the amount of churn to get things into a working state with the current implementation far outweighs the benefit of the dangers. Glen --4ZLFUWh1odzi/v6L Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJXjhxEAAoJEAMUWKVHj+KT5xIP/jg4d79vtnMxLGSMprTEJ5ex +zXlcL1hut+Ja8tdBOdR0XxwnMZJKG77a4nRpi1pd3F+tohHxbthUHPzpq4Bs6Os LUftWJL9Zrkc/oGP3AziPPYtQmutNgmeKKBM6O88UEqp/cb8jtm6nmF56E0HxyMI uXQ6cjm9x6eRcRt1J8ta9QkmReUWWtoDc34br7pphEB/6O2174uvYWqtTof/SCKL KT2V6R9fX98ThhL3OgqO6a9pF51uLMN3UpDqrluOOQxk4NuJTMfEC51WQtY+HGAL ZRDmUj+/fWgNVgk6sbdBgJPgH+W+EJtCgg15ietOofur+7kJBbT9NxvaIfHJhN3N TCADGLSZfCryX7/bUET8xkqXUOHRxxF3Ukp5oby6NRUwzE6H3hqk0/MJgLWHsVyI 1E46Ay2DN9+gLuhkIhx6rNF7a3xvA6ZmMIk6uPZoi1V7BAou9XS6cTZiR8xAqy9T 5O0fU8dBQ4oeWJuZp2i1Yjq/9KmQ+GavO4LCdzLKThzTmJAuCm2I1L7HpnNfaQIR 6J0nIuoGczy7vFXZ7ZfMKi34p1g8WqVETwbr9lOPc/GLOhY/px13G1UdOKFU5+bs mgpkcNy1A625hrBRkjDhueHgXIOWB2zc+Iq3KGzeMqO1uuUoH9I3BFtFJhjSUROZ ZvFeKzepJ0aTtJzS+jXR =gsnG -----END PGP SIGNATURE----- --4ZLFUWh1odzi/v6L--