[quicwg/base-drafts] Clarification on Setting the Loss Detection Timer in the loss recovery draft (#2093)

Nasif Ekiz <notifications@github.com> Mon, 03 December 2018 21:01 UTC

Date: Mon, 03 Dec 2018 21:01:18 +0000
From: Nasif Ekiz <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab47536e7008c911a089eb94fe7e8efec200eef36c92cf00000001181d5b9d92a169ce171134fd@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/2093@github.com>
Subject: [quicwg/base-drafts] Clarification on Setting the Loss Detection Timer in the loss recovery draft (#2093)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5c05999de7aa8_388a3fdc732d45bc18453b"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/I87JQGoz5ywniKaP1DeXhepcDTI>

Setting the Loss Detection Timer section states:

    // Don't arm timer if there are no retransmittable packets
    // in flight.
    if (bytes_in_flight == 0):
      loss_detection_timer.cancel()
      return

If a QUIC sender sends multiple retransmittable packets and some ACK/PADDING packets and only retransmittable packets are ACKed, it leaves the bytes_in_flight > 0 but no retransmittable packets inflight. In this case, Loss Detection Timer need not be scheduled as there is not retransmittable packets inflight but the above check results in scheduling the timer again.

if (bytes_in_flight == 0): check does not seem enough by itself to decide if there are outstanding retransmittable packets. Do we need to update that section? 


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/quicwg/base-drafts/issues/2093

[quicwg/base-drafts] Clarification on Setting the… Nasif Ekiz
Re: [quicwg/base-drafts] Clarification on Setting… janaiyengar
Re: [quicwg/base-drafts] Clarification on Setting… ianswett