From owner-freebsd-net@FreeBSD.ORG Wed Aug 27 08:01:54 2014 Return-Path: Delivered-To: freebsd-net@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 1EFC854C; Wed, 27 Aug 2014 08:01:54 +0000 (UTC) Received: from mx11.netapp.com (mx11.netapp.com [216.240.18.76]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (Client CN "mx11.netapp.com", Issuer "VeriSign Class 3 International Server CA - G3" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id E5AFD30F6; Wed, 27 Aug 2014 08:01:53 +0000 (UTC) X-IronPort-AV: E=Sophos;i="5.04,409,1406617200"; d="asc'?scan'208";a="142517018" Received: from vmwexceht06-prd.hq.netapp.com ([10.106.77.104]) by mx11-out.netapp.com with ESMTP; 27 Aug 2014 01:01:52 -0700 Received: from HIOEXCMBX06-PRD.hq.netapp.com (10.122.105.39) by vmwexceht06-prd.hq.netapp.com (10.106.77.104) with Microsoft SMTP Server (TLS) id 14.3.123.3; Wed, 27 Aug 2014 01:01:30 -0700 Received: from HIOEXCMBX07-PRD.hq.netapp.com (10.122.105.40) by hioexcmbx06-prd.hq.netapp.com (10.122.105.39) with Microsoft SMTP Server (TLS) id 15.0.913.22; Wed, 27 Aug 2014 01:01:28 -0700 Received: from HIOEXCMBX07-PRD.hq.netapp.com ([::1]) by hioexcmbx07-prd.hq.netapp.com ([fe80::55e3:a7dc:11bd:462%21]) with mapi id 15.00.0913.011; Wed, 27 Aug 2014 01:01:16 -0700 From: "Eggert, Lars" To: Adrian Chadd Subject: Re: Patches for RFC6937 and draft-ietf-tcpm-newcwv-00 Thread-Topic: Patches for RFC6937 and draft-ietf-tcpm-newcwv-00 Thread-Index: AQHPIYzc3HaTSw//+U6b0HMYTUDKSZt6PvOAgGprrICAAAMkgIAANjaAgADd6ICAAASAgIAAAQaAgAAC2YCAAAI+AA== Date: Wed, 27 Aug 2014 08:01:15 +0000 Message-ID: <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> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: yes X-MS-TNEF-Correlator: x-mailer: Apple Mail (2.1878.6) x-originating-ip: [10.122.56.79] Content-Type: multipart/signed; boundary="Apple-Mail=_20BD719E-DC1D-49F8-AE38-EC46D310C47C"; protocol="application/pgp-signature"; micalg=pgp-sha1 MIME-Version: 1.0 Cc: Tom Jones , 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: Wed, 27 Aug 2014 08:01:54 -0000 --Apple-Mail=_20BD719E-DC1D-49F8-AE38-EC46D310C47C Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=us-ascii Yep On 2014-8-27, at 9:53, Adrian Chadd wrote: > Ok. Is it the same patch you sent out in Feb? >=20 >=20 > -a >=20 >=20 > On 27 August 2014 00:43, Eggert, Lars wrote: >> Not as far as I know. >>=20 >> Lars >>=20 >> On 2014-8-27, at 9:39, Adrian Chadd wrote: >>=20 >>> Is there a PR for it? >>>=20 >>>=20 >>> -a >>>=20 >>>=20 >>> On 27 August 2014 00:23, Eggert, Lars wrote: >>>> It would be great if people could also review Aris' PRR patch - = RFC6937 has been out for a while. >>>>=20 >>>> Lars >>>>=20 >>>>=20 >>>>=20 >>>>=20 >>>> On 2014-8-26, at 20:09, Adrian Chadd wrote: >>>>=20 >>>>> Hi! >>>>>=20 >>>>> I'm going to merge Tom's work in a week unless someone gives me a >>>>> really good reason not to. >>>>>=20 >>>>> I think there's been enough work and discussion about it since the >>>>> first post from Lars in Feburary and enough review opportunity. >>>>>=20 >>>>>=20 >>>>> -a >>>>>=20 >>>>>=20 >>>>> On 26 August 2014 07:55, Tom Jones wrote: >>>>>> On Tue, Aug 26, 2014 at 02:43:49PM +0000, Eggert, Lars wrote: >>>>>>> Hi, >>>>>>>=20 >>>>>>> the newcwv patch is probably stale now with Tom Jones' recent = patch based on >>>>>>> a more up-to-date version of the Internet-Draft, but the PRR = patch should >>>>>>> still be useful? >>>>>>=20 >>>>>> My newcwv patch is much more up to date than Aris's, but it is = slightly >>>>>> different in implementation. I have had a few suggestions from = Adrian, but he >>>>>> couldn't comment on how it relates to the tcp internals. >>>>>>=20 >>>>>> There is a PR: = https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D191520 >>>>>>=20 >>>>>> 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. >>>>>>=20 >>>>>>> On 2014-6-19, at 23:35, George Neville-Neil = wrote: >>>>>>>=20 >>>>>>>> On 4 Feb 2014, at 1:38, Eggert, Lars wrote: >>>>>>>>=20 >>>>>>>>> Hi, >>>>>>>>>=20 >>>>>>>>> below are two patches that implement RFC6937 ("Proportional = Rate Reduction for TCP") and draft-ietf-tcpm-newcwv-00 ("Updating TCP to = support Rate-Limited Traffic"). They were done by Aris = Angelogiannopoulos for his MS thesis, which is at = https://eggert.org/students/angelogiannopoulos-thesis.pdf. >>>>>>>>>=20 >>>>>>>>> The patches should apply to -CURRENT as of Sep 17, 2013. = (Sorry for the delay in sending them, we'd been trying to get some = feedback from committers first, without luck.) >>>>>>>>>=20 >>>>>>>>> Please note that newcwv is still a work in progress in the = IETF, and the patch has some limitations with regards to the "pipeACK = Sampling Period" mentioned in the Internet-Draft. Aris says this in his = thesis about what exactly he implemented: >>>>>>>>>=20 >>>>>>>>> "The second implementation choice, is in regards with the = measurement of pipeACK. This variable is the most important introduced = by the method and is used to compute the phase that the sender currently = lies in. In order to compute pipeACK the approach suggested by the = Internet Draft (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 the value of the highest acknowledged byte so far. = pipeACK is measured once per RTT meaning that when an ACK covering = prevHighACK is received, pipeACK becomes the difference between the = current ACK and prevHighACK. This is called a pipeACK sample. A newer = version of the draft suggests that multiple pipeACK samples can be used = during the pipeACK sampling period." >>>>>>>>>=20 >>>>>>>>> Lars >>>>>>>>>=20 >>>>>>>>>=20 >>>>>>>>> [prr.patch] >>>>>>>>>=20 >>>>>>>>> [newcwv.patch] >>>>>>>>=20 >>>>>>>> Apologies for not looking at this as yet. It is now closer to = the top of my list. >>>>>>>>=20 >>>>>>>> Best, >>>>>>>> George >>>>>>>=20 >>>>>>=20 >>>>>>=20 >>>>>>=20 >>>>>> -- >>>>>> Tom >>>>>> @adventureloop >>>>>> adventurist.me >>>>>>=20 >>>>>> :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.org" >>>>=20 >>>>=20 >>=20 --Apple-Mail=_20BD719E-DC1D-49F8-AE38-EC46D310C47C Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="signature.asc" Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Message signed with OpenPGP using GPGMail -----BEGIN PGP SIGNATURE----- iQCVAwUBU/2QYNZcnpRveo1xAQJNVQP8CpkjY1XgwWGPKnCOcgkEsAR7Qu2LAEkE apAYWU++tyfB9m21lMqlqnZsiwIPb4VlTSfTv0g3q/qxPKsCwXLk9I4NTkt1/e3l s7yuWzPA+CWCWdqVF2nFLJAs2odDjXG0aAWY/qkSGQefHqStCL8XgQqM6Jt9NPmz doKQ2fT4XBU= =PUXd -----END PGP SIGNATURE----- --Apple-Mail=_20BD719E-DC1D-49F8-AE38-EC46D310C47C--