[6lo] Tsvart last call review of draft-ietf-6lo-plc-05

Joseph Touch via Datatracker <noreply@ietf.org> Sat, 20 February 2021 01:15 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: 6lo@ietf.org
Delivered-To: 6lo@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 567353A0BC5; Fri, 19 Feb 2021 17:15:14 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Joseph Touch via Datatracker <noreply@ietf.org>
To: tsv-art@ietf.org
Cc: 6lo@ietf.org, draft-ietf-6lo-plc.all@ietf.org, last-call@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 7.26.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <161378371431.3198.14769807764668038244@ietfa.amsl.com>
Reply-To: Joseph Touch <touch@strayalpha.com>
Date: Fri, 19 Feb 2021 17:15:14 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/6lo/pB40QL4pr8MbUJjEr54FCRMHJpQ>
Subject: [6lo] Tsvart last call review of draft-ietf-6lo-plc-05
X-BeenThere: 6lo@ietf.org
X-Mailman-Version: 2.1.29
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: Sat, 20 Feb 2021 01:15:14 -0000

Reviewer: Joseph Touch
Review result: Ready with Issues

This document has been reviewed as part of the transport area review team's
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 and WG to allow them to address any issues raised and also to the IETF
discussion list for information.

When done at the time of IETF Last Call, the authors should consider this
review as part of the last-call comments they receive. Please always CC
tsv-art@ietf.org if you reply to or forward this review.


The only significant transport issue in this doc is the issue of MTU support.
Secs 3.3 and 4.6 refers to underlying frag/reassembly per RFC4944. First, these
sections seem redundant; normative requirements should appear in only one
section if both are retained.

More notably, the use of a 16-bit tag in that spec is already known to be
problematic for IPv4 fragmentation and could cause problems here as well, e.g.,
per RFC4963. This issue should be addressed, notably if there is a reason why a
16-bit tag is considered sufficient for this use it should be stated or some
other shim layer should be proposed with a more robust tag (e.g., 32 bits).

Some minor additional suggestions follow:

The intro refers to “6lo”; this term should be defined before being used. The
scenarios should include a citation if available. Similarly, LLN should be
defined. Work that did not receive consensus might be mentioned elsewhere or
even omitted completely, but seems premature in the intro. Also, “the previous
work” in the last sentence is ambiguous; it would be useful to refer to the
RFCs, the draft, or whatever else to which it refers.

Sec 3 includes “Moreover, recently a new …”, which seems redundant; it might be
just “A new…”. Again, this section (and later ones too) refers to “6lo” as a
category of sorts, which needs to be defined (and included in the
acronym/terminology list).