Re: Adding ECN to Transport and Recovery

[Magnus Westerlund <[e-mail]>]

Hi,

I have just added one more paragraph to the pull request to deal with an 
issue that arises if there is persistent ACK loss such that the sender 
never receives an acknowledgement for some packet that was actually 
received by receiver and counted in the ECN counters. To make 
implementers aware of the issue I have added this paragraph in Section 
6.6.1. Continuous Verification of ECN:


    If the acknowledgements from the receiver are lost such that one or
    more packet are received by the receiver, but never acknowledged to
    the sender an insensitivity to bleaching will be created.  In this
    situation the ECN counters reported will have increase, but the
    sender side total for acknowledged packets will not have increased.
    Thus, a number of bleached packets equal to the number of packets
    that failed to be acknowledged can be received before triggering the
    continuous verification.  To address this issue the sender detect the
    case when the ECN counters grows more than number of acknowledged
    packets when a ACK_ECN frame is received.  In such cases a new
    comparison point is created by storing the current number of totally
    acknowledged packets and latest ECN counters.  Then comparison are
    done by subtracting these stored values from the respective counters
    prior to the comparison.  Note that any out-of-order ACK_ECN frames
    can't be used for determining any loss of acknowledgements.

Please provide feedback on this addition.

Cheers

Magnus

Den 2018-05-28 kl. 22:29, skrev Magnus Westerlund:
> WG,
>
> I have been working on a Pull Request 
> (https://github.com/quicwg/base-drafts/pull/1372) that implements the 
> ECN proposal from the design team 
> (https://github.com/quicwg/base-drafts/wiki/ECN-in-QUIC) into 
> something reasonably well integrated into the current drafts. Thus, I 
> think it is time that people review and consider this proposal so that 
> any issues can be addressed.
>
> I would very much like to present at the interim this proposal, some 
> of the choices made, and determine if there are additional changes to 
> be made before merging this.
>
> Cheers
>
> Magnus Westerlund
>
> ----------------------------------------------------------------------
> Network Architecture & Protocols, Ericsson Research
> ----------------------------------------------------------------------
> Ericsson AB                 | Phone  [phone]
> Torshamnsgatan 23           | Mobile [phone]
> SE-164 80 Stockholm, Sweden | mailto: [e-mail]
> ----------------------------------------------------------------------
>

-- 

Magnus Westerlund

----------------------------------------------------------------------
Network Architecture & Protocols, Ericsson Research
----------------------------------------------------------------------
Ericsson AB                 | Phone  [phone]
Torshamnsgatan 23           | Mobile [phone]
SE-164 80 Stockholm, Sweden | mailto: [e-mail]
----------------------------------------------------------------------