[tcpm] AD review of PRR draft

Wesley Eddy <wes@mti-systems.com> Mon, 26 November 2012 04:48 UTC

Return-Path: <wes@mti-systems.com>
X-Original-To: tcpm@ietfa.amsl.com
Delivered-To: tcpm@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id EFBC321F8661 for <tcpm@ietfa.amsl.com>; Sun, 25 Nov 2012 20:48:15 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.977
X-Spam-Level:
X-Spam-Status: No, score=-0.977 tagged_above=-999 required=5 tests=[AWL=1.022, BAYES_00=-2.599, J_CHICKENPOX_42=0.6]
Received: from mail.ietf.org ([64.170.98.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 6RBSNTeK5xar for <tcpm@ietfa.amsl.com>; Sun, 25 Nov 2012 20:48:15 -0800 (PST)
Received: from atl4mhob08.myregisteredsite.com (atl4mhob08.myregisteredsite.com [209.17.115.46]) by ietfa.amsl.com (Postfix) with ESMTP id 1C26421F85D6 for <tcpm@ietf.org>; Sun, 25 Nov 2012 20:48:14 -0800 (PST)
Received: from mailpod.hostingplatform.com (mail.networksolutionsemail.com [205.178.146.50]) by atl4mhob08.myregisteredsite.com (8.14.4/8.14.4) with ESMTP id qAQ4mDpj023474 for <tcpm@ietf.org>; Sun, 25 Nov 2012 23:48:14 -0500
Received: (qmail 16923 invoked by uid 0); 26 Nov 2012 04:48:13 -0000
Received: from unknown (HELO ?192.168.1.112?) (wes@mti-systems.com@69.81.143.209) by 0 with ESMTPA; 26 Nov 2012 04:48:13 -0000
Message-ID: <50B2F48A.9040603@mti-systems.com>
Date: Sun, 25 Nov 2012 23:48:10 -0500
From: Wesley Eddy <wes@mti-systems.com>
Organization: MTI Systems
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/20121026 Thunderbird/16.0.2
MIME-Version: 1.0
To: "draft-ietf-tcpm-proportional-rate-reduction@tools.ietf.org"@atl4mhob08.myregisteredsite.com
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: 7bit
Cc: "tcpm@ietf.org" <tcpm@ietf.org>
Subject: [tcpm] AD review of PRR draft
X-BeenThere: tcpm@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
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: <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: Mon, 26 Nov 2012 04:48:16 -0000

I've completed AD review of this document and have a few small
comments, some of which should be cleared up before going to
the IESG review.  It is basically in really good shape, but I
see painful DISCUSS ballots coming if we don't clean up these
small things:


(1) It's really minor, but SACKs report received data, not lost
data.  Lost data is inferred from what wasn't SACK'ed.  A couple
places (like the last sentence of the 3rd paragraph in Section 1)
indicate that SACKs report missing data.  I know what you really
mean, but it's technically wrong.


(2) All the references to 3517 probably made sense when the doc
was first written, but will really confuse the IESG, now that
6675 obsoletes 3517.  I *think* we can change all instances of
3517 to 6675 and they all still work ... I know that it already
says:
"""
The analysis and measurement study presented here predates [RFC6675],
which updates RFC 3517.
"""
How about something like:
"""
The analysis and measurement study presented here predate [RFC6675],
which updates RFC 3517.  Thus, RFC 3517 was used as the basis for
comparisons and some definitions, however, it is believed that the
changes in RFC 6675 do not significantly alter the results.
"""
Would that be true?  If so, we should do this and change the
references.  If not, then we need to do a little bit more
explaining about what in 6675 would change things.


(3) We are constantly seeing the IESG ask "What is the experiment?"
for Experimental RFCs.  Indeed, the WG even asked this during the
last call when we looked at going to Standards Track.  We should
have a paragraph (probably in section 6) that indicates a bit more
why we stuck with Experimental.  A less cryptic version of what
Matt said in the WGLC thread would probably be good:
"""
Finally there are some details in PRR that are effected by Laminar.  I
would rather not go to the effort of fixing the current draft to make
it a proper normative PS, but not totally correct.  (Hint: Yoshifumi
Nishida's "off by one" comment reflects the difference between pipe
and total_pipe).
"""

-- 
Wes Eddy
MTI Systems