[Gen-art] Gen-ART Last Call review of draft-ietf-trill-rfc7180bis-06

Meral Shirazipour <meral.shirazipour@ericsson.com> Tue, 20 October 2015 00:02 UTC

Return-Path: <meral.shirazipour@ericsson.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com []) by ietfa.amsl.com (Postfix) with ESMTP id C73A21B2E51 for <gen-art@ietfa.amsl.com>; Mon, 19 Oct 2015 17:02:35 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.2
X-Spam-Status: No, score=-4.2 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001] autolearn=ham
Received: from mail.ietf.org ([]) by localhost (ietfa.amsl.com []) (amavisd-new, port 10024) with ESMTP id SWW-yZponnzL for <gen-art@ietfa.amsl.com>; Mon, 19 Oct 2015 17:02:33 -0700 (PDT)
Received: from usevmg21.ericsson.net (usevmg21.ericsson.net []) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id BD9B71B2BB4 for <gen-art@ietf.org>; Mon, 19 Oct 2015 17:02:33 -0700 (PDT)
X-AuditID: c6180641-f792c6d00000686a-e0-562517f5f75a
Received: from EUSAAHC002.ericsson.se (Unknown_Domain []) by usevmg21.ericsson.net (Symantec Mail Security) with SMTP id BE.2B.26730.5F715265; Mon, 19 Oct 2015 18:19:01 +0200 (CEST)
Received: from EUSAAMB107.ericsson.se ([]) by EUSAAHC002.ericsson.se ([]) with mapi id 14.03.0248.002; Mon, 19 Oct 2015 20:02:32 -0400
From: Meral Shirazipour <meral.shirazipour@ericsson.com>
To: "draft-ietf-trill-rfc7180bis.all@tools.ietf.org" <draft-ietf-trill-rfc7180bis.all@tools.ietf.org>, "gen-art@ietf.org" <gen-art@ietf.org>
Thread-Topic: Gen-ART Last Call review of draft-ietf-trill-rfc7180bis-06
Thread-Index: AdEKylYB8mLqYqOZTdu3xN6/R7wo/Q==
Date: Tue, 20 Oct 2015 00:02:30 +0000
Message-ID: <ABCAA4EF18F17B4FB619EA93DEF7939A4544BA5E@eusaamb107.ericsson.se>
Accept-Language: en-US
Content-Language: en-US
x-originating-ip: []
Content-Type: multipart/alternative; boundary="_000_ABCAA4EF18F17B4FB619EA93DEF7939A4544BA5Eeusaamb107erics_"
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFrrCLMWRmVeSWpSXmKPExsUyuXSPn+5XcdUwg1mX1Cz6r2xls7j66jOL A5PHkiU/mTy+XP7MFsAUxWWTkpqTWZZapG+XwJUx++oF9oJLHhUPn09nbGBcYN/FyMkhIWAi sfTcPlYIW0ziwr31bCC2kMBRRokXi126GLmA7OWMElve/mcCSbAJWEhs//2cFSQhIjCBUeL7 9Z9AHRwcwgLOEqc+CoLUiAh4SMxYfIEZwtaTaG57B2azCKhK7PswkxHE5hXwlVh17AxYnBFo 8fdTa8DmMwuIS9x6Mp8J4iABiSV7zjND2KISLx//gzpUUWJf/3R2iPp8ib71V6BmCkqcnPmE ZQKj0Cwko2YhKZuFpAwiriOxYPcnNghbW2LZwtfMMPaZA4+ZkMUXMLKvYuQoLU4ty003MtzE CIyGYxJsjjsYF3yyPMQowMGoxMOb8F0lTIg1say4MvcQozQHi5I477wZ90OFBNITS1KzU1ML Uovii0pzUosPMTJxcEo1MIbk8h6MbsrYtExVyyBf4Z2A4d8fhk/niJpOf7+r86GTzhuee63/ M455HZhdWX8kM7ptLd/68tklh19rVDtem1e9UuD25OQleY8sLmmUrr0efGNp0L/oiKPNsjNc Z5YZXK6YIHvxi0GZ7ZEtvC5q2zpf3jH76MKdcUGxfOLEjPYYj0MSD5fenarEUpyRaKjFXFSc CAApxVHgZwIAAA==
Archived-At: <http://mailarchive.ietf.org/arch/msg/gen-art/Puzf-j4J593mqA6F3Xe2puqZmjQ>
Subject: [Gen-art] Gen-ART Last Call review of draft-ietf-trill-rfc7180bis-06
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: <https://mailarchive.ietf.org/arch/browse/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: Tue, 20 Oct 2015 00:02:35 -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-trill-rfc7180bis-06
Reviewer: Meral Shirazipour (was originally assigned to another gen-art)
Review Date: 2015-10-19
IETF LC End Date:  2015-10-19
IESG Telechat date: 2015-10-22

This draft is ready to be published as Standards Track RFC but I have some comments .

Major issues:

Minor issues:
-[Page 5], Section 1.1: this section updates section 1.2 of RFC6325. The update is about conflict resolution between sections of the RFC.
Shouldn't this bis highlight those conflicts if any?

-[Page 14], Section 3.4. Should this section have a MUST sentence just before the last sentence?
"All RBridges in a campus MUST determine distribution trees in the same way "

-[Page 10], Section , gives an example, then the first bullet after the figure explains the problem with that scenario and says "MUST NOT be locally distributed in native form ".

Is it possible to clarify what should be done instead?

-[Page 11], last line, "forwards the packet on that tree."
Just checking if that is supposed to say "packet" or if it should say "frame" or "TRILL Data packet"?

Naming ("frame" or "TRILL Data packet") are used throughout,  but it would help to mention the convention at the beginning of the draft.

Nits/editorial comments:
-[Page 6], Section 1.3,  "RBridge - An alternative name for a TRILL Switch."
To remain true to RFC7325, better to add Routing Bridge: "RBridge - Routing Bridge, an alternative name for a TRILL Switch."

-[Page 15], Section 3.6 , "can implemented"--typo-->"can implement"

-[Page 16], Section 3.6.1 , "program their hardware tables",
is it assumed that TRILL fast path will only/always be HW based?

-[Page 17], "RB1 is show with three ports"--typo-->"RB1 is shown with three ports"

-[Page 34], "then behavior is as specified"---> "the behavior" or "then the behavior"

-[Page 35], Section 10.2.2, "those capabilites"--typo-->"those capabilities"

Best Regards,
Meral Shirazipour