Re: [tcpm] [EXTERNAL] Re: [Technical Errata Reported] RFC8257 (6697)

Praveen Balasubramanian <pravb@microsoft.com> Thu, 30 September 2021 16:07 UTC

Return-Path: <pravb@microsoft.com>
X-Original-To: tcpm@ietfa.amsl.com
Delivered-To: tcpm@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 752653A0D79 for <tcpm@ietfa.amsl.com>; Thu, 30 Sep 2021 09:07:18 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.452
X-Spam-Level:
X-Spam-Status: No, score=-2.452 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.452, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, HTML_MESSAGE=0.001, HTTPS_HTTP_MISMATCH=0.1, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=microsoft.com
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 6Vh4ScaowoG7 for <tcpm@ietfa.amsl.com>; Thu, 30 Sep 2021 09:07:13 -0700 (PDT)
Received: from na01-obe.outbound.protection.outlook.com (mail-oln040093003012.outbound.protection.outlook.com [40.93.3.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 386FF3A0D7C for <tcpm@ietf.org>; Thu, 30 Sep 2021 09:07:13 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ZjNk7sgpFSiluIouXRUSFAJcZujQcbYhColYsW/sO7xsVN026sL4LzTGIJEbGmZHUJvd8D/Xu7CZ4bUKz5hmGAIxn7RD57odj4sFbiIDuzE40t080GrQnhXJBneka4CZ8YNgzdHG9j/k2MwFYOHxHYAUvvdtbBNfLz512hd9Prlmotl7Z/DGQoMLh6f8RGbwK8vAoLX+8GAZkXa6jGzzpY69QY65OE2nvoHa6myrmBi6U4msWS/sxUD7YzxxXa1Uc/QFbarjyDjJCwTkc3yKdiVrAACqqg01xYbEn1XdkrDamtoGs82mFFl7dM8aGKhuQbWLhhf8MPuO9T/iUoz9sw==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=9M5oluvSqyEmiad6rPHNOn9v6bghABiYhlP8tXxrm4c=; b=MSLTQyZxVVStWUtwRIFAnyJSZWWnqxsxJhI30RHu9AiZmj3O9kciDcIODqC9V28hV5Ytn/T8Sdg76ci/sbU95QrOUtmCX7YEMPsIWy5kqjpIAQoOSA4HP5w2PSpaoJqzUJlYaEfDouzxNqXY93xGz/Lnkx1n6uzC5y4RiGekLcJ9dTQ7FESZYv2vlGGd2t21hMmNSnrU+LQQLtxFgZit4/gYo+61Xx8lDAsR0J4+R0xNJMlDHDHMeVFZYoIorJUbVrAQBTRLnaxZ3DYyGUCpB7h1wpD9vUl9AE+nzyf8OwxQDy61NO1Ivk8fVgEOcyYL63FSd8cQ8DQBKm3pyB1ofg==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microsoft.com; dmarc=pass action=none header.from=microsoft.com; dkim=pass header.d=microsoft.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=9M5oluvSqyEmiad6rPHNOn9v6bghABiYhlP8tXxrm4c=; b=TgjaJqVaqVtN6Veva+h2DPEWtaz877Cn3xZ0SwMZ/Gvd3SKQcCuAWGT+4hPN2WKXWODvDlU7A8giHAwepLgi+mFlAuHEfunrkHj8zsi7mS307MkY7tM5LnHwPTK7+ufrN6vPYjKhVUul69KLr6w/495xbZT91/pDhPAbc+VjMIU=
Received: from PH0PR00MB1032.namprd00.prod.outlook.com (2603:10b6:510:48::9) by PH0PR00MB1215.namprd00.prod.outlook.com (2603:10b6:510:9f::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4609.0; Thu, 30 Sep 2021 16:07:04 +0000
Received: from PH0PR00MB1032.namprd00.prod.outlook.com ([fe80::b15d:a862:9200:bff7]) by PH0PR00MB1032.namprd00.prod.outlook.com ([fe80::b15d:a862:9200:bff7%6]) with mapi id 15.20.4606.000; Thu, 30 Sep 2021 16:07:04 +0000
From: Praveen Balasubramanian <pravb@microsoft.com>
To: Neal Cardwell <ncardwell@google.com>, Bob Briscoe <in@bobbriscoe.net>
CC: RFC Errata System <rfc-editor@rfc-editor.org>, "sbens@microsoft.com" <sbens@microsoft.com>, Dave Thaler <dthaler@microsoft.com>, "lars@netapp.com" <lars@netapp.com>, "glenn.judd@morganstanley.com" <glenn.judd@morganstanley.com>, "martin.h.duke@gmail.com" <martin.h.duke@gmail.com>, "Zaheduzzaman.Sarker@ericsson.com" <Zaheduzzaman.Sarker@ericsson.com>, "Michael.Scharf@hs-esslingen.de" <Michael.Scharf@hs-esslingen.de>, "tuexen@fh-muenster.de" <tuexen@fh-muenster.de>, "nsd.ietf@gmail.com" <nsd.ietf@gmail.com>, "tcpm@ietf.org" <tcpm@ietf.org>
Thread-Topic: [EXTERNAL] Re: [tcpm] [Technical Errata Reported] RFC8257 (6697)
Thread-Index: AQHXtfFZSrjSllPiWEy87cXloZf7p6u8uBOAgAAEmCA=
Date: Thu, 30 Sep 2021 16:07:03 +0000
Message-ID: <PH0PR00MB1032DC9B26B2FE11EFC07481B6AA9@PH0PR00MB1032.namprd00.prod.outlook.com>
References: <20210928071818.BE0D7F40865@rfc-editor.org> <96ce4984-3678-9bdf-6b76-d7ba1bd42dcc@bobbriscoe.net> <CADVnQymMRzvs_4QRuSziYXfwu6ttKfak5cv5G=eBRvX8qOQKWw@mail.gmail.com>
In-Reply-To: <CADVnQymMRzvs_4QRuSziYXfwu6ttKfak5cv5G=eBRvX8qOQKWw@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
msip_labels: MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_ActionId=bae00bb1-caf8-4559-a613-4e998af02599; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_ContentBits=0; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Enabled=true; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Method=Standard; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Name=Internal; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SetDate=2021-09-30T16:00:23Z; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SiteId=72f988bf-86f1-41af-91ab-2d7cd011db47;
authentication-results: google.com; dkim=none (message not signed) header.d=none;google.com; dmarc=none action=none header.from=microsoft.com;
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 1a867206-b30d-454a-1fe5-08d9842c5831
x-ms-traffictypediagnostic: PH0PR00MB1215:
x-ms-exchange-transport-forked: True
x-microsoft-antispam-prvs: <PH0PR00MB12156FF3E505739B460D9BD8B6AA9@PH0PR00MB1215.namprd00.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:2089;
x-ms-exchange-senderadcheck: 1
x-ms-exchange-antispam-relay: 0
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: cKVVHatVhDG5j7pPGW29bkxb00beahRh4q6ssHZlSz2o+QI8K9zz8CO9ibEFjhDSwIXJBKMoBbyS93l34R1uWieFN0l2oi3g4iYYiFLlc/H6W1Yz/Cv8y9p+hrZgHsZIz8T9MTAwIH3IXqxlV5Aq30n3oc/pB9Q1UGn3WsdE9ADAYaiZruNqbFO1pQXL6xt9apIpA4sGGZixokAuK+X3RihN2SIvxNDWQ+LvFUJgswDVYbUfcDfNsSRI5KVRZkm6SWJU05X/SsxoGbR7pmwz09OuDqjfszBKPd3KPv1V44V4N9oKIoaLxrvTHHBZw/+KAsi31U4eQMtdZU+4H6m+ppPnAVIE3OYrIvtaywglUu09noKO4y5M5SceBcdAoT2IF+gxH1z5yI6HQJ+EWa2kv5OlDhkxat258As7tJKuA9RAcDWbZxeZCieEzU/CsGMiEUzNB5AYvpRJs9oJNfZXNljuTdoz71MzeGNZ7f9p1LaZ9fpW4iYacYme+K4M9qbNrakkoqzlka1SeDBX7Nu+8BOvdTp+CztyN7LHAng3oDn3r+IJbjaQHagpMsh72xzZWFfZ4DEuiAqibW2mimv2pcuyjEOVNoZWWH3zZ0hLdXph9fTIjvB47yuWIqwAiyQbTF2G2hIiuPnEHIxklHTeWkprrHYotMg/EbtOO1DNmFH/xVRaQsDPAkSi5LwyW0ZUt3sdFbsnL/zeNcjUAndXcRFGjkVoGePtNRyYxAgIO8W7LBxTBghgdKyNQvwMQZAzBugE8c3dIUf513XyWmT8tOPxGHV3yr1bJRY8n+8wOPS0JhUDQ5gR/diLNMS8ub4p+ziFgLnxfrfOqDPo60WHmQ==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PH0PR00MB1032.namprd00.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(366004)(5660300002)(33656002)(316002)(55016002)(38100700002)(122000001)(2906002)(8936002)(7416002)(86362001)(52536014)(38070700005)(8676002)(54906003)(9686003)(8990500004)(110136005)(508600001)(66556008)(66476007)(64756008)(66446008)(7696005)(66946007)(6506007)(53546011)(83380400001)(76116006)(966005)(186003)(166002)(10290500003)(82950400001)(4326008)(82960400001)(71200400001); DIR:OUT; SFP:1102;
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?3+pHQ2g1cZuPy0FD8WredRxnAkJINLsC5g9Q+KFn+GbpxneYUXkyK48bMtVa?= =?us-ascii?Q?Gk4pMhQLqHDHLF5qT5ILtyJt/99VPynImjTq7CLQ8YZlnsH1swndDg/HqUiE?= =?us-ascii?Q?/36gmefSUaReclocGybcXylkfLPzcXHdFYGEe8zLe8Uj2vTFsRT9/7/USPo5?= =?us-ascii?Q?MyjZCmP2Bw4thtpMbHjUc2ZjUYxbB8LclEveObbGXsuPo2q6gRTP+nDr+qLu?= =?us-ascii?Q?bH8QHlAKWkM3muRqXlYZTwOVrAMSRWLGUrMe8i8eC4u6LZJwDxNlDSC8608x?= =?us-ascii?Q?NLhvJUjFl6L/z6ziBwdgU2gR25ooD19Nk4uQbu05M5h1pUjpwT61pLeZfegJ?= =?us-ascii?Q?n2tMMncnBSj9zvza7xB+bP8Su6jOgVnswh4kW1CP/C3TCZSGHrrq0qotnvUT?= =?us-ascii?Q?TGtxT271er1XxUngPvaypeWrBm1P4HtnooWvejjuC3EJyeyWYvZMT0cmynbh?= =?us-ascii?Q?B7uIrfS8J7w2LUscgZcpURfoCYHH0J5kIAIA0l22kn08Bd5hUShD4/vEbH3C?= =?us-ascii?Q?5bYKmGMcOCjP8hcS/u6hh8ELFSYRwVu7//wfHXIJpWpnvsNoEsh8OKpCHAmY?= =?us-ascii?Q?M4FLoxCQ4u3luW5HDuXCK0QQcVSGecDDowAOwqxKpVu3quioutNGC5D+n5/6?= =?us-ascii?Q?PYP5LiTUWP9Q68iZZ+vS2ZWL9hLJArtU2slZ3SmQ4z7YdQW0GLsoO/2kfa/n?= =?us-ascii?Q?0FLz2vr8H1PQPH2WOQbipfjBs9lgccEzNJSmOUWxvVF0yLJbva2Q5NbN8Bhl?= =?us-ascii?Q?Pq2ShgL2Z2J/UWsXDf7GhmK61eaV6xFDY1tMtCTOEwcZnFFDFhrjynE/SKGO?= =?us-ascii?Q?0Kp6xpYdviDx7t2ibIKVSl/JCxvYxsKTfJS0prrefcrQpoKLlYo8OGn82Azw?= =?us-ascii?Q?gqm2mLUbrqRSmjX2AB580RjDSoEHXvhLdWvOF8nKzDnuXZHAzlqY5k999oQO?= =?us-ascii?Q?huWp90WQeYeikdDL7UiUCU/CiXqMW60gz+YY71cRygaeEokytSdiIyzbCbi8?= =?us-ascii?Q?u77WDazUOo2Z5hQT95sYdPoJDUGL9LBz3BJV7cwrLG18Yi+RacTOmbLRdxYK?= =?us-ascii?Q?FKCpxgPs88DLpXnshqPCC3sNOKkQ7CEQoIQiQnie/iTMCGI5TM3RjAc5u5jv?= =?us-ascii?Q?f2M0+vnRkP6GmGjT3jSmk4IUAUBN1AX2jtQEvq4YzGyPZkFyAea94gNmLUPk?= =?us-ascii?Q?3H1li2ckzvCIBZsMTjhKXaC4bTmO+RD+nOobcIzWdyeR+Fup8vPgeo/iQpyc?= =?us-ascii?Q?CqugPMMpUqV89efKuTosco5PbdKncsGo01YFbC73la4RbFTUDlsSsX4usSCG?= =?us-ascii?Q?/H1s/54i90T66SamoNbNAiIGhFysI5xmq5ZcE1/9f5Ps62SEydMw6hvkYCAJ?= =?us-ascii?Q?98K3SnlTZxZvAFw5Igq6iPBP5ut/?=
Content-Type: multipart/alternative; boundary="_000_PH0PR00MB1032DC9B26B2FE11EFC07481B6AA9PH0PR00MB1032namp_"
MIME-Version: 1.0
X-OriginatorOrg: microsoft.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PH0PR00MB1032.namprd00.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 1a867206-b30d-454a-1fe5-08d9842c5831
X-MS-Exchange-CrossTenant-originalarrivaltime: 30 Sep 2021 16:07:04.0596 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 72f988bf-86f1-41af-91ab-2d7cd011db47
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 7aQ1VW055SthK+nH0rF8VBTqnDuddMjRtg1D/ZQdwqIegO5AJ5sF3Uzg3nRlAwhaaufZ4nGI+V5fPJD/usXS/KL+/K3tNgj/qNMV1UmAx6g=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR00MB1215
Archived-At: <https://mailarchive.ietf.org/arch/msg/tcpm/azGw4FpaBVTgNgYu6JPXjjk2XCk>
X-Mailman-Approved-At: Thu, 30 Sep 2021 14:33:36 -0700
Subject: Re: [tcpm] [EXTERNAL] Re: [Technical Errata Reported] RFC8257 (6697)
X-BeenThere: tcpm@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: TCP Maintenance and Minor Extensions Working Group <tcpm.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tcpm>, <mailto:tcpm-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/tcpm/>
List-Post: <mailto:tcpm@ietf.org>
List-Help: <mailto:tcpm-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tcpm>, <mailto:tcpm-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 30 Sep 2021 16:07:19 -0000

Great catch Vidhi!

We don't need to change the section numbers. ECE is processed on acceptable ACK packets and there is already another bullet 3 which specifies that condition:

3.  If the ECE flag is set, update the bytes marked:

          DCTCP.BytesMarked += BytesAcked

So all we need is:

  9.  If the ECE flag is set, rather than always halving the congestion window as described in
       [RFC3168<https://datatracker.ietf.org/doc/html/rfc3168>]8>], the sender SHOULD update cwnd as follows:

          cwnd = cwnd * (1 - DCTCP.Alpha / 2)


I don't think we need the additional explanation of how to implement the once per window reaction, even RFC 3168 does not specify those details.

From: Neal Cardwell <ncardwell@google.com>
Sent: Thursday, September 30, 2021 8:44 AM
To: Bob Briscoe <in@bobbriscoe.net>
Cc: RFC Errata System <rfc-editor@rfc-editor.org>rg>; sbens@microsoft.com; Dave Thaler <dthaler@microsoft.com>om>; Praveen Balasubramanian <pravb@microsoft.com>om>; lars@netapp.com; glenn.judd@morganstanley.com; martin.h.duke@gmail.com; Zaheduzzaman.Sarker@ericsson.com; Michael.Scharf@hs-esslingen.de; tuexen@fh-muenster.de; nsd.ietf@gmail.com; tcpm@ietf.org
Subject: [EXTERNAL] Re: [tcpm] [Technical Errata Reported] RFC8257 (6697)

I agree with the points made by Vidhi and Bob, and really like Bob's text.

In the suggested text there may be a typo; I believe we want s/SND.UNA/SND.NXT/. And probably we want to be more specific about only suppressing further ECN-based reductions (further loss-triggered reductions would be good to allow). I'm posting my suggested tweaks in blue, starting from Bob's nice green text:

SUGGESTED:
==========



3.4. Congestion Window Reduction

   Rather than always halving the congestion window as described in

   [RFC3168], on the arrival of ECN congestion feedback, the sender SHOULD

   update cwnd as follows:



      cwnd = cwnd * (1 - DCTCP.Alpha / 2)

   Just as specified in [RFC3168], DCTCP does not react to congestion

   indications more than once for every window of data. Therefore, as

   for RFC3168 ECN, it sets the variable for the end of congestion

   window reduced (CWR) state to SND.NXT and suppresses further

   ECN-triggered reductions until this TCP sequence number is acknowledged. Periods

   of CWR state are triggered by congestion feedback, and therefore

   occur at times unrelated to the continuous cycle of observation

   windows used to update DCTCP.Alpha in Section 3.3.



   The setting of the CWR bit is also as per [RFC3168].  This is

   required for interoperation with classic ECN receivers due to

   potential misconfigurations.



3.5.  Handling of Congestion Window Growth...

neal


On Thu, Sep 30, 2021 at 11:22 AM Bob Briscoe <in@bobbriscoe.net<mailto:in@bobbriscoe.net>> wrote:
Vidhi,

You're right. It's incorrect to have the window reduction hanging off the end of the list of steps for updating the EWMA.

To make this concrete, here's some specific additional text (in green for those with HTML mail readers). Also, rather than splitting into sub-subsections, I have suggested that Item 9. of the list in subsection 3.3 is moved out of the list, and instead forms the basis of a new subsection 3.4. entitled "Congestion Window Reduction".

CURRENT:
========

   9.  Rather than always halving the congestion window as described in

       [RFC3168], the sender SHOULD update cwnd as follows:



          cwnd = cwnd * (1 - DCTCP.Alpha / 2)



   Just as specified in [RFC3168], DCTCP does not react to congestion

   indications more than once for every window of data.  The setting of

   the CWR bit is also as per [RFC3168].  This is required for

   interoperation with classic ECN receivers due to potential

   misconfigurations.



3.4.  Handling of Congestion Window Growth...


SUGGESTED:
==========

3.4. Congestion Window Reduction

   Rather than always halving the congestion window as described in

   [RFC3168], on the arrival of congestion feedback, the sender SHOULD

   update cwnd as follows:



      cwnd = cwnd * (1 - DCTCP.Alpha / 2)

   Just as specified in [RFC3168], DCTCP does not react to congestion

   indications more than once for every window of data. Therefore, as

   for RFC3168 ECN, it sets the variable for the end of congestion

   window reduced (CWR) state to SND.UNA and suppresses further

   reductions until this TCP sequence number is acknowledged. Periods

   of CWR state are triggered by congestion feedback, and therefore

   occur at times unrelated to the continuous cycle of observation

   windows used to update DCTCP.Alpha in Section 3.3.



   The setting of the CWR bit is also as per [RFC3168].  This is

   required for interoperation with classic ECN receivers due to

   potential misconfigurations.



3.5.  Handling of Congestion Window Growth...

Then the of numbering all subsequent subsections of section 3. will increment by 0.1.



Bob
On 28/09/2021 08:18, RFC Errata System wrote:

The following errata report has been submitted for RFC8257,

"Data Center TCP (DCTCP): TCP Congestion Control for Data Centers".



--------------------------------------

You may review the report below and at:

https://www.rfc-editor.org/errata/eid6697<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.rfc-editor.org%2Ferrata%2Feid6697&data=04%7C01%7Cpravb%40microsoft.com%7C1bfca5149bd24bc83e3608d9842927dc%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637686134599340937%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=vA5l%2Be1sLhWo8%2BRg9yrBlahGry2U2rsQwjkEB0hKrUo%3D&reserved=0>



--------------------------------------

Type: Technical

Reported by: Vidhi Goel <vidhi_goel@apple.com><mailto:vidhi_goel@apple.com>



Section: 3.3



Original Text

-------------

The below pseudocode follows after DCTCP.Alpha is updated on ACK processing. This is wrong as cwnd should only be reduced using DCTCP.Alpha when ECE is received.



9. Rather than always halving the congestion window as described in

       [RFC3168], the sender SHOULD update cwnd as follows:



          cwnd = cwnd * (1 - DCTCP.Alpha / 2)



Corrected Text

--------------

Instead, a new paragraph for Congestion Response to ECN feedback would be much clearer. First start with RFC 3168's response to ECE and then provide DCTCP's response to ECE.



I am thinking splitting section 3.3 into two sub-sections -

3.3.1 Computation of DCTCP.Alpha

3.3.2 Congestion Response to ECE at sender







Notes

-----

Although RFC 8257 refers to RFC 3168 congestion window halving at step 9, but it is confusing to put it right after step 8.



Instructions:

-------------

This erratum is currently posted as "Reported". If necessary, please

use "Reply All" to discuss whether it should be verified or

rejected. When a decision is reached, the verifying party

can log in to change the status and edit the report, if necessary.



--------------------------------------

RFC8257 (draft-ietf-tcpm-dctcp-10)

--------------------------------------

Title               : Data Center TCP (DCTCP): TCP Congestion Control for Data Centers

Publication Date    : October 2017

Author(s)           : S. Bensley, D. Thaler, P. Balasubramanian, L. Eggert, G. Judd

Category            : INFORMATIONAL

Source              : TCP Maintenance and Minor Extensions

Area                : Transport

Stream              : IETF

Verifying Party     : IESG



_______________________________________________

tcpm mailing list

tcpm@ietf.org<mailto:tcpm@ietf.org>

https://www.ietf.org/mailman/listinfo/tcpm<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.ietf.org%2Fmailman%2Flistinfo%2Ftcpm&data=04%7C01%7Cpravb%40microsoft.com%7C1bfca5149bd24bc83e3608d9842927dc%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637686134599350894%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=nfQWrs6AQILOXyUMDakUfjbLZ4rO%2FxPboBCt%2F3oAg9A%3D&reserved=0>



--

________________________________________________________________

Bob Briscoe                               http://bobbriscoe.net/<https://nam06.safelinks.protection.outlook.com/?url=http%3A%2F%2Fbobbriscoe.net%2F&data=04%7C01%7Cpravb%40microsoft.com%7C1bfca5149bd24bc83e3608d9842927dc%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637686134599350894%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=%2B7pyxiEVcF4tkQ3G5IDlVmUKvpG7Tfs5HumS71Pt1Qs%3D&reserved=0>
_______________________________________________
tcpm mailing list
tcpm@ietf.org<mailto:tcpm@ietf.org>
https://www.ietf.org/mailman/listinfo/tcpm<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.ietf.org%2Fmailman%2Flistinfo%2Ftcpm&data=04%7C01%7Cpravb%40microsoft.com%7C1bfca5149bd24bc83e3608d9842927dc%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637686134599360856%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=vlPWpQfej2%2BcLgrJqWMLnoNFeJ%2BtFdKaw%2BR4%2BQPtJOM%3D&reserved=0>