Re: [tcpm] WGLC on draft-ietf-tcpm-early-rexmt-01

Mark Allman <mallman@icir.org> Wed, 01 July 2009 13:41 UTC

Return-Path: <mallman@icir.org>
X-Original-To: tcpm@core3.amsl.com
Delivered-To: tcpm@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id E8CD23A6F1B for <tcpm@core3.amsl.com>; Wed, 1 Jul 2009 06:41:56 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.599
X-Spam-Level:
X-Spam-Status: No, score=-2.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id JELOf7IKBFyQ for <tcpm@core3.amsl.com>; Wed, 1 Jul 2009 06:41:56 -0700 (PDT)
Received: from pork.ICSI.Berkeley.EDU (pork.ICSI.Berkeley.EDU [192.150.186.19]) by core3.amsl.com (Postfix) with ESMTP id 31AC43A6EED for <tcpm@ietf.org>; Wed, 1 Jul 2009 06:41:56 -0700 (PDT)
Received: from guns.icir.org (adsl-69-222-35-58.dsl.bcvloh.ameritech.net [69.222.35.58]) by pork.ICSI.Berkeley.EDU (8.12.11.20060308/8.12.11) with ESMTP id n61Df0YZ025556; Wed, 1 Jul 2009 06:41:01 -0700
Received: from lawyers.icir.org (adsl-69-222-35-58.dsl.bcvloh.ameritech.net [69.222.35.58]) by guns.icir.org (Postfix) with ESMTP id 68E8B3B1E7F1; Wed, 1 Jul 2009 09:40:52 -0400 (EDT)
Received: from lawyers.icir.org (localhost [127.0.0.1]) by lawyers.icir.org (Postfix) with ESMTP id 2B1CD31BF38; Wed, 1 Jul 2009 09:40:53 -0400 (EDT)
To: "Eddy, Wesley M. (GRC-MS00)[Verizon]" <wesley.m.eddy@nasa.gov>
From: Mark Allman <mallman@icir.org>
In-Reply-To: <C304DB494AC0C04C87C6A6E2FF5603DB2217AB69DC@NDJSSCC01.ndc.nasa.gov>
Organization: International Computer Science Institute (ICSI)
Song-of-the-Day: Lights
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="--------ma26468-1"; micalg="pgp-sha1"; protocol="application/pgp-signature"
Date: Wed, 01 Jul 2009 09:40:53 -0400
Sender: mallman@icir.org
Message-Id: <20090701134053.2B1CD31BF38@lawyers.icir.org>
Cc: "k.avrachenkov@sophia.inria.fr" <k.avrachenkov@sophia.inria.fr>, "tcpm@ietf.org" <tcpm@ietf.org>, jblanton@cs.ohiou.edu
Subject: Re: [tcpm] WGLC on draft-ietf-tcpm-early-rexmt-01
X-BeenThere: tcpm@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
Reply-To: mallman@icir.org
List-Id: TCP Maintenance and Minor Extensions Working Group <tcpm.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/tcpm>, <mailto:tcpm-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/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: Wed, 01 Jul 2009 13:41:57 -0000

> (1) In the Introduction, paragraph 2, we have latex-style
>     double single-quotes instead of normal double quotes
> 
> (2) "When using small congestion windows"
>     should more correctly be
>     "When the congestion window is small".
> 
> (3) "Small windows can occur" ->
>     "Small congestion windows can occur"
> 
> (4) "Consider a congestion window (cwnd)"
>     The document only uses "cwnd" in one other place, in the
>     Related Work, so probably doesn't need to define an abbreviation
>     for congestion window here.
> 
> (5) Need to either define SMSS in this document or cite where we
>     pull the definition of the acronym from.  Section 2 possibly
>     should have a sentence that says we're going to reuse the
>     set of terminology from the 2581bis draft or something like
>     that so we don't have to worry about explaining all the state
>     variables here.

OK, I fixed all these things in my working version.

Also, I replaced all references to RFC2581 to the bis document since it
has been forwarded and seemingly would not block this document.  If
needed we can revert this change if ER somehow passes 2581bis in the
processing. 

> (6) "good to bad retransmits" ->
>     "needed to unneeded retransmissions" ???

I used the language from the cited paper ([Pax97]) and am inclined to
leave it because I think it is clear.  If you want to insist I'll change
it because it doesn't seem worth fighting over.

> (7) check IDNITS :)

Added intended status (experimental).

Added a couple missing refs.

A bunch of the refs flagged as having no reference do in fact have a
reference but the tool doesn't seem to grok [ref1,ref2,ref3] sorts of
constructs.

RFC3522 is not normative and so I moved it.

Updated the NewReno reference to RFC 3782.

There is new boilerplate since we submitted -01.  Big surprise.  This
will be fixed in -02, obviously.

Many thanks!

allman