[6lo] Genart last call review of draft-ietf-6lo-blemesh-08

Pete Resnick via Datatracker <noreply@ietf.org> Fri, 04 December 2020 21:05 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 91EA33A0C37; Fri, 4 Dec 2020 13:05:51 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Pete Resnick via Datatracker <noreply@ietf.org>
To: gen-art@ietf.org
Cc: last-call@ietf.org, draft-ietf-6lo-blemesh.all@ietf.org, 6lo@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 7.23.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <160711595155.19658.8436612763808758937@ietfa.amsl.com>
Reply-To: Pete Resnick <resnick@episteme.net>
Date: Fri, 04 Dec 2020 13:05:51 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/6lo/yHLCcGh4D0j6dfMNJf4woEYc8xU>
Subject: [6lo] Genart last call review of draft-ietf-6lo-blemesh-08
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: Fri, 04 Dec 2020 21:05:52 -0000

Reviewer: Pete Resnick
Review result: Ready with Issues

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

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.

Document: draft-ietf-6lo-blemesh-08
Reviewer: Pete Resnick
Review Date: 2020-12-04
IETF LC End Date: 2020-10-21
IESG Telechat date: Not scheduled for a telechat

Summary: Looks good to me, with two items that seemed confusing enough that
they should be addressed.

Note that this review is being done well after the close of the Last Call.
Since this is not yet scheduled for a telechat, the AD asked that I go ahead
and complete the review anyway.

Major issues: None

Minor issues:

3.3.2, #4:

   Implementations of this specification MAY support
   the features described in sections 8.1 and 8.2 of RFC 6775, as
   updated by RFC 8505, unless some alternative ("substitute") from some
   other specification is supported by the implementation.

This bit confused me. I think it was the "unless". Do you mean it MAY support
the 6775/8505 features, or MAY support some substitute, or MAY support neither,
or do you mean that it MUST support either the 6775/8505 features or MUST
support some substitute? I think you want to rewrite this to make it clear
which one you mean.

3.3.3, last two paragraphs:

   When a 6LN transmits a packet, with a non-link-local source address
   that the 6LN has registered with EARO in the next-hop router for the
   indicated prefix, the source address MUST be fully elided if it is
   the latest address that the 6LN has registered for the indicated
   prefix (SAC=1, SAM=11).  If the source non-link-local address is not
   the latest registered by the 6LN, then the 64 bits of the IID SHALL
   be fully carried in-line (SAC=1, SAM=01) or if the first 48 bits of
   the IID match with the latest address registered by the 6LN, then the
   last 16 bits of the IID SHALL be carried in-line (SAC=1, SAM=10).

   When a router transmits a packet to a neighboring 6LN, with a non-
   link-local destination address, the router MUST fully elide the
   destination IPv6 address if the destination address is the latest
   registered by the 6LN with EARO for the indicated context (DAC=1,
   DAM=11).  If the destination address is a non-link-local address and
   not the latest registered, then the 6LN MUST either include the IID
   part fully in-line (DAM=01) or, if the first 48 bits of the IID match
   to the latest registered address, then elide those 48 bits (DAM=10).

Both of these were a bit confusing to me. I think you want to reverse the order
of the last two choices. Say something like (for the first paragraph), "If the
source non-link-local address is not the latest registered by the 6LN and the
first 48 bits match..., then the last 16 bits SHALL be carried in-line.
Otherwise, if first 48 bits do not match, then the 64 bits shall be carried
inline." Similarly for the second. As it is, it takes a while to figure out
what it means.

Nits/editorial comments: Do fix the reference nits noted by the nits tool; they
appear to be simple typos.