[Tsv-art] Tsvart last call review of draft-ietf-6lo-backbone-router-13

Kyle Rose via Datatracker <noreply@ietf.org> Mon, 03 February 2020 18:57 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: tsv-art@ietf.org
Delivered-To: tsv-art@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id C3C2E120C0B; Mon, 3 Feb 2020 10:57:44 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Kyle Rose via Datatracker <noreply@ietf.org>
To: tsv-art@ietf.org
Cc: last-call@ietf.org, draft-ietf-6lo-backbone-router.all@ietf.org, 6lo@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.116.1
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Kyle Rose <krose@krose.org>
Message-ID: <158075626468.28650.2983535903394534987@ietfa.amsl.com>
Date: Mon, 03 Feb 2020 10:57:44 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/tsv-art/XFwQl0vr4PBHsQ_AOfCBH1IqOCw>
Subject: [Tsv-art] Tsvart last call review of draft-ietf-6lo-backbone-router-13
X-BeenThere: tsv-art@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Transport Area Review Team <tsv-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tsv-art>, <mailto:tsv-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/tsv-art/>
List-Post: <mailto:tsv-art@ietf.org>
List-Help: <mailto:tsv-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tsv-art>, <mailto:tsv-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 03 Feb 2020 18:57:45 -0000

Reviewer: Kyle Rose
Review result: Almost Ready

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.

This document describes a mechanism for efficient neighbor discovery across
links sharing a common IPv6 prefix, i.e., a multi-link subnet.

I see no specifically transport-related issues in this draft, but I note a few
nits as well as unaddressed issues raised by RFC 4903.

Potential issues: The draft addresses ND, DAD, and to some extent link-scoped
multicast and broadcast (sections 2.3 and 2.4 of 4903), but does not address
either the IP Model (section 2.1) or hop limit issue (2.2). For the first, does
the presumption that a multi-link subnet exists as a recognized and supportable
network configuration require update of RFC 4291, which AFAICT is still
authoritative for the statement that:

    "Currently, IPv6 continues the IPv4 model in that a subnet prefix is
    associated with one link."

For the second, since I'm assuming something called a "router" will in fact
decrement the hop limit when forwarding a packet (I couldn't find the answer in
a brief perusal of the references that seemed relevant), for completeness it
might be helpful to mention something about how multicast traffic e.g. with hop
limit 1 will not successfully transit to hosts in the same subnet but on a
different link. In general, making clear that the issues raised in 4903 are
systematically addressed with respect to the unique requirements of 6lo traffic
would be useful to the reader.

Nit: This text is confusing:

      Either respond using NA messages as a proxy or bridge as a unicast
      frame the IPv6 ND messages (multicast DAD and Address Lookup, and
      unicast NUD) received for the Registered Address over the
      Backbone.

In particular, I'm struggling with what the second option here is. Is it that a
6BBR could bridge incoming ND messages to other links? Is it an option in lieu
of the first, or are NA messages always to be proxied and all other messages to
be bridged?

Nit: Please use a single form to specify a multi-link subnet: I see "MultiLink"
and "Multi-Link" used in different places.