Complete Gen-ART Last Call review of draft-ietf-6lo-6lobac-06

Orit Levin <oritl@microsoft.com> Mon, 28 November 2016 19:34 UTC

Return-Path: <oritl@microsoft.com>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id C6339129F8B; Mon, 28 Nov 2016 11:34:57 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.022
X-Spam-Level:
X-Spam-Status: No, score=-2.022 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H4=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=microsoft.com
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 6UwDSAAFfqCi; Mon, 28 Nov 2016 11:34:55 -0800 (PST)
Received: from NAM01-SN1-obe.outbound.protection.outlook.com (mail-sn1nam01on0112.outbound.protection.outlook.com [104.47.32.112]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 6716012950B; Mon, 28 Nov 2016 11:34:55 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=q0L+ovgWK8G9e17HK6AIL3GRTP9DrYwKAGa66bhnMo0=; b=mFXliMIc7kSuvirv3RiZSaySlBCvXT4YUkdhX7HMnKb3SsM1pBAH8Vxh20R6R323AdQETMyemM0HiBEuNHLKCrmU2Zn3HXJeU0GoPAKCpmwUf5hSpLIDXmR5dZDiKxMvGCQK+R8+hArH0nMznYt1tEHfEcgX9Kl0QCCTEAvXE3M=
Received: from BY2PR0301MB2120.namprd03.prod.outlook.com (10.163.197.149) by BY2PR0301MB2118.namprd03.prod.outlook.com (10.163.197.147) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.747.13; Mon, 28 Nov 2016 19:34:53 +0000
Received: from BY2PR0301MB2120.namprd03.prod.outlook.com ([10.163.197.149]) by BY2PR0301MB2120.namprd03.prod.outlook.com ([10.163.197.149]) with mapi id 15.01.0747.015; Mon, 28 Nov 2016 19:34:53 +0000
From: Orit Levin <oritl@microsoft.com>
To: "gen-art@ietf.org" <gen-art@ietf.org>
Subject: Complete Gen-ART Last Call review of draft-ietf-6lo-6lobac-06
Thread-Topic: Complete Gen-ART Last Call review of draft-ietf-6lo-6lobac-06
Thread-Index: AdJJrlJbwU2izo0jQJCvu3YZnXl5pA==
Date: Mon, 28 Nov 2016 19:34:53 +0000
Message-ID: <BY2PR0301MB21202FDAC6578BBECBC4A86AAD8A0@BY2PR0301MB2120.namprd03.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=oritl@microsoft.com;
x-originating-ip: [50.232.32.226]
x-ms-office365-filtering-correlation-id: 39a9dc06-560b-467d-b954-08d417c5a08a
x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001); SRVR:BY2PR0301MB2118;
x-microsoft-exchange-diagnostics: 1; BY2PR0301MB2118; 7:Dh4IEXrDQaYeBI/34w82qtfbez146eOrYerQUMYa75QPs/SPseFXYWENXN10sA2AZTBrEN0OknHS1ue7bun1ngrg4Us3UKJEeceE+eBT6BybGp7NiOHp7wSogQ7xxXr/rPgOgMDZFioS09xuGjM+r/sE9gChmHSMclFhWjSf+VpZ700fstAaJTePSmPAtpu1SNjHN89BlagDSmRTQXPNATiiRYSGz8RG6+uhrHJ7VH5gJisvv1Fvhd00gsBlIwGc1CXfFpHVJBfWTbtkkYDdAH7EPsAriXXeSHidpyqjbBiSmgBtdmFzz4vWsbc8bZCioc4fPglcgWYfkbTwAMx31EwOqS1fBpKvqJB5Jje0vKgX0SFgE7kg7nd/V0nPEfs+3PaPa++FdctZ56khOFTm6wSKo9IebKMnaK8nuxvMwmyTaitpTPr6uFpZnohYVkQlWq17kKEyaTzxZa4UeLKOkAdx+6llDkZa5VSWb3Qlta0=
x-microsoft-antispam-prvs: <BY2PR0301MB2118F44471EE6027C85157A9AD8A0@BY2PR0301MB2118.namprd03.prod.outlook.com>
x-exchange-antispam-report-test: UriScan:(131327999870524);
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(61425038)(6060326)(6040361)(6045199)(601004)(2401047)(8121501046)(5005006)(10201501046)(3002001)(6055026)(61426038)(61427038)(6046074)(6041248)(6061324)(20161123562025)(20161123564025)(20161123560025)(20161123555025)(6047074)(6072148); SRVR:BY2PR0301MB2118; BCL:0; PCL:0; RULEID:; SRVR:BY2PR0301MB2118;
x-forefront-prvs: 01401330D1
x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(6009001)(7916002)(189002)(377424004)(199003)(92566002)(76576001)(230783001)(106356001)(4001150100001)(86612001)(74316002)(105586002)(99286002)(9686002)(3280700002)(3660700001)(7736002)(7846002)(305945005)(10090500001)(86362001)(8676002)(5005710100001)(39400400001)(6506003)(97736004)(7696004)(66066001)(110136003)(39450400002)(101416001)(6916009)(33656002)(81166006)(6116002)(38730400001)(5640700001)(81156014)(8936002)(2900100001)(5660300001)(10290500002)(8990500004)(122556002)(102836003)(3846002)(2501003)(4326007)(189998001)(2351001)(68736007)(50986999)(54356999)(77096006)(2906002)(39380400001)(39410400001); DIR:OUT; SFP:1102; SCL:1; SRVR:BY2PR0301MB2118; H:BY2PR0301MB2120.namprd03.prod.outlook.com; FPR:; SPF:None; PTR:InfoNoRecords; MX:1; A:1; LANG:en;
received-spf: None (protection.outlook.com: microsoft.com does not designate permitted sender hosts)
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: microsoft.com
X-MS-Exchange-CrossTenant-originalarrivaltime: 28 Nov 2016 19:34:53.2972 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 72f988bf-86f1-41af-91ab-2d7cd011db47
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY2PR0301MB2118
Archived-At: <https://mailarchive.ietf.org/arch/msg/ietf/qB5h1DGpDMnU4M61jTr7dNrX6JI>
Cc: "draft-ietf-6lo-6lobac.all@tools.ietf.org" <draft-ietf-6lo-6lobac.all@tools.ietf.org>, "ietf@ietf.org" <ietf@ietf.org>
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 28 Nov 2016 19:34:57 -0000

This is the complete version of my review.
Comments to sections 6 and higher have been added. No changes to other sections have been made.
Sorry for any possible confusion!
--------------------------------------------

I am the assigned Gen-ART reviewer for this draft. The General Area Review Team (Gen-ART) reviews all IETF documents being processed by the IESG for the IETF Chair.  Please treat these comments just like any other last call comments.

For more information, please see the FAQ at

<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Document: draft-ietf-6lo-6lobac-06
Reviewer: Orit Levin
Review Date: 2016-11-26
IETF LC End Date: 2016-11-30
IESG Telechat date: (if known)

Summary:
This draft is basically ready for publication, but has editorial nits that should be fixed before publication.

Abstract:
Remove the word "extensively" from the first sentence.

Introduction:
1. Remove the word "extensively" from the first sentence. (Not a standard-appropriate language.) 2.Consider rephrasing to "... these constraints are similar to those faced in 6LoWPAN networks, which suggests that some elements of that solution might be leveraged."
3. Consider rephrasing the last sentence to "This document also specifies a mandatory header compression mechanism, based on [RFC6282], which reduces latency and makes IPv6 practical on MS/TP networks."

Section 1.3
1. This section is called "MS/TP Overview". The overview of the existing specifications is "mingled" with the new features and profiling defined in "this specification". By just reading this section, it is not always clear which statements refer to the "baseline" specifications and which to the new "features" defined in this document. Either consider introducing/improving "linking" sentences to clarify the text or reorganize/split the text into two independent summaries: of baseline functionality and of new functionality. 
2. In the second paragraph, rephrase to "These features make MS/TP a cost-effective field bus applicable to building an automation network." (Not a standard-appropriate language: "for the most numerous and least expensive devices".) 3. Add the word "only" to "A master node may initiate the transmission of a data frame only when it holds the token."
4. Consider changing "MS/TP COBS-encoded frame fields have the following descriptions:" to "MS/TP COBS-encoded frame fields are defined as follows:"
5. Remove "MUST"s from "Frame Types 32 - 127 designate COBS-encoded frames and MUST convey Encoded Data and Encoded CRC-32K fields.  All master nodes MUST understand Token, Poll For Master, and Reply to Poll For Master control frames." (See my first comment to this section above. Where is this defined? In the baseline specs or in this document?)

Section 3
Rephrase to "The method specified in Section 6 for creating a MAC-layer-derived Interface Identifier (IID) ensures that an IID of all zeros can never be generated."

Section 4
Consider rephrasing to "This specification restricts an MSDU length for at least 1280 octets and at most 1500 octets (before encoding)."

Section 5
1. Rephrase to "Because of the relatively low data rates of MS/TP, header compression is used as a means to reduce latency."
2. Add "of" after "comprises" in "The encapsulation format defined in this section ... comprises of the MSDU of an IPv6 over MS/TP frame."
3. In "The Dispatch value may be treated as an unstructured namespace", it would be simpler to say "is treated" unless there is a special significance to "may be". In later case, it needs to be explained.

Section 6
Consider replacing ", as" by "and is" in "The general procedure for creating a MAC-address-derived IID is described in [RFC4291] Appendix A, "Creating Modified EUI-64 Format Interface Identifiers", as updated by [RFC7136]."

Section 10
Consider replacing the second and the third sentences with "This section provides the text substitutions for [RFC6282] that make it applicable to LoBAC as follows:"

Section 12
Consider rephrasing to "MS/TP networks are by definition wired and thus not susceptible to casual eavesdropping. Furthermore, because MS/TP nodes are stationary, correlation of activities or location tracking of individuals is unlikely."

Thank you,
Orit Levin. 

_______________________________________________
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art