[Gen-art] Gen-ART LC review of draft-ietf-mpls-multipath-use-03

"Peter Yee" <peter@akayla.com> Sat, 18 January 2014 07:21 UTC

Return-Path: <peter@akayla.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 22CD71A1F56; Fri, 17 Jan 2014 23:21:44 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 0.4
X-Spam-Level:
X-Spam-Status: No, score=0.4 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, MANGLED_LIST=2.3, RCVD_IN_DNSWL_NONE=-0.0001] autolearn=no
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 G1tcia87JT9x; Fri, 17 Jan 2014 23:21:42 -0800 (PST)
Received: from p3plsmtpa07-09.prod.phx3.secureserver.net (p3plsmtpa07-09.prod.phx3.secureserver.net [173.201.192.238]) by ietfa.amsl.com (Postfix) with ESMTP id 5B72B1A1F1F; Fri, 17 Jan 2014 23:21:42 -0800 (PST)
Received: from spectre ([173.8.184.78]) by p3plsmtpa07-09.prod.phx3.secureserver.net with id FKMR1n0061huGat01KMRgb; Sat, 18 Jan 2014 00:21:29 -0700
From: Peter Yee <peter@akayla.com>
To: draft-ietf-mpls-multipath-use.all@tools.ietf.org
Date: Fri, 17 Jan 2014 23:21:28 -0800
Message-ID: <02d101cf141d$ea1a8f50$be4fadf0$@akayla.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
X-Mailer: Microsoft Outlook 14.0
Thread-Index: Ac8UHcdwianJ3UTFS4ylM1d/AUj2Sg==
Content-Language: en-us
Cc: gen-art@ietf.org, ietf@ietf.org
Subject: [Gen-art] Gen-ART LC review of draft-ietf-mpls-multipath-use-03
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 18 Jan 2014 07:21:44 -0000

I am the assigned Gen-ART reviewer for this draft. For background on
Gen-ART, please see the FAQ at
<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>

Please resolve these comments along with any other Last Call comments you
may receive.

Document: draft-ietf-mpls-multipath-use-03
Reviewer: Peter Yee
Review Date: January-16-2014
IETF LC End Date: January-17-2014
IESG Telechat date: January-23-2014

Summary: This draft is basically ready for publication as an Informational
RFC, but has nits that should be fixed before publication. [Ready with
nits.]

This document discusses multipath MPLS operations and describes the means of
supporting MPLS as a server layer for MPLS-TP and vice versa.  It lists
requirements and implications for these operations.


Nits:

General nits:

Make all references to the expansion of ECMP read "Equal-Cost Multipath" for
consistency with RFC 2991.

In the usage RFCxxxx, use a space between RFC and xxxx except when used as a
reference, e.g., [RFCxxxx].

There are many unexplained acronyms without definition or reference.  Unless
these are abundantly clear to most readers, expand them on first usage and
provide a reference pointer where feasible.

Specific nits:

Page 1, Abstract, 2nd paragraph, 1st sentence: change "MPLS Entropy label"
to "the MPLS Entropy Label".

Page 2, Section 1, 1st paragraph,  1st sentence: delete the comma after
"traffic".

Page 2, Section 1, 1st paragraph, last sentence: change "load split" to
"load splitting".

Page 2, Section 1, 2nd paragraph, last sentence: hyphenate "fate sharing".

Page 3, Section 1, 1st paragraph, 1st sentence: hyphenate "equal cost".

Page 3, definition of "Loop Free Alternate Paths (LFA)": hyphenate "Loop
Free".

Page 3, definition of "Loop Free Alternate Paths (LFA), 1st sentence: change
the terminating period to a colon.

Page 4, Section 3.1, 1st paragraph, 1st sentence: change "date" to "data".

Page 4, Section 3.1, 1st paragraph, 2nd sentence: change the terminating
period to a colon.

Page 5, 1st full paragraph, 1st sentence: insert "Section 3.1.1" between
"[RFC5960]" and "paragraph 3".  Capitalize "paragraph".

Page 5, 1st full paragraph, 3rd sentence: same as previous instructions.

Page 5, 2nd paragraph, 2nd sentence: change ' "Section 1 Introduction"' to '
Section 1 ("Introduction")'.

Page 6, 1st paragraph, 3rd sentence: change ' "Section 2.9.4 Equal Cost
Multipath") to ' Section 2.9.4 ("Equal Cost Multipath")'.

Page 6, Section 3.2, 1st paragraph, 1st sentence: hyphenate "MPLS-TP
conformant" and delete the comma after "multipath".

Page 7, 2nd paragraph, 2nd sentence: change "A" to "An".

Page 7, 2nd paragraph, 5th and 6th sentences: change all occurrences of
"LSP" to "LSPs".

Page 7, 3rd paragraph, 2nd sentence: change "LSP" to "LSPs".

Page 7, 4th paragraph, 2nd sentence: change "LSR" to "LSRs".

Page 7, 5th paragraph: change all occurrences of "LSP" to "LSPs".

Page 7, 5th paragraph last sentence: change "co-existance" to "coexistence".

Page 8, 1st partial paragraph, 2nd full sentence: change all occurrences of
"LSP" to "LPSs" and change "LSR" to "LSRs".

Page 8, 2nd full paragraph: change all occurrences of "LSP" to "LSPs".

Page 8, 4th paragraph, 1st sentence: change "LSP" to "LSPs".

Page 8, 4th paragraph, 2nd sentence: is there a quantitative definition for
"very infrequent" that can be substituted or explained?

Page 8, 4th paragraph, 3rd sentence: change "presense" to "presence".
Insert "a" before "GAL".

Page 9, 1st partial paragraph: delete "label" following "EL" in two
locations.

Page 9, Section 4, 1st paragraph, 1st sentence: change first "LSP" to
"LSPs".  Change "a MPLS-TP" to "an MPLS-TP" unless MPLS is pronounced liked
"mipples" or something similar and not just spelled out.

Page 9, Section 4, 2nd paragraph: replace all occurrences of "LSP" with
"LSPs".

Page 9, Section 4, 2nd paragraph, 2nd sentence: insert "the" between "in"
and "number".

Page 9, Section 4, 2nd paragraph, 3rd sentence: change "ILM" to "ILMs".

Page 9, Section 4, 3rd paragraph, 1st sentence: change "Server Layer" to
"server layer" for consistency of usage.  Insert "of" between "use" and
"MPLS-TP".  Change "LSP" to "LSPs".

Page 9, Section 4, 3rd paragraph, 4th sentence: change "LSP" to "LSPs".

Page 9, Section 4, 3rd paragraph, 5th sentence: change "LSP" to "LSPs".
Change "capacity" to "capacities".  Insert "generally" before "not integer"
unless it is always the case that the capacity is not a multiple of 10 Gb/s.

Page 9, Section 4, 4th paragraph, 1st sentence: change "Server Layer" to
"server layer".

Page 10, Section 6, 2nd paragraph, 2nd sentence: hyphenate "high end" and
append a comma after that.  Hyphenate "packet processing".

Page 10, Section 6, 3rd paragraph, last sentence: same as previous
instructions.

Page 10, Section 8, 1st sentence: insert "a" between "of" and "framework".


I'm not particularly knowledgeable about MPLS, so I'm unable to express much
about the correctness of the operational points made in the document.

		-Peter Yee