Date: Wed, 07 Nov 2018 02:32:51 -0800
From: ianswett <notifications@github.com>
Reply-To: quicwg/base-drafts
 <reply+0166e4abecc984e31330dd4249e4d337c134a2f99d65608d92cf0000000117fa815392a169ce168ac763@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/1977/436579227@github.com>
In-Reply-To: <quicwg/base-drafts/issues/1977@github.com>
References: <quicwg/base-drafts/issues/1977@github.com>
Subject: Re: [quicwg/base-drafts] Exempt all ACK frames from congestion
 control (#1977)
Mime-Version: 1.0
Content-Type: multipart/alternative;
 boundary="--==_mimepart_5be2bf5352215_670d3f9b786d45b82021e0";
 charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/Z6gTAwPbibwZLpORhKgVrE72Bt0>


----==_mimepart_5be2bf5352215_670d3f9b786d45b82021e0
Content-Type: text/plain;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

Agreed, I'm happy with the status quo.  If you want to game congestion control, there are plenty of ways to be unilaterally more aggressive.

The reason ACK-only packets are the only ones that aren't congestion controlled is to allow sending ACKs even if there's no available congestion control or you'd otherwise be pacing limited.  And the main reason they don't count towards bytes in flight is because they don't elicit an ACK to allow them to be removed from bytes in flight.

-- 
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/1977#issuecomment-436579227
----==_mimepart_5be2bf5352215_670d3f9b786d45b82021e0
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<p>Agreed, I'm happy with the status quo.  If you want to game congestion=
 control, there are plenty of ways to be unilaterally more aggressive.</p=
>
<p>The reason ACK-only packets are the only ones that aren't congestion c=
ontrolled is to allow sending ACKs even if there's no available congestio=
n control or you'd otherwise be pacing limited.  And the main reason they=
 don't count towards bytes in flight is because they don't elicit an ACK =
to allow them to be removed from bytes in flight.</p>

<p style=3D"font-size:small;-webkit-text-size-adjust:none;color:#666;">&m=
dash;<br />You are receiving this because you are subscribed to this thre=
ad.<br />Reply to this email directly, <a href=3D"https://github.com/quic=
wg/base-drafts/issues/1977#issuecomment-436579227">view it on GitHub</a>,=
 or <a href=3D"https://github.com/notifications/unsubscribe-auth/AWbkqx34=
bGiKM7bD6QErOtdeDui7AilDks5usrbTgaJpZM4YSAuG">mute the thread</a>.<img sr=
c=3D"https://github.com/notifications/beacon/AWbkq05sxOvGE1uyBa4Oy_oqHou2=
4KLnks5usrbTgaJpZM4YSAuG.gif" height=3D"1" width=3D"1" alt=3D"" /></p>
<script type=3D"application/json" data-scope=3D"inboxmarkup">{"api_versio=
n":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name"=
:"GitHub"},"entity":{"external_key":"github/quicwg/base-drafts","title":"=
quicwg/base-drafts","subtitle":"GitHub repository","main_image_url":"http=
s://assets-cdn.github.com/images/email/message_cards/header.png","avatar_=
image_url":"https://assets-cdn.github.com/images/email/message_cards/avat=
ar.png","action":{"name":"Open in GitHub","url":"https://github.com/quicw=
g/base-drafts"}},"updates":{"snippets":[{"icon":"PERSON","message":"@ians=
wett in #1977: Agreed, I'm happy with the status quo.  If you want to gam=
e congestion control, there are plenty of ways to be unilaterally more ag=
gressive.\r\n\r\nThe reason ACK-only packets are the only ones that aren'=
t congestion controlled is to allow sending ACKs even if there's no avail=
able congestion control or you'd otherwise be pacing limited.  And the ma=
in reason they don't count towards bytes in flight is because they don't =
elicit an ACK to allow them to be removed from bytes in flight."}],"actio=
n":{"name":"View Issue","url":"https://github.com/quicwg/base-drafts/issu=
es/1977#issuecomment-436579227"}}}</script>
<script type=3D"application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/quicwg/base-drafts/issues/1977#issuecomment=
-436579227",
"url": "https://github.com/quicwg/base-drafts/issues/1977#issuecomment-43=
6579227",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [quicwg/base-drafts] Exempt all ACK frames from congestion =
control (#1977)",
"sections": [
{
"text": "",
"activityTitle": "**ianswett**",
"activityImage": "https://assets-cdn.github.com/images/email/message_card=
s/avatar.png",
"activitySubtitle": "@ianswett",
"facts": [

]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \=
"quicwg/base-drafts\",\n\"issueId\": 1977,\n\"IssueComment\": \"{{IssueCo=
mment.value}}\"\n}"
}
]
},
{
"name": "Close issue",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueClose\",\n\"repositoryFullName\": \"q=
uicwg/base-drafts\",\n\"issueId\": 1977\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/quicwg/base-drafts/issues/1977#issuecomment-43=
6579227"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 4073747=
26\n}"
}
],
"themeColor": "26292E"
}
]</script>=

----==_mimepart_5be2bf5352215_670d3f9b786d45b82021e0--