Re: [quicwg/base-drafts] Notify the congestion controller of losses first (#3540)

ianswett <> Tue, 31 March 2020 16:21 UTC

Return-Path: <>
Received: from localhost (localhost []) by (Postfix) with ESMTP id E96E13A23CD for <>; Tue, 31 Mar 2020 09:21:07 -0700 (PDT)
X-Virus-Scanned: amavisd-new at
X-Spam-Flag: NO
X-Spam-Score: -0.5
X-Spam-Status: No, score=-0.5 tagged_above=-999 required=5 tests=[DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, HTML_IMAGE_ONLY_20=0.7, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: (amavisd-new); dkim=pass (1024-bit key)
Received: from ([]) by localhost ( []) (amavisd-new, port 10024) with ESMTP id T5t6KqCewB1t for <>; Tue, 31 Mar 2020 09:21:06 -0700 (PDT)
Received: from ( []) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by (Postfix) with ESMTPS id 9B5153A23CC for <>; Tue, 31 Mar 2020 09:21:06 -0700 (PDT)
Received: from ( []) by (Postfix) with ESMTP id BE04A2829B6 for <>; Tue, 31 Mar 2020 09:21:05 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=pf2014; t=1585671665; bh=GBJ1ZWga24QC8dBBIn282LTXiOJ6mlsYIgNhiomBMyw=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=XT0mtBP8l5d7+8YufoHGHdnLDENMBrhfZ99zAjp8Qy4LDofUo6Glja4DMOV4BNV7h zwJjI/PKHERZ4b/5nrO6DSI/wi5hFV7deMrLYIUCPc1F08ZGSBC12ARVwtTsNCyCpe iFwtPwnVZGUTdQgRQ4axGP8K1dVP/ETZmxDJ5P4w=
Date: Tue, 31 Mar 2020 09:21:05 -0700
From: ianswett <>
Reply-To: quicwg/base-drafts <>
To: quicwg/base-drafts <>
Cc: Subscribed <>
Message-ID: <quicwg/base-drafts/pull/3540/>
In-Reply-To: <quicwg/base-drafts/pull/>
References: <quicwg/base-drafts/pull/>
Subject: Re: [quicwg/base-drafts] Notify the congestion controller of losses first (#3540)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5e836df1adc5b_7ec93f8ac06cd95c1280d9"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: ianswett
X-GitHub-Recipient: quic-issues
X-GitHub-Reason: subscribed
X-Auto-Response-Suppress: All
Archived-At: <>
X-Mailman-Version: 2.1.29
List-Id: Notification list for GitHub issues related to the QUIC WG <>
List-Unsubscribe: <>, <>
List-Archive: <>
List-Post: <>
List-Help: <>
List-Subscribe: <>, <>
X-List-Received-Date: Tue, 31 Mar 2020 16:21:08 -0000

@marten-seemann my reasoning was I thought it was critical to determine what packets were newly acked before invoking loss detection.  In some cases it may be(depending upon details of one's loss detection), but since we update largest_acked_packet[pn_space] outside the code which detects newly acked packets, it's not critical to the pseudocode as it stands today.

That being said, I think the parallelism is nice, so I'd prefer to keep it.

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub: