Re: [tcpm] Comments on draft-ietf-tcpm-accurate-ecn

Bob Briscoe <ietf@bobbriscoe.net> Wed, 11 July 2018 22:52 UTC

Return-Path: <ietf@bobbriscoe.net>
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 98EF4130EAF; Wed, 11 Jul 2018 15:52:54 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.001
X-Spam-Level:
X-Spam-Status: No, score=-2.001 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=bobbriscoe.net
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 d0Ac78xN4VnR; Wed, 11 Jul 2018 15:52:50 -0700 (PDT)
Received: from server.dnsblock1.com (server.dnsblock1.com [85.13.236.178]) (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 23486130E5D; Wed, 11 Jul 2018 15:52:49 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=bobbriscoe.net; s=default; h=Content-Transfer-Encoding:Content-Type: In-Reply-To:MIME-Version:Date:Message-ID:From:References:Cc:To:Subject:Sender :Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help: List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=w+TG7jZXdE69mN0idnRyW/vJnXvvYFsbmrya89ZaQco=; b=ScbKh4+rjku1aK+8leLf4HDeXl wHmXhBPBAVip3BoWS9wVZEACplW5sFC7HtixjgR7KUM40sEp/p6/ThtVDQ0JanXH+7Tq7dr/prlQK vkkBtJUpyuGvgqOae0xrgQc9C1FocDzY8cBaWeluUhE0c6vj7iOBN1tq6mPpowTMUFTOf1m4iVRen P9QoEkbqAitgRaoPmUC+9wYPXYu49+6xTpKPEW3X87DPffwlYtU9+9ot5ogF8Ku6ZK80bp/zBcIts TzBMa8DkSSFkJxuWQxwBjTRvDJGv/3GTcaZ2nYHzyfD1lzIqIiI3PSNgP8UPmphaSB4rWkozcPfny d/mo6VOA==;
Received: from 70.245.199.146.dyn.plus.net ([146.199.245.70]:51134 helo=[192.168.0.4]) by server.dnsblock1.com with esmtpsa (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128) (Exim 4.91) (envelope-from <ietf@bobbriscoe.net>) id 1fdNyZ-0000Ub-1D; Wed, 11 Jul 2018 23:52:47 +0100
To: Yuchung Cheng <ycheng@google.com>
Cc: "Scharf, Michael (Nokia - DE/Stuttgart)" <michael.scharf@nokia.com>, "draft-ietf-tcpm-accurate-ecn@ietf.org" <draft-ietf-tcpm-accurate-ecn@ietf.org>, "tcpm@ietf.org" <tcpm@ietf.org>
References: <AM5PR0701MB25477BD5BEB403A98AA2B983933F0@AM5PR0701MB2547.eurprd07.prod.outlook.com> <44FDECF5-A031-4343-BA1A-AE0D9C2C078C@tik.ee.ethz.ch> <VI1PR0701MB2558F5DE5FCE5CDC6A43F94793D30@VI1PR0701MB2558.eurprd07.prod.outlook.com> <E729457B-96C5-493D-9B14-70663C24DFB4@tik.ee.ethz.ch> <db66271d-3654-6066-fecc-a405bb88b7f5@bobbriscoe.net> <CAK6E8=dkuyD+PJv9+4iwdXNu0pEv8n59acHx1Q-yBeCBQ=CcEg@mail.gmail.com>
From: Bob Briscoe <ietf@bobbriscoe.net>
Message-ID: <07dacfdf-232a-b82e-7f41-fa5ce1f1853a@bobbriscoe.net>
Date: Wed, 11 Jul 2018 23:52:46 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0
MIME-Version: 1.0
In-Reply-To: <CAK6E8=dkuyD+PJv9+4iwdXNu0pEv8n59acHx1Q-yBeCBQ=CcEg@mail.gmail.com>
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Transfer-Encoding: 8bit
Content-Language: en-GB
X-AntiAbuse: This header was added to track abuse, please include it with any abuse report
X-AntiAbuse: Primary Hostname - server.dnsblock1.com
X-AntiAbuse: Original Domain - ietf.org
X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12]
X-AntiAbuse: Sender Address Domain - bobbriscoe.net
X-Get-Message-Sender-Via: server.dnsblock1.com: authenticated_id: in@bobbriscoe.net
X-Authenticated-Sender: server.dnsblock1.com: in@bobbriscoe.net
Archived-At: <https://mailarchive.ietf.org/arch/msg/tcpm/dG8jrC23wkfmSi-iFLWwJwvFr0A>
Subject: Re: [tcpm] Comments on draft-ietf-tcpm-accurate-ecn
X-BeenThere: tcpm@ietf.org
X-Mailman-Version: 2.1.27
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: Wed, 11 Jul 2018 22:52:55 -0000

Yuchung,

On 11/07/18 19:35, Yuchung Cheng wrote:
> Hi Bob,
>
> Neal and I evaluated the earlier draft. It is well-thought out but
> we're concerned about the options. Option is not mandatory but the
> lack of it also reduces accuracy. Option runs into space issues w/
> SACK and offload issues w/ TSO/GRO. They can be addressed for sure but
> aren't easy.
We had various ideas for smart schemes to reduce the size of the option 
(the fields for the counters don't always need to be so wide and you 
often only need one field). However, we ended up opting for simplicity 
at the expense of space. If you're interested, we could dig out the 
ideas we had (between the co-authors) and see if you would prefer fewer 
option bytes but less simplicity.

I assume you saw the rules around when an option can be suppressed. 
Search for the text around "Receiver MUST give precedence to SACK 
information about loss."

Is there a specific issue with the option and TSO/GRO, or just that the 
h/w wouldn't understand what to do with this option?

> We're curious how much more "accuracy" it buys over current
> DCTCP-style ECN. Is there any study to show trade-offs of
> full-ACE-w-options vs ACE-wo-options vs current DCTCP-ECN?
There's a brief summary of the problem with DCTCP-style feedback in the 
appendix of the AccECN Requirements RFC here:
https://tools.ietf.org/html/rfc7560#appendix-A

Essentially, the DCTCP scheme relies on the sender using the feedback to 
keep its copy of the receiver's state machine in sync, which can get out 
of sync in the presence of pure ACK losses and therefore become highly 
ambiguous. Even if there are no ACK losses for a while, it can look like 
a gap could have been caused by a loss - the sender cannot guess what 
might have happened.

The state machine gets back into sync once losses stop. However, DCTCP 
feedback is change-triggered so the sender cannot predict which ACKs it 
should receive and which not. That combines with 2 potential reasons for 
a gap in the ACK stream: delayed ACK; or a lost pure ACK. All three 
factors combined cause considerable ambiguity.

Bob

>
>
> On Wed, Jul 11, 2018 at 11:00 AM, Bob Briscoe <ietf@bobbriscoe.net> wrote:
>> Michael, tcpm list,
>>
>> As well as addressing your points, as Mirja has already mentioned below, we
>> added a whole new appendix giving the rationale for the bits and codepoints
>> that AccECN has proposed to use on 1) the SYN and 2) SYN/ACK. A 3rd
>> subsection also identifies space for future evolution. It also points to
>> where rationale was already given in the body of the draft.
>>
>> The appendix is in the draft submitted last week, available here:
>> https://tools.ietf.org/html/draft-ietf-tcpm-accurate-ecn-07#appendix-B
>>
>> We'd be interested to hear whether this allays your concerns.
>>
>> We have asked to present this in Montreal as well.
>>
>> Cheers
>>
>>
>> Bob
>>
>>
>> On 02/07/18 16:54, Mirja Kühlewind wrote:
>>> Hi Micheal,
>>>
>>> I addressed a couple of your comments below.
>>>
>>> For the other, bigger comments regarding extensibility, that I did not yet
>>> address below, we plan to add a new section to the appendix to explain
>>> extensibility options as previously discussed by mail. We will probably send
>>> a separate email on that part.
>>>
>>> Mirja
>>>
>>>
>>>> Am 12.03.2018 um 01:59 schrieb Scharf, Michael (Nokia - DE/Stuttgart)
>>>> <michael.scharf@nokia.com>:
>>>>
>>>> Hi Mirja,
>>>>
>>>> Thanks a lot for the explanation. I won't follow-up on some of the
>>>> editorial suggestions.
>>>>
>>>> Yet, I continue to believe that some formal wording in the document needs
>>>> to change, as explained below.
>>>>
>>>> Thanks
>>>>
>>>> Michael (with no hat on)
>>>>
>>>>
>>>>> -----Original Message-----
>>>>> From: Mirja Kühlewind [mailto:mirja.kuehlewind@tik.ee.ethz.ch]
>>>>> Sent: Monday, March 05, 2018 1:54 PM
>>>>> To: Scharf, Michael (Nokia - DE/Stuttgart) <michael.scharf@nokia.com>
>>>>> Cc: draft-ietf-tcpm-accurate-ecn@ietf.org; tcpm@ietf.org
>>>>> Subject: Re: Comments on draft-ietf-tcpm-accurate-ecn
>>>>>
>>>>> Hi Micheal,
>>>>>
>>>>> thanks for your feedback and sorry for my late reply.
>>>>>
>>>>> Please see inline.
>>>>>
>>>>>> Am 03.12.2017 um 20:17 schrieb Scharf, Michael (Nokia - DE/Stuttgart)
>>>>> <michael.scharf@nokia.com>:
>>>>>> Hi all,
>>>>>>
>>>>>> I have read draft-ietf-tcpm-accurate-ecn-05 (without the appendix). I
>>>>> believe this document needs further work before moving forward.
>>>>>> Please find below my comments marked as [ms]. I have read the
>>>>> document independent of the review from Gorry. I apologize if there is
>>>>> duplication.
>>>>>> Thanks
>>>>>>
>>>>>> Michael (with no hat on)
>>>>>>
>>>>>>
>>>>>> ******************************
>>>>>>
>>>>>> * Abstract:
>>>>>>
>>>>>>     Recently, new TCP mechanisms like Congestion Exposure (ConEx) or
>>>>>> Data
>>>>> Center TCP
>>>>>>     (DCTCP) need more accurate ECN feedback information whenever more
>>>>>>     than one marking is received in one RTT.
>>>>>>
>>>>>> [ms] I don't think this statement is fully backed by RFC 8257. I
>>>>>> suggest to
>>>>> remove this, or replace it by a more generic statement that more
>>>>> accurate
>>>>> information can be useful for several TCP extensions.
>>>>>
>>>>> I disagree. Both ConEx and DCTCP need more accurate information. They do
>>>>> not need the mechanism that is specified in this draft, however, this is
>>>>> not
>>>>> what the sentences is saying.
>>>> In my understanding (as a non-native speaker), the use of the word "need"
>>>> is not correct here. DCTCP as specified in RFC 8257 can be implemented
>>>> without any such mechanism.
>>>>
>>>> What would work for me is something of the form "... Data Center TCP
>>>> cannot get precise ECN feedback whenever more than one marking is received
>>>> in one RTT“.
>>> This is not correct. DCTP need more than one feedback signal per RTT and
>>> therefore cannot use RFC3168; instead it implement it’s own feedback
>>> mechanism. However, to avoid confusion such that people could assume DCTP
>>> would not work without the accECN scheme as specified in this doc, I
>>> rephrased to:
>>>
>>> "Recently, proposed
>>>         mechanisms like Congestion Exposure (ConEx <xref
>>> target="RFC7713"/>),
>>>         DCTCP <xref target="RFC8257"/> or L4S <xref
>>>         target="I-D.ietf-tsvwg-l4s-arch"/> need to know when more than one
>>>         marking is received in one RTT which is
>>>         information that cannot be provided by the feedback scheme as
>>> specified in
>>>         <xref target="RFC3168"/>."
>>>
>>>
>>>>>>     This document specifies an
>>>>>>     experimental scheme to provide more than one feedback signal per RTT
>>>>>>     in the TCP header.  Given TCP header space is scarce, it overloads
>>>>>>     the three existing ECN-related flags in the TCP header and provides
>>>>>>     additional information in a new TCP option.
>>>>>>
>>>>>> [ms] This statement needs to be rewritten to correctly reflect what is
>>>>> requested from IANA. My understanding is that this experimental document
>>>>> asks for allocation of a reserved TCP header flag. This needs to be
>>>>> called out
>>>>> prominently, IMHO. In addition, since this is not a standard, the
>>>>> suggested
>>>>> experimentation with the main TCP header must IMHO be explicitly
>>>>> mentioned. I also suggest to have later in a document a section that
>>>>> explicitly
>>>>> explains why it is appropriate to modify the main TCP header in an
>>>>> experiment.
>>>>>
>>>>> I don’t know if any requirement that IANA assignment need to be called
>>>>> out
>>>>> in the abstract but we can do that. However, I believe the question if
>>>>> this
>>>>> document should or should not assign the bit is still not completely
>>>>> solved, or
>>>>> is it?
>>>> I believe this question will have to be reviewed during WGLC and, more
>>>> importantly, IETF last call. For the moment, my concern is that the document
>>>> correctly describes the IANA allocation.
>>>>
>>>> I would like to see here a statement such as : "Given TCP header space is
>>>> scarce, this specification allocates a reserved header bit and overloads the
>>>> two ECN flags in the TCP header ...“.
>>> A bit lengthy but now:
>>>
>>> "Given TCP header space is
>>>         scarce, it allocates a reserved header bit, that was previously
>>> used for
>>>         ECN-Nonce which was recently declared historic, and overloads the
>>>         two existing ECN flags in the TCP header. Further, additional
>>>         information can be provided in a new TCP option that however is not
>>> used
>>>         on the TCP SYN."
>>>
>>>>>> * 1.  Introduction
>>>>>>
>>>>>>     Recently, proposed mechanisms like Congestion Exposure (ConEx
>>>>>>     [RFC7713]), DCTCP [RFC8257] or L4S [I-D.ietf-tsvwg-l4s-arch] need
>>>>>>     more accurate ECN feedback information whenever more than one
>>>>> marking
>>>>>>     is received in one RTT.
>>>>>>
>>>>>> [ms] At least for RFC 8257 seems to be implementable withoit this.
>>>>>> Instead
>>>>> of stating a "need", it would IMHO make more sense to discuss the
>>>>> benefits
>>>>> of the suggested mechanism in this document of its own, independent of
>>>>> other proposals. To me, this document should be independent of other
>>>>> documents and specifically other experiments. We have to think about
>>>>> cases
>>>>> where not all experiments are successful. Then independent documents
>>>>> will
>>>>> be more future-proof in future.
>>>>>
>>>>> This is a naming collision… The sentence was meant to say that these
>>>>> mechanisms new more accurate ECN feedback than provided today by
>>>>> RFC3168 but it was not meant to say that these mechanism have to use the
>>>>> scheme as specified in this document.
>>>>>
>>>>> I added the following part sentence:
>>>>>
>>>>> „Recently, proposed mechanisms like Congestion Exposure (ConEx
>>>>> [RFC7713]), DCTCP [RFC8257] or L4S [I-D.ietf-tsvwg-l4s-arch] need more
>>>>> accurate ECN feedback information than provided by the feedback scheme
>>>>> as specified in [RFC3168] whenever more than one marking is received in
>>>>> one
>>>>> RTT. This document specifies an alternative feedback scheme that
>>>>> provides
>>>>> more accurate information and could be used by these new TCP
>>>>> extensions.“
>>>>>
>>>>> Does this help?
>>>> See my proposal for the abstract. I continue to disagree with the term
>>>> "need" but I think this can be sorted out by another term.
>>>>
>>>>>>     If AccECN progresses from experimental to the standards
>>>>>>     track, it is intended to be a complete replacement for classic TCP/
>>>>>>     ECN feedback, not a fork in the design of TCP.
>>>>>>
>>>>>> [ms] This sentence should be removed, as this is speculation.
>>>>> Why? It states an intent… and that’s the intent that we have.
>>>>>
>>>>>>     Until the AccECN experiment succeeds, [RFC3168] will remain as the
>>>>>>     standards track specification for adding ECN to TCP.
>>>>>>
>>>>>> [ms] This sentence should be removed (or reworded)
>>>>> Why? Does it help to add an only here:
>>>>>
>>>>> "Until the AccECN experiment succeeds, [RFC3168] will remain as the only
>>>>> standards track specification for adding ECN to TCP.“
>>>> This wording is better.
>>>>
>>>>>>     AccECN feedback overloads flags and fields in the main TCP header
>>>>>>     with new definitions, so both ends have to support the new wire
>>>>>>     protocol before it can be used.
>>>>>>
>>>>>> [ms] In my reading this experimental document asks for *new* allocation
>>>>> of a reserved TCP header flag.
>>>>>
>>>>> Is this better?
>>>>>
>>>>> "AccECN feedback overloads the two existing ECN flags as well as the
>>>>>        currently reserved and previously called NS flag in the main TCP
>>>>> header
>>>>>        with new definitions, so both ends have to support the new wire
>>>>> protocol
>>>>>        before it can be used.“
>>>>>
>>>>> I understand that you are not happy with the word „overload“ here but
>>>>> the
>>>>> point of this sentence really is that the flags can/could be used
>>>>> differently
>>>>> and therefore we need a new negotiation before we can use them.
>>>> For me the following would work: "AccECN feedback overloads the two
>>>> existing ECN flags and
>>>> allocates the currently reserved and previously called NS flag in the
>>>> main TCP header.
>>>> Given the new definitions, both ends have to support the new wire
>>>> protocol
>>>> before it can be used."
>>>>
>>>> I believe the wording has to be crystal clear on the reservation of bit 7
>>>> when it is discussed the first time in the text. In follow-up sections,
>>>> maybe shorter terms could be used.
>>> Okay, now:
>>>
>>> "AccECN feedback overloads the two existing ECN flags and
>>>       allocates the currently reserved and previously called NS flag in the
>>>       TCP header, to be used as one field indicating the number of
>>> congestion
>>>       experienced marked packets. Given the new definitions of these three
>>> bits,
>>>       both ends     have to support the new wire protocol before it can be
>>> used.
>>>       Therefore during the TCP handshake the two ends use these three bit
>>> in
>>>       the TCP header to negotiate the most advanced feedback protocol
>>>       that they can both support in a backward compatible way to
>>>       <xref target="RFC3168"/>."
>>>>> If you prefer, we can also remove the NS flag in this list, as ECN Nonce
>>>>> was
>>>>> anyway never deployed.
>>>>>
>>>>>>     For that we refer to [RFC3168] or any RFC that
>>>>>>     specifies a different response to TCP ECN feedback, for example:
>>>>>>     [RFC8257]; or the ECN experiments referred to in
>>>>>>     [I-D.ietf-tsvwg-ecn-experimentation], namely: a TCP-based Low
>>>>>> Latency
>>>>>>     Low Loss Scalable (L4S) congestion control
>>>>>> [I-D.ietf-tsvwg-l4s-arch];
>>>>>>     ECN-capable TCP control packets [I-D.ietf-tcpm-generalized-ecn], or
>>>>>>     Alternative Backoff with ECN (ABE)
>>>>>>     [I-D.ietf-tcpm-alternativebackoff-ecn].
>>>>>>
>>>>>> [ms] At least ABE seems orthogonal. Anyway, I think this paragraph can
>>>>>> just
>>>>> be deleted. If other experiments need more accurate feedback, it is up
>>>>> to
>>>>> them to explain how they would use this mechanism. This document should
>>>>> focus on how to signal the feedback, not how to use that.
>>>>>
>>>>> Yes, that is what the paragraph says. Isn’t it better to be explicit
>>>>> about this?
>>>>>
>>>>>>     It is likely (but not required) that the AccECN protocol will be
>>>>>>     implemented along with the following experimental additions to the
>>>>>>     TCP-ECN protocol: ECN-capable TCP control packets and
>>>>>> retransmissions
>>>>>>     [I-D.ietf-tcpm-generalized-ecn], which includes the ECN-capable SYN/
>>>>>>     ACK experiment [RFC5562]; and testing receiver non-compliance
>>>>>>     [I-D.moncaster-tcpm-rcv-cheat].
>>>>>>
>>>>>> [ms] I am a big fan of simple, standalone documents. In my view, the
>>>>>> TCPM
>>>>> working group should publish draft-ietf-tcpm-accurate-ecn and
>>>>> draft-ietf-
>>>>> tcpm-generalized-ecn independent documents, which probably implies that
>>>>> draft-ietf-tcpm-generalized-ecn does not use AccECN. If experimentation
>>>>> with ECT in SYN requires a combination, this could be done in a new,
>>>>> third
>>>>> document. Apart from having simpler focused documents, this could
>>>>> significantly help later with moving forward documents to standards
>>>>> track.
>>>>>
>>>>> I disagree, however, this is a discussion to have on draft-ietf-tcpm-
>>>>> generalized-ecn. I don’t see a problem in  providing a reference here
>>>>> that
>>>>> says „it is likely…“ and nothing more.
>>>>>>
>>>>>>
>>>>>> * 1.1.  Document Roadmap
>>>>>>
>>>>>> [ms] A macroscopic comment is that this document has a lot of
>>>>>> introduction
>>>>> and tutorial text with lot's of redundancy towards other documents. I
>>>>> think
>>>>> the document can be made much easier to read by shorten it. In many
>>>>> cases
>>>>> this is just an editorial change as there is redundancy. As one such
>>>>> example,
>>>>> just remove this section.
>>>>>
>>>>> I guess this a matter of taste. As an AD, I’m a big fan of short and
>>>>> concise
>>>>> documents, however, some redundancy can also help understanding,
>>>>> especially if you explain things multiple times but with a different
>>>>> level of
>>>>> detail. I personally would not need the roadmap but I know many people
>>>>> who find these things helpful and to be honest I don’t see how removing
>>>>> this
>>>>> part makes the doc any better. If you don’t want it, don’t read it.
>>>>>
>>>>>> * 1.2.  Goals
>>>>>>
>>>>>> [ms] I think this section can also just be removed.
>>>>> I have to say I also don’t see the point of removing this part. Given
>>>>> we’ve
>>>>> done the work on requirements, I think we should also link to this doc
>>>>> somewhere.
>>>>>>
>>>>>> * 1.3.  Experiment Goals
>>>>>>
>>>>>>     TCP is critical to the robust functioning of the Internet, therefore
>>>>>>     any proposed modifications to TCP need to be thoroughly tested. The
>>>>>>     present specification describes an experimental protocol that adds
>>>>>>     more accurate ECN feedback to the TCP protocol.  The intention is to
>>>>>>     specify the protocol sufficiently so that more than one
>>>>>>     implementation can be built in order to test its function,
>>>>>> robustness
>>>>>>     and interoperability (with itself and with previous version of ECN
>>>>>>     and TCP).
>>>>>>
>>>>>> [ms] I think all what is written in this paragraph is obvious, no?
>>>>>> Can't we just
>>>>> delete this?
>>>>>
>>>>> Sure, however, I don’t think it hurts to spell it out. For me both is
>>>>> fine, keep it
>>>>> or remove it.
>>>>>
>>>>>>     The experimental protocol will be considered successful if it is
>>>>>>     deployed and if it satisfies the requirements of [RFC7560] in the
>>>>>>     consensus opinion of the IETF tcpm working group.  In short, this
>>>>>>     requires that it improves the accuracy and timeliness of TCP's ECN
>>>>>>     feedback, as claimed in Section 5, while striking a balance between
>>>>>>     the conflicting requirements of resilience, integrity and
>>>>>>     minimisation of overhead.  It also requires that it is not unduly
>>>>>>     complex, and that it is compatible with prevalent equipment
>>>>>>     behaviours in the current Internet (e.g. hardware offloading and
>>>>>>     middleboxes), whether or not they comply with standards.
>>>>>>
>>>>>>     Testing will mostly focus on fall-back strategies in case of
>>>>>>     middlebox interference.  Current recommended strategies are
>>>>>> specified
>>>>>>     in Sections 3.1.2, 3.2.3, 3.2.4 and 3.2.7.  The effectiveness of
>>>>>>     these strategies depends on the actual deployment situation of
>>>>>>     middleboxes.  Therefore experimental verification to confirm large-
>>>>>>     scale path traversal in the Internet is needed before finalizing
>>>>>> this
>>>>>>     specification on the Standards Track.
>>>>>>
>>>>>> [ms] These two paragraphs must be entirely rewritten. As I have
>>>>> mentioned before, I don't think an RFC should speculate about TCPM and
>>>>> its
>>>>> consensus opinion. I would suggest a wording along the lines of:
>>>>>> <ms>
>>>>>>     The experimental protocol will be considered successful if
>>>>>>     testing confirms that the proposed mechanism can be deployed at
>>>>>> large
>>>>> scale.
>>>>>>     Testing will mostly focus on fall-back strategies in case of
>>>>>>     middlebox interference.  Current recommended strategies are
>>>>>> specified
>>>>>>     in Sections 3.1.2, 3.2.3, 3.2.4 and 3.2.7.  The effectiveness of
>>>>>>     these strategies depends on the actual deployment situation of
>>>>>>     middleboxes.  Therefore experimental verification to confirm large-
>>>>>>     scale path traversal in the Internet is needed, e.g., by support in
>>>>>>     major TCP stacks.
>>>>>> </ms>
>>>>>>
>>>>> I don’t understand your point here. I don’t think that the paraphrase
>>>>> speculates about the consensus of tcpm, in contrast it say tcpm has to
>>>>> decided if the requirements previously specified by tcpm are
>>>>> sufficiently
>>>>> fulfilled. I don’t see a reason to not mention the requirement draft as
>>>>> this
>>>>> draft as tcpm consensus and was written for this purpose.
>>>> My suggested wording uses the expression "can be deployed at large scale"
>>>> and I believe this is relevant.
>>>>
>>>> The document already describes in Section 5 how the protocol satisfies
>>>> the agreed requirements for a more accurate ECN feedback protocol [RFC7560].
>>>> So, if the TCPM working group publishes this document with the content of
>>>> Section 5, I believe the TCPM working group already has reached consensus
>>>> that the protocol meets requirements. In addition, it is possible that new
>>>> requirements would be identified in future, e.g., as an outcome of the
>>>> experiment, and that would obviously have to be considered by TCPM. In that
>>>> case, for the success of the experiment not only RFC 7560 would matter, but
>>>> also further requirements. My proposed wording does not have all these
>>>> problems.
>>>>
>>>> In a nutshell, I continue to believe that this section has to change.
>>>
>>> Okay, used your proposed wording. You have a point about the requirement
>>> and I mis-read you proposal earlier as „has to be deployed large-scale“.
>>>
>>>>>> * 1.5.  Recap of Existing ECN feedback in IP/TCP
>>>>>>
>>>>>> [ms] This section could probably be shortened as well.
>>>>>>
>>>>>>     The last bit in byte 13 of the TCP header was defined as the Nonce
>>>>>>     Sum (NS) for the ECN Nonce [RFC3540].  RFC 3540 was never deployed
>>>>>> so
>>>>>>     it is being reclassified as historic, making this TCP flag available
>>>>>>     for use by the AccECN experiment instead.
>>>>>>
>>>>>> [ms] This wording, as well as Figure 1, needs to take into account the
>>>>>> IANA
>>>>> status when draft-ietf-tsvwg-ecn-experimentation is published.
>>>>>
>>>>> Is does. However, I can explicitly say that is has be re-clssified as
>>>>> reserved.
>>>>>
>>>>> "RFC 3540 was never deployed so it is being reclassified as historic
>>>>> [I-D.ietf-
>>>>> tsvwg-ecn-experimentation] and the respective flag has been marked as
>>>>> „reserved“ in the IANA TCP Header Flags registry, making this TCP flag
>>>>> available for use by the AccECN experiment instead.“
>>>>>
>>>>> Better?
>>>>>
>>>>>> In my understanding, this experimental document asks for new assignment
>>>>> of a reserved TCP header flag.
>>>>>
>>>>> As I said I’m not sure if we have fully concluded this discussion yet.
>>>>> However,
>>>>> what we really would want to is mention somewhere that this experiment
>>>>> with this flags is running. I guess there are three options:
>>>>> 1) keep it in the registry as reserved and conserve the knowledge in
>>>>> tcpm
>>>>> that this experiment is running and no other experimental RFC such use
>>>>> this
>>>>> flags as long as this experiment is running.
>>>>> 2) Keep is marked as reserved but add a note about this experiment in
>>>>> the
>>>>> IANA registry
>>>>> 3) Or assign it right away with IESG approval. I guess in this case tcpm
>>>>> could
>>>>> also consider to change the registration policy to „IETF Review“.
>>>> The current registration policy for the TCP header flags is "standards
>>>> action". I understand that the IESG could approve exceptions. But given the
>>>> policy, I believe the document has to be very precise on the request
>>>> regarding bit 7.
>>> Okay, it now says:
>>>
>>> "[TO BE REMOVED: IANA is requested to update the existing entry in the
>>> Transmission Control Protocol (TCP) Header Flags registration
>>> (https://www.iana.org/assignments/tcp-header-flags/tcp-header-flags.xhtml#tcp-header-flags-1)
>>> for Bit 7 to "AE (Accurate ECN), previously used by Historic as NS (Nonce
>>> Sum) [RFC3540, RFC8311]" and change the reference to this RFC-to-be instead
>>> of RFC8311.]“
>>>
>>> I guess we could also ask IANA to add an additional comment column instead
>>> (but not sure if we then have to update RFC3168, which I think we really
>>> don’t want. Should be fine now.
>>>
>>>>>> * 2.  AccECN Protocol Overview and Rationale
>>>>>>
>>>>>>     o  an essential part that re-uses ECN TCP header bits to feed back
>>>>>>        the number of arriving CE marked packets.  This provides more
>>>>>>        accuracy than classic ECN feedback, but limited resilience
>>>>>> against
>>>>>>        ACK loss;
>>>>>>
>>>>>> [ms] The word "re-use" is IMHO not correct.
>>>>> I think this is nit picking. Using a different phrasing here makes the
>>>>> sentence
>>>>> unnecessary complicated. We don’t try to some how get a round the fact
>>>>> that we need to handle the flag registration correctly. However, here
>>>>> the
>>>>> point really is to explain how the protocol word. The main point of
>>>>> using the
>>>>> work „re-use“ here is really that we say that these flags are or have
>>>>> been
>>>>> used different by other TCP extension (and we therefore need a proper
>>>>> negotiation scheme).
>>>> If the allocation of a reserved flag is correctly explained in the
>>>> abstract and introduction, I think these sentences can use a bit relaxed
>>>> terminology.
>>>>
>>>>>>     The two part design was necessary, given limitations on the space
>>>>>>     available for TCP options and given the possibility that certain
>>>>>>     incorrectly designed middleboxes prevent TCP using any new options

-- 
________________________________________________________________
Bob Briscoe                               http://bobbriscoe.net/