http2 draft feedback on flow control

Patrick McManus <pmcmanus@mozilla.com> Wed, 22 May 2013 19:35 UTC

Return-Path: <ietf-http-wg-request@listhub.w3.org>
X-Original-To: ietfarch-httpbisa-archive-bis2Juki@ietfa.amsl.com
Delivered-To: ietfarch-httpbisa-archive-bis2Juki@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 9992D21F8D69 for <ietfarch-httpbisa-archive-bis2Juki@ietfa.amsl.com>; Wed, 22 May 2013 12:35:02 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -9.671
X-Spam-Level:
X-Spam-Status: No, score=-9.671 tagged_above=-999 required=5 tests=[AWL=0.305, BAYES_00=-2.599, FM_FORGED_GMAIL=0.622, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-8]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id o+nuncI6Lul1 for <ietfarch-httpbisa-archive-bis2Juki@ietfa.amsl.com>; Wed, 22 May 2013 12:34:56 -0700 (PDT)
Received: from frink.w3.org (frink.w3.org [128.30.52.56]) by ietfa.amsl.com (Postfix) with ESMTP id B87CA21F9193 for <httpbisa-archive-bis2Juki@lists.ietf.org>; Wed, 22 May 2013 12:34:53 -0700 (PDT)
Received: from lists by frink.w3.org with local (Exim 4.72) (envelope-from <ietf-http-wg-request@listhub.w3.org>) id 1UfEmM-00060N-T3 for ietf-http-wg-dist@listhub.w3.org; Wed, 22 May 2013 19:32:54 +0000
Resent-Date: Wed, 22 May 2013 19:32:54 +0000
Resent-Message-Id: <E1UfEmM-00060N-T3@frink.w3.org>
Received: from maggie.w3.org ([128.30.52.39]) by frink.w3.org with esmtp (Exim 4.72) (envelope-from <patrick.ducksong@gmail.com>) id 1UfEm4-0005zW-8R for ietf-http-wg@listhub.w3.org; Wed, 22 May 2013 19:32:36 +0000
Received: from mail-ob0-f181.google.com ([209.85.214.181]) by maggie.w3.org with esmtps (TLS1.0:RSA_ARCFOUR_SHA1:16) (Exim 4.72) (envelope-from <patrick.ducksong@gmail.com>) id 1UfElz-0006ku-6Y for ietf-http-wg@w3.org; Wed, 22 May 2013 19:32:36 +0000
Received: by mail-ob0-f181.google.com with SMTP id dn14so2858246obc.40 for <ietf-http-wg@w3.org>; Wed, 22 May 2013 12:32:05 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:date:x-google-sender-auth:message-id:subject :from:to:content-type; bh=2zyaKsd2QuXJ/rFUPSjBDzRvonhncfGfddP+1IAEDhA=; b=iDiKYxWE5PBkrkys8VAYIuhC3hLnT++88yz0qrPb+h2YP4dLZPIiI9wxQDi3j+/Yht nVp0+Ve7+ypOojtrCthiKhRV4cwoffjAbjfziF+w1fe+MzJeUTJENDVzmMdfUNdYY9FB 1WqGktGL8E+0dFnK+pvNDsowV2BybWBt+xD/CWgsgq/mkOE4M0dVq4fPD9sDX/T2FJTC LpIvC5GXRsTGaAmd/kl1+WCa9dNDwqjeBIJwe9KG2VuIwoHNpI3p+i1S785hu8WpaYBN VnoqdH7JiiKpvm0hqYTj1P6bmkyq/WMdNXnNVbl1LI4qxyXy18OLPgajH0QjBAw3qAZI arPw==
MIME-Version: 1.0
X-Received: by 10.182.233.227 with SMTP id tz3mr6159401obc.23.1369251125221; Wed, 22 May 2013 12:32:05 -0700 (PDT)
Sender: patrick.ducksong@gmail.com
Received: by 10.76.13.193 with HTTP; Wed, 22 May 2013 12:32:05 -0700 (PDT)
Date: Wed, 22 May 2013 15:32:05 -0400
X-Google-Sender-Auth: W1gooTU0ZkVEDgNMfF3XVxEsmNs
Message-ID: <CAOdDvNpksxHi1z8tVNN9VZ1sQ0SA5mWgxOY001P48qtBGbDeFg@mail.gmail.com>
From: Patrick McManus <pmcmanus@mozilla.com>
To: HTTP Working Group <ietf-http-wg@w3.org>
Content-Type: multipart/alternative; boundary="001a11c2029aed6b1404dd539ef0"
Received-SPF: pass client-ip=209.85.214.181; envelope-from=patrick.ducksong@gmail.com; helo=mail-ob0-f181.google.com
X-W3C-Hub-Spam-Status: No, score=-3.3
X-W3C-Hub-Spam-Report: AWL=-2.645, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001
X-W3C-Scan-Sig: maggie.w3.org 1UfElz-0006ku-6Y 4aea51d6caa57b9aca782b1a9d010306
X-Original-To: ietf-http-wg@w3.org
Subject: http2 draft feedback on flow control
Archived-At: <http://www.w3.org/mid/CAOdDvNpksxHi1z8tVNN9VZ1sQ0SA5mWgxOY001P48qtBGbDeFg@mail.gmail.com>
Resent-From: ietf-http-wg@w3.org
X-Mailing-List: <ietf-http-wg@w3.org> archive/latest/18085
X-Loop: ietf-http-wg@w3.org
Resent-Sender: ietf-http-wg-request@w3.org
Precedence: list
List-Id: <ietf-http-wg.w3.org>
List-Help: <http://www.w3.org/Mail/>
List-Post: <mailto:ietf-http-wg@w3.org>
List-Unsubscribe: <mailto:ietf-http-wg-request@w3.org?subject=unsubscribe>

Hello,

While reading the flow control text again there was some ambiguity in my
mind about whether or not the SETTINGS_INITIAL_WINDOW_SIZE adjusted the
connection window or just all current and future stream windows. I've
decided the text is really going for "just stream windows - connection
windows are only adjusted by WINDOW_UPDATE with stream=0" (an
interpretation I favor)..

I think it would be nice if it said that explicitly to clear up confusion.
There are some PoC implementations of this floating around out on them thar
Internets (admittedly they may predate the current spec language) and I've
seen different interpretations of what is logical.. so it makes sense to be
explicit.

A couple related followup tweaks to the text:


section 3.8.9.2

* *

*"When a HTTP/2.0 connection is first established, new streams are created
with an initial flow control window size of 65535 bytes. The connection
flow control window is 65536 bytes. Both endpoints can adjust the initial
window size for new streams by including a value for
SETTINGS_INITIAL_WINDOW_SIZE in the SETTINGS frame that forms part of the
connection header. *"

I suggest s/in the SETTINGS frame that forms part of the connection
header/with a SETTINGS frame/ .. The section is titled "Initial Flow
Control Window Size" so the existing text could be ok if we mean initial to
be at connection establishment time, but I think it more clearly applies to
the life cycle of each stream (at least when talking about stream windows).

**
**

**


*When the value of SETTINGS_INITIAL_WINDOW_SIZE changes, a receiver MUST
adjust the size of all flow control windows that it maintains by the
difference between the new value and the old value. *

uh-Oh.. "all flow control windows that it maintains" definitely includes
the connection window :( I suggest s/flow control windows/stream flow
control windows/
-Patrick