[tcpm] [Technical Errata Reported] RFC8257 (6697)

RFC Errata System <rfc-editor@rfc-editor.org> Tue, 28 September 2021 07:18 UTC

Return-Path: <wwwrun@rfc-editor.org>
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 DAC323A21B2 for <tcpm@ietfa.amsl.com>; Tue, 28 Sep 2021 00:18:34 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.901
X-Spam-Level:
X-Spam-Status: No, score=-1.901 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
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 XnkXOFFI2cKr for <tcpm@ietfa.amsl.com>; Tue, 28 Sep 2021 00:18:30 -0700 (PDT)
Received: from rfc-editor.org (rfc-editor.org [4.31.198.49]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id AA6AE3A21B1 for <tcpm@ietf.org>; Tue, 28 Sep 2021 00:18:28 -0700 (PDT)
Received: by rfc-editor.org (Postfix, from userid 30) id BE0D7F40865; Tue, 28 Sep 2021 00:18:18 -0700 (PDT)
To: sbens@microsoft.com, dthaler@microsoft.com, pravb@microsoft.com, 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
X-PHP-Originating-Script: 1005:errata_mail_lib.php
From: RFC Errata System <rfc-editor@rfc-editor.org>
Cc: vidhi_goel@apple.com, tcpm@ietf.org, rfc-editor@rfc-editor.org
Content-Type: text/plain; charset=UTF-8
Message-Id: <20210928071818.BE0D7F40865@rfc-editor.org>
Date: Tue, 28 Sep 2021 00:18:18 -0700 (PDT)
Archived-At: <https://mailarchive.ietf.org/arch/msg/tcpm/rFF2Vxf7HuAmkYao01j_VHGxBfQ>
X-Mailman-Approved-At: Tue, 28 Sep 2021 08:02:09 -0700
Subject: [tcpm] [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: Tue, 28 Sep 2021 07:18:35 -0000

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

--------------------------------------
Type: Technical
Reported by: Vidhi Goel <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