[Gen-art] GEN-Art review of draft-ietf-isis-layer2-09

<kathleen.moriarty@emc.com> Wed, 19 January 2011 17:23 UTC

Return-Path: <kathleen.moriarty@emc.com>
X-Original-To: gen-art@core3.amsl.com
Delivered-To: gen-art@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id C265028B56A for <gen-art@core3.amsl.com>; Wed, 19 Jan 2011 09:23:49 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.599
X-Spam-Level:
X-Spam-Status: No, score=-6.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, RCVD_IN_DNSWL_MED=-4]
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 PXfIKIJJCUDW for <gen-art@core3.amsl.com>; Wed, 19 Jan 2011 09:23:48 -0800 (PST)
Received: from mexforward.lss.emc.com (mexforward.lss.emc.com [128.222.32.20]) by core3.amsl.com (Postfix) with ESMTP id A96FD3A7013 for <gen-art@ietf.org>; Wed, 19 Jan 2011 09:23:48 -0800 (PST)
Received: from hop04-l1d11-si02.isus.emc.com (HOP04-L1D11-SI02.isus.emc.com [10.254.111.55]) by mexforward.lss.emc.com (Switch-3.4.3/Switch-3.4.3) with ESMTP id p0JHQREq024339 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Wed, 19 Jan 2011 12:26:27 -0500
Received: from mailhub.lss.emc.com (mailhub.lss.emc.com [10.254.222.226]) by hop04-l1d11-si02.isus.emc.com (RSA Interceptor); Wed, 19 Jan 2011 12:26:18 -0500
Received: from mxhub20.corp.emc.com (mxhub20.corp.emc.com [10.254.93.49]) by mailhub.lss.emc.com (Switch-3.4.3/Switch-3.4.3) with ESMTP id p0JHMeNs026285; Wed, 19 Jan 2011 12:24:15 -0500
Received: from mx06a.corp.emc.com ([169.254.1.172]) by mxhub20.corp.emc.com ([10.254.93.49]) with mapi; Wed, 19 Jan 2011 12:24:13 -0500
From: kathleen.moriarty@emc.com
To: ayabaner@cisco.com, dward@juniper.net
Date: Wed, 19 Jan 2011 12:24:12 -0500
Thread-Topic: GEN-Art review of draft-ietf-isis-layer2-09
Thread-Index: AQHLt/2wAob0rzb7ikmRNGokzKa8iA==
Message-ID: <AE31510960917D478171C79369B660FA0DAD9D6714@MX06A.corp.emc.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
acceptlanguage: en-US
Content-Type: text/plain; charset="Windows-1252"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-EMM-MHVC: 1
Cc: draft-ietf-isis-layer2@tools.ietf.org, gen-art@ietf.org
Subject: [Gen-art] GEN-Art review of draft-ietf-isis-layer2-09
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/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: Wed, 19 Jan 2011 17:23:49 -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-isis-layer2-09

Reviewer: Kathleen Moriarty

Review Date: 19 January 2011

IETF LC End Date:

IESG Telechat date: (if known)



Summary:  The document looks pretty good overall, but there are some edits that are recommended below.


Major issues:



Minor issues:







Nits/editorial comments:

Abstract:

Include name for acronym TLV and IS-IS, both are used before being defined.



Include commas in the following sentence:

Change from: “in this document are generic layer 2 additions and specific ones as needed are”

To: “in this document are generic layer 2 additions and specific ones, as needed, are”



Section 1:

Paragraph 1, Line 5:

Change From: Layer

To: layer



Paragraph 2, Line 2:

Include acronym for PDU



Paragraph 2, Line 3:

Change from: “generic layer 2 additions and specific ones as needed are defined in”

To: “generic layer 2 additions and specific ones, as needed, are defined in”



Section 2:

Line 1:

Change from: “In this section we specify the enhancements”

To either: “This section specifies the enhancements”

Or: “In this section, we specify the enhancements”



Line 2:

Change from: “Layer-2”

To: “layer 2”

OR make the use of this consistent in the document.


Section 2.1:

Bullet 4:

Make use of “Layer-2-IS-IS” consistent in the document.



Paragraph 2:

Note use of Level 1 and Layer 2 in this paragraph to make consistent (dash/no dash, upper or lower case “L”)



Define acronym LSP in first line



Section 2.2:

Line 1:

Change from: “Multi Topology aware”

To: “Multi-Topology Aware”



Bullet 5 (acronym already defined, it can be completely spelled out or use the defined acronym):

Change From: “sub-TLVs: The MT aware Port Capabilities TLV”

To: “sub-TLVs: The MT-PORT_CAP TLV”



Last sentence: Include acronym for IIH.