From owner-freebsd-net@FreeBSD.ORG Thu Aug 28 17:29:27 2014 Return-Path: Delivered-To: freebsd-net@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 419A2823; Thu, 28 Aug 2014 17:29:27 +0000 (UTC) Received: from vps.hungerhost.com (vps.hungerhost.com [216.38.53.176]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 0AAF8151E; Thu, 28 Aug 2014 17:29:26 +0000 (UTC) Received: from [96.232.17.80] (port=64837 helo=[10.189.221.169]) by vps.hungerhost.com with esmtpsa (TLSv1:DHE-RSA-AES256-SHA:256) (Exim 4.82) (envelope-from ) id 1XN2jO-0001AK-HX; Thu, 28 Aug 2014 12:39:27 -0400 From: "George Neville-Neil" To: "Eggert, Lars" Subject: Re: Patches for RFC6937 and draft-ietf-tcpm-newcwv-00 Date: Thu, 28 Aug 2014 12:39:09 -0400 Message-ID: <1BF6E077-0B47-41C9-8B47-848E048517B4@neville-neil.com> In-Reply-To: <4F10C5BC-03B2-486D-A5EF-C55582869ED1@netapp.com> References: <259C9434-C6FE-42EA-823D-ECB024DBF3D7@netapp.com> <814E0886-1B6B-4316-8BAB-684DAFDE1983@netapp.com> <20140826145517.GD12732@gmail.com> <76D986F7-72A8-4ABE-8731-064C6C77A56F@netapp.com> <3ABE6D0D-1D98-425B-BDAD-8D1B9026AD8F@netapp.com> <4F10C5BC-03B2-486D-A5EF-C55582869ED1@netapp.com> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=_MailMate_D05B247A-8A62-410E-A48B-05C2B800C34B_="; micalg=pgp-sha1; protocol="application/pgp-signature" X-Mailer: MailMate (1.8r4214) X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - vps.hungerhost.com X-AntiAbuse: Original Domain - freebsd.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - neville-neil.com X-Get-Message-Sender-Via: vps.hungerhost.com: authenticated_id: gnn@neville-neil.com Cc: Tom Jones , Adrian Chadd , FreeBSD Net X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: Networking and TCP/IP with FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 28 Aug 2014 17:29:27 -0000 This is an OpenPGP/MIME signed message (RFC 3156 and 4880). --=_MailMate_D05B247A-8A62-410E-A48B-05C2B800C34B_= Content-Type: text/plain Content-Transfer-Encoding: quoted-printable Adrian, Can you put this into a Phabricator for review? Lars, How have you been testing this? Best, George On 27 Aug 2014, at 4:01, Eggert, Lars wrote: > Yep > > On 2014-8-27, at 9:53, Adrian Chadd wrote: > >> Ok. Is it the same patch you sent out in Feb? >> >> >> -a >> >> >> On 27 August 2014 00:43, Eggert, Lars wrote: >>> Not as far as I know. >>> >>> Lars >>> >>> On 2014-8-27, at 9:39, Adrian Chadd wrote: >>> >>>> Is there a PR for it? >>>> >>>> >>>> -a >>>> >>>> >>>> On 27 August 2014 00:23, Eggert, Lars wrote: >>>>> It would be great if people could also review Aris' PRR patch - RFC= 6937 has been out for a while. >>>>> >>>>> Lars >>>>> >>>>> >>>>> >>>>> >>>>> On 2014-8-26, at 20:09, Adrian Chadd wrote: >>>>> >>>>>> Hi! >>>>>> >>>>>> I'm going to merge Tom's work in a week unless someone gives me a >>>>>> really good reason not to. >>>>>> >>>>>> I think there's been enough work and discussion about it since the= >>>>>> first post from Lars in Feburary and enough review opportunity. >>>>>> >>>>>> >>>>>> -a >>>>>> >>>>>> >>>>>> On 26 August 2014 07:55, Tom Jones wrote: >>>>>>> On Tue, Aug 26, 2014 at 02:43:49PM +0000, Eggert, Lars wrote: >>>>>>>> Hi, >>>>>>>> >>>>>>>> the newcwv patch is probably stale now with Tom Jones' recent pa= tch based on >>>>>>>> a more up-to-date version of the Internet-Draft, but the PRR pat= ch should >>>>>>>> still be useful? >>>>>>> >>>>>>> My newcwv patch is much more up to date than Aris's, but it is sl= ightly >>>>>>> different in implementation. I have had a few suggestions from Ad= rian, but he >>>>>>> couldn't comment on how it relates to the tcp internals. >>>>>>> >>>>>>> There is a PR: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D= 191520 >>>>>>> >>>>>>> The biggest difference in structure between mine and Aris's patch= is the use of >>>>>>> tcp timers. It would be good to hear if my approach or Aris's is = prefered. >>>>>>> >>>>>>>> On 2014-6-19, at 23:35, George Neville-Neil wrote: >>>>>>>> >>>>>>>>> On 4 Feb 2014, at 1:38, Eggert, Lars wrote: >>>>>>>>> >>>>>>>>>> Hi, >>>>>>>>>> >>>>>>>>>> below are two patches that implement RFC6937 ("Proportional Ra= te Reduction for TCP") and draft-ietf-tcpm-newcwv-00 ("Updating TCP to su= pport Rate-Limited Traffic"). They were done by Aris Angelogiannopoulos f= or his MS thesis, which is at https://eggert.org/students/angelogiannopou= los-thesis.pdf. >>>>>>>>>> >>>>>>>>>> The patches should apply to -CURRENT as of Sep 17, 2013. (Sorr= y for the delay in sending them, we'd been trying to get some feedback fr= om committers first, without luck.) >>>>>>>>>> >>>>>>>>>> Please note that newcwv is still a work in progress in the IET= F, and the patch has some limitations with regards to the "pipeACK Sampli= ng Period" mentioned in the Internet-Draft. Aris says this in his thesis = about what exactly he implemented: >>>>>>>>>> >>>>>>>>>> "The second implementation choice, is in regards with the meas= urement of pipeACK. This variable is the most important introduced by the= method and is used to compute the phase that the sender currently lies i= n. In order to compute pipeACK the approach suggested by the Internet Dra= ft (ID) is followed [ncwv]. During initialization, pipeACK is set to the = maximum possible value. A helper variable prevHighACK is introduced that = is initialized to the initial sequence number (iss). prevHighACK holds th= e value of the highest acknowledged byte so far. pipeACK is measured once= per RTT meaning that when an ACK covering prevHighACK is received, pipeA= CK becomes the difference between the current ACK and prevHighACK. This i= s called a pipeACK sample. A newer version of the draft suggests that mu= ltiple pipeACK samples can be used during the pipeACK sampling period." >>>>>>>>>> >>>>>>>>>> Lars >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> [prr.patch] >>>>>>>>>> >>>>>>>>>> [newcwv.patch] >>>>>>>>> >>>>>>>>> Apologies for not looking at this as yet. It is now closer to = the top of my list. >>>>>>>>> >>>>>>>>> Best, >>>>>>>>> George >>>>>>>> >>>>>>> >>>>>>> >>>>>>> >>>>>>> -- >>>>>>> Tom >>>>>>> @adventureloop >>>>>>> adventurist.me >>>>>>> >>>>>>> :wq >>>>>>> _______________________________________________ >>>>>>> freebsd-net@freebsd.org mailing list >>>>>>> http://lists.freebsd.org/mailman/listinfo/freebsd-net >>>>>>> To unsubscribe, send any mail to "freebsd-net-unsubscribe@freebsd= =2Eorg" >>>>> >>>>> >>> --=_MailMate_D05B247A-8A62-410E-A48B-05C2B800C34B_= Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename=signature.asc Content-Type: application/pgp-signature; name=signature.asc -----BEGIN PGP SIGNATURE----- Comment: GPGTools - http://gpgtools.org iEYEARECAAYFAlP/Wy0ACgkQYdh2wUQKM9KbKwCgvkUFUxesZm8kGHmy4xdijhfX AQEAoL7jyeWL0nu/+ewTdstPI5DMOcHG =x/qB -----END PGP SIGNATURE----- --=_MailMate_D05B247A-8A62-410E-A48B-05C2B800C34B_=--