[tcpm] Barry Leiba's No Objection on draft-ietf-tcpm-newcwv-12: (with COMMENT)
"Barry Leiba" <barryleiba@computer.org> Mon, 22 June 2015 20:57 UTC
Return-Path: <barryleiba@computer.org>
X-Original-To: tcpm@ietfa.amsl.com
Delivered-To: tcpm@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id EFFC91A86EF; Mon, 22 Jun 2015 13:57:14 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.9
X-Spam-Level:
X-Spam-Status: No, score=-1.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9] autolearn=ham
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 49H6lvaUsmXG; Mon, 22 Jun 2015 13:57:13 -0700 (PDT)
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 3159A1A8704; Mon, 22 Jun 2015 13:57:08 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Barry Leiba <barryleiba@computer.org>
To: The IESG <iesg@ietf.org>
X-Test-IDTracker: no
X-IETF-IDTracker: 6.0.3.p3
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <20150622205708.32143.3139.idtracker@ietfa.amsl.com>
Date: Mon, 22 Jun 2015 13:57:08 -0700
Archived-At: <http://mailarchive.ietf.org/arch/msg/tcpm/Trrvmw_ihq44TQIWld0jKOX2fBE>
Cc: tcpm@ietf.org, draft-ietf-tcpm-newcwv@ietf.org, tcpm-chairs@ietf.org
Subject: [tcpm] Barry Leiba's No Objection on draft-ietf-tcpm-newcwv-12: (with COMMENT)
X-BeenThere: tcpm@ietf.org
X-Mailman-Version: 2.1.15
List-Id: TCP Maintenance and Minor Extensions Working Group <tcpm.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tcpm>, <mailto:tcpm-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/tcpm/>
List-Post: <mailto:tcpm@ietf.org>
List-Help: <mailto:tcpm-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tcpm>, <mailto:tcpm-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 22 Jun 2015 20:57:15 -0000
Barry Leiba has entered the following ballot position for draft-ietf-tcpm-newcwv-12: No Objection When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html for more information about IESG DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-tcpm-newcwv/ ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- Just a few very minor comments -- nothing that needs any discussion. -- Section 1 -- RFC 2616 is an obsolete reference for HTTP. The current reference is RFC 7230 o To incentivise the use of long-lived connections Will you please appease my inner pedant and not say "incentivise" (which, by the way, Chrome doesn't think is a word either)? You can take advantage of the previous bullet's use of "To remove the incentive for" by using this parallel construction: "To provide an incentive for the use of long-lived connections". -- Section 4.2 -- The method RECOMMENDS that the TCP SACK option [RFC2018] is enabled and the method defined in [RFC6675] is used to recover missing segments. Even more ridiculously pedantic than the other: subjunctive mood with "recommends", please. Make both "is" into "be". -- Section 4.4 -- A TCP sender implementing this specification MUST enter the non- validated phase when the pipeACK is less than (1/2)*cwnd. Given that there are "MAY"s and a "SHOULD" involved in how pipeACK is computed, it seems rather odd to have a MUST that relates to its value. You couldn't possibly determine whether an implementation was doing this "right" because there's so much variability in the value of pipeACK anyway. No need to discuss this, but I suggest that you just do this: NEW A TCP sender implementing this specification enters the non- validated phase when the pipeACK is less than (1/2)*cwnd. END