[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