Re: [quicwg/base-drafts] Sending probe packet has duplicated text (#3468)

mirjak <notifications@github.com> Mon, 17 February 2020 17:53 UTC

Return-Path: <noreply@github.com>
X-Original-To: quic-issues@ietfa.amsl.com
Delivered-To: quic-issues@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id E091F120867 for <quic-issues@ietfa.amsl.com>; Mon, 17 Feb 2020 09:53:38 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.382
X-Spam-Level:
X-Spam-Status: No, score=-6.382 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_IMAGE_ONLY_24=1.618, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=github.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 9vS5hEfMXcum for <quic-issues@ietfa.amsl.com>; Mon, 17 Feb 2020 09:53:37 -0800 (PST)
Received: from out-3.smtp.github.com (out-3.smtp.github.com [192.30.252.194]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 2B88E120882 for <quic-issues@ietf.org>; Mon, 17 Feb 2020 09:53:37 -0800 (PST)
Date: Mon, 17 Feb 2020 09:53:36 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1581962016; bh=3bLt8jivqjBoiLLc7tvFWFunnex4JNsPkgYWZOcnSFs=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=YCOvhxE0vBeH8nwrGg2ElmTy+yLc5fSPldwE42L1/GvCPkDpiz4LyTWgHmusU9tib /XEsc/tzdFMExUM/k+CThmIm/IiQLWuseB3kjrlrsS32LPAdPpj7bHrtvrWF4oDAgO ZTjidSr3U9TfAoa+RHS4b9c2ki+LD6375YN3hNhk=
From: mirjak <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJKZW7MD6AQ6G4ABV2LV4LAC2BEVBNHHCDQ2AF4@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/3468/c587102790@github.com>
In-Reply-To: <quicwg/base-drafts/pull/3468@github.com>
References: <quicwg/base-drafts/pull/3468@github.com>
Subject: Re: [quicwg/base-drafts] Sending probe packet has duplicated text (#3468)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5e4ad3207179b_19e63fcabf0cd964160429"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: mirjak
X-GitHub-Recipient: quic-issues
X-GitHub-Reason: subscribed
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address: quic-issues@ietf.org
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/TAUXj-t8CHeS4Zc_Jcq5nUCciW4>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Notification list for GitHub issues related to the QUIC WG <quic-issues.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/quic-issues>, <mailto:quic-issues-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/quic-issues/>
List-Post: <mailto:quic-issues@ietf.org>
List-Help: <mailto:quic-issues-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/quic-issues>, <mailto:quic-issues-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 17 Feb 2020 17:53:45 -0000

I also just noticed that there is this sentence in the "sending probe packets" section:
" Alternatively, instead of sending an ack-eliciting packet, the sender
MAY mark any packets still in flight as lost."

and this sentence in the next "detection loss" section:
" A PTO timer expiration event does not indicate packet loss and MUST
NOT cause prior unacknowledged packets to be marked as lost."

So maybe that first sentence should have be removed as well? Not sure what is intended here...?

-- 
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/pull/3468#issuecomment-587102790