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
- http2 draft feedback on flow control Patrick McManus