[Gen-art] Gen-ART review of draft-ietf-bess-mvpn-global-table-mcast-02

Christer Holmberg <christer.holmberg@ericsson.com> Sun, 06 September 2015 13:27 UTC

Return-Path: <christer.holmberg@ericsson.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 109201B2F60 for <gen-art@ietfa.amsl.com>; Sun, 6 Sep 2015 06:27:01 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.2
X-Spam-Level:
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 ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id kxsU3PVoNOOT for <gen-art@ietfa.amsl.com>; Sun, 6 Sep 2015 06:26:58 -0700 (PDT)
Received: from sessmg22.ericsson.net (sessmg22.ericsson.net [193.180.251.58]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D995F1B338A for <gen-art@ietf.org>; Sun, 6 Sep 2015 06:26:57 -0700 (PDT)
X-AuditID: c1b4fb3a-f79136d0000071e2-c0-55ec3f1fd89c
Received: from ESESSHC010.ericsson.se (Unknown_Domain [153.88.253.124]) by sessmg22.ericsson.net (Symantec Mail Security) with SMTP id D5.54.29154.F1F3CE55; Sun, 6 Sep 2015 15:26:56 +0200 (CEST)
Received: from ESESSMB209.ericsson.se ([169.254.9.81]) by ESESSHC010.ericsson.se ([153.88.183.48]) with mapi id 14.03.0248.002; Sun, 6 Sep 2015 15:26:55 +0200
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: "gen-art@ietf.org" <gen-art@ietf.org>
Thread-Topic: Gen-ART review of draft-ietf-bess-mvpn-global-table-mcast-02
Thread-Index: AdDWWJZV/cQ2/rGxQs68/UCmOs3Efg==
Date: Sun, 06 Sep 2015 13:26:55 +0000
Message-ID: <7594FB04B1934943A5C02806D1A2204B37A4C033@ESESSMB209.ericsson.se>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [153.88.183.150]
Content-Type: multipart/alternative; boundary="_000_7594FB04B1934943A5C02806D1A2204B37A4C033ESESSMB209erics_"
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrDLMWRmVeSWpSXmKPExsUyM+Jvja6C/ZtQgzctLBYbL7xjtLj66jOL A5PHkiU/mTy+XP7MFsAUxWWTkpqTWZZapG+XwJXRuOQQW8HWlIoHezewNzC2RnYxcnJICJhI vH02lxnCFpO4cG89WxcjF4eQwFFGiT+3epghnEWMEm8+d7N3MXJwsAlYSHT/0wZpEBHQlJi7 4i0TiM0sUC9x+sJdVhBbWMBV4unqTWwg5SICXhINk4shyvUk/s7qBtvFIqAiMWHmcUYQm1fA V6J5xRYWEJsR6Ibvp9ZAjRSXuPVkPhPEbQISS/ach7pTVOLl43+sELaSxIrtlxgh6vMlmn4f ZoKYKShxcuYTlgmMwrOQjJqFpGwWkjKIuI7Egt2f2CBsbYllC18zw9hnDjxmQhZfwMi+ilG0 OLW4ODfdyEgvtSgzubg4P08vL7VkEyMweg5u+W21g/Hgc8dDjAIcjEo8vArzXocKsSaWFVfm HmKU5mBREudtZnoQKiSQnliSmp2aWpBaFF9UmpNafIiRiYNTqoEx/1rV5t155nF2PT+/Bgv9 iurhSXgnHKgyiWFqZcOvXT5BcZ8nPLxxJ2nlifLqm5bLSr4f6tZ7m9dwLeUwC5e45D71vtm6 2zvmTD8huJMhzONn35Fd0ZcN/+Y+ylKdJWDz7d2FubO6L6WeKpqwPHFqYpqNR9nGrdtdRQtE n7Z45X0/ZfzK2D1MiaU4I9FQi7moOBEA2SrpdX8CAAA=
Archived-At: <http://mailarchive.ietf.org/arch/msg/gen-art/vyAkbfXfua5XaYl_FXKkxYPucBA>
Cc: "draft-ietf-bess-mvpn-global-table-mcast.all@tools.ietf.org" <draft-ietf-bess-mvpn-global-table-mcast.all@tools.ietf.org>
Subject: [Gen-art] Gen-ART review of draft-ietf-bess-mvpn-global-table-mcast-02
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: Sun, 06 Sep 2015 13:27:01 -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>
Document:                                     draft-ietf-bess-mvpn-global-table-mcast-02.txt
Reviewer:                                        Christer Holmberg
Review Date:                                  6 September 2015
IETF LC End Date:                          18 August 2015
IETF Telechat Date:                       3 September 2015
Summary:          The document is well written, and almost ready for publication. I do have a few editorial comments, however, that the authors may want to address.
Major Issues: None
Minor Issues: None
Editorial Issues:

General:
-----------

QG_1:

There are a number of abbreviations which are not expanded on first occurrence, e.g. BGP and PIM. I guess there should also be a reference associated with them?


QG_2:

In a few places throughout the document the text says "The document [RFC7524] extends...", "The document [RFC7524] also defines..." etc.

I suggest to remove "The document".


QG_3:


In a few places throughout the document the text says "procedures of [RFCXXXX]".

I suggest to say "procedures in [RFCXXXX]".


QG_4:

Sometimes the text says "Section X of [RFCXXXX]", and sometimes "[RFCXXX] section X".

Please use consistent terminology.


Section 1:
------------

Q1_1:

In general, I think the Introduction section is very long and detailed. Would it be possible to move some of the stuff to dedicated sections (or, at least sub sections)?


Q1_2:

In the first sentence, should it be "an architecture" instead of "architecture"?


Q1_3:

In a few places the text says "This architecture". Sometimes it is a little difficult to figure out whether that refers to an architecture somewhere else, or something defined in this document.

Would it be possible to say e.g. "The architecture defined in [REFERENCE]...".