Re: [6lo] TSV ART early review of draft-ietf-6lo-dect-ule

Joe Touch <touch@isi.edu> Mon, 26 September 2016 21:35 UTC

Return-Path: <touch@isi.edu>
X-Original-To: 6lo@ietfa.amsl.com
Delivered-To: 6lo@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id C042812B350; Mon, 26 Sep 2016 14:35:58 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -9.215
X-Spam-Level:
X-Spam-Status: No, score=-9.215 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, RP_MATCHES_RCVD=-2.316] autolearn=ham autolearn_force=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 O15wIKmup47n; Mon, 26 Sep 2016 14:35:56 -0700 (PDT)
Received: from vapor.isi.edu (vapor.isi.edu [128.9.64.64]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 7C36812B357; Mon, 26 Sep 2016 14:35:56 -0700 (PDT)
Received: from [128.9.184.244] ([128.9.184.244]) (authenticated bits=0) by vapor.isi.edu (8.13.8/8.13.8) with ESMTP id u8QLZJeM001607 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NOT); Mon, 26 Sep 2016 14:35:20 -0700 (PDT)
To: draft-ietf-6lo-dect-ule.all@ietf.org, 6lo@ietf.org
References: <6df03900-789c-1c93-977a-93f0a9d663b3@isi.edu>
From: Joe Touch <touch@isi.edu>
Message-ID: <df3a455b-d60e-fcb7-f7c4-6497b052cd53@isi.edu>
Date: Mon, 26 Sep 2016 14:35:19 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0
MIME-Version: 1.0
In-Reply-To: <6df03900-789c-1c93-977a-93f0a9d663b3@isi.edu>
Content-Type: multipart/alternative; boundary="------------3F59FF95882C097A80989D07"
X-ISI-4-43-8-MailScanner: Found to be clean
X-MailScanner-From: touch@isi.edu
Archived-At: <https://mailarchive.ietf.org/arch/msg/6lo/k2FwTHW_v4XRsfoUPQIVkatUn6s>
Cc: "tsv-art@ietf.org" <tsv-art@ietf.org>
Subject: Re: [6lo] TSV ART early review of draft-ietf-6lo-dect-ule
X-BeenThere: 6lo@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: "Mailing list for the 6lo WG for Internet Area issues in IPv6 over constrained node networks." <6lo.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/6lo>, <mailto:6lo-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/6lo/>
List-Post: <mailto:6lo@ietf.org>
List-Help: <mailto:6lo-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/6lo>, <mailto:6lo-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 26 Sep 2016 21:35:59 -0000

CC'ing tsv-art...


On 9/26/2016 2:34 PM, Joe Touch wrote:
>
> Hi, all,
>
> I've reviewed this document as part of the Transport Area Review
> Team's (TSVART) ongoing effort to review key IETF documents. These
> comments were written primarily for the transport area directors, but
> are copied to the document's authors for their information and to
> allow them to address any issues raised. When done at the time of IETF
> Last Call, the authors should consider this review together with any
> other last-call comments they receive. Please always CC
> ​tsv-art@ietf.org <mailto:tsv-art@ietf.org> if you reply to or forward
> this review.
>
> Please resolve these comments along with any other Last Call comments
> you may receive. Please wait for direction from your document shepherd
> or AD before posting a new version of the draft.
>
> Document: draft-ietf-6lo-dect-ule 
> Transmission of IPv6 Packets over DECT Ultra Low Energy
> Reviewer: J. Touch
> Review Date: Sept 26, 2016
> IETF Last Call Date: TBD
>
> Summary: The document impacts transport protocols only indirectly
> through its need to support the minimum required IPv6 MTU, but is
> imprecise about the corresponding details of those requirements.
>
> Major issues: There are no major TSV issues in this document.
>
> Minor issues: The following modification should be considered for
> increased clarity:
>
> Sec 2.4 indicates that the default MTU for DECT UL is 500 octets and
> does note that:
>
>    ...In order to
>    support complete IP packets, the DLC layer of DECT ULE SHALL per this
>    specification be configured with a MTU size that fits the
>    requirements from IPv6 data packets, hence [RFC4944 <https://tools.ietf.org/html/rfc4944>] fragmentation/
>    reassembly is not required.
>
> It would be useful to update this text to be more clear about this
> referring to IPv6 only (as per the title of the doc), to indicate the
> minimum MTU, and to explain the reference to RFC4944:
>
>     ...In order to support IPv6, the DLC layer of DECT ULE MUST be
>     configured with a MTU of at least 1280B to avoid the need for
>     an RFC4944-style shim layer for additional support for larger
>     payload fragmentation/reassembly [RFC4944]."
>
> ---
>