Re: Opsdir telechat review of draft-ietf-6man-ndpioiana-02

Ole Troan <otroan@employees.org> Thu, 08 February 2018 17:39 UTC

Return-Path: <otroan@employees.org>
X-Original-To: ipv6@ietfa.amsl.com
Delivered-To: ipv6@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 18D0812700F; Thu, 8 Feb 2018 09:39:28 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.91
X-Spam-Level:
X-Spam-Status: No, score=-1.91 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01, URIBL_BLOCKED=0.001] 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 1aRX5_wIwJDf; Thu, 8 Feb 2018 09:39:25 -0800 (PST)
Received: from accordion.employees.org (accordion.employees.org [198.137.202.74]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id E97EE127078; Thu, 8 Feb 2018 09:39:25 -0800 (PST)
Received: from h.hanazo.no (96.51-175-103.customer.lyse.net [51.175.103.96]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by accordion.employees.org (Postfix) with ESMTPSA id 61B9B2D5145; Thu, 8 Feb 2018 17:39:24 +0000 (UTC)
Received: from [IPv6:::1] (localhost [IPv6:::1]) by h.hanazo.no (Postfix) with ESMTP id 8504C201BC6BBF; Thu, 8 Feb 2018 18:39:22 +0100 (CET)
From: Ole Troan <otroan@employees.org>
Message-Id: <95310C4E-AC57-4F86-BCC4-EE564CD3CF5E@employees.org>
Content-Type: multipart/signed; boundary="Apple-Mail=_493F802D-884B-4D12-B174-AC88B4E6C1CA"; protocol="application/pgp-signature"; micalg="pgp-sha512"
Mime-Version: 1.0 (Mac OS X Mail 11.2 \(3445.5.20\))
Subject: Re: Opsdir telechat review of draft-ietf-6man-ndpioiana-02
Date: Thu, 08 Feb 2018 18:39:21 +0100
In-Reply-To: <151786957299.25004.5603443267450220799@ietfa.amsl.com>
Cc: ops-dir@ietf.org, draft-ietf-6man-ndpioiana.all@ietf.org, 6man WG <ipv6@ietf.org>
To: Al Morton <acmorton@att.com>
References: <151786957299.25004.5603443267450220799@ietfa.amsl.com>
X-Mailer: Apple Mail (2.3445.5.20)
Archived-At: <https://mailarchive.ietf.org/arch/msg/ipv6/ai7b7Iiip9SGYeq0ikjRfYE7LF4>
X-BeenThere: ipv6@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "IPv6 Maintenance Working Group \(6man\)" <ipv6.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ipv6>, <mailto:ipv6-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ipv6/>
List-Post: <mailto:ipv6@ietf.org>
List-Help: <mailto:ipv6-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ipv6>, <mailto:ipv6-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 08 Feb 2018 17:39:28 -0000

Al,

Thank you for the review!
Inline.

> Reviewer: Al Morton
> Review result: Has Nits
> 
> IPv6 ND PIO Flags IANA considerations
> draft-ietf-6man-ndpioiana-02
> 
> Reviewer: Al Morton
> Review Result: Almost Ready/Has Nits
> 
> I have reviewed this document as part of the Operational directorate's
> ongoing effort to review all IETF documents being processed by the IESG.  These
> comments were written with the intent of improving the operational aspects of the
> IETF drafts. Comments that are not addressed in last call may be included in AD reviews
> during the IESG review.  Document editors and WG chairs should treat these comments
> just like any other last call comments.
> 
> Summary:
> Mobile IPv6 extends Neighbor Discovery to allow a router to advertise
> its global address, by the addition of a single flag bit in the
> format of a Prefix Information option for use in Router Advertisement
> messages. RFC6275 modified the original RFC4861 Neighbor Discovery (ND)
> Prefix Information Option format, allocating a "Reserved1" bit to "R"
> for this purpose, but did not establish an IANA registry to track such
> extensions (and did not indicate this as an update of RFC4681, possibly
> creating issues for further allocation of the "Reserved1" bits (note that
> there are two Reserved fields in the Prefix Information Option, the other
> is denoted "Reserved2").
> 
> This memo requests IANA to create a new registry that brings the
> extension of "Reserved1" bits under control, and therefore solves
> a potential future Operations problem.
> 
> Suggestions in Section 4, IANA Considerations:
> 
> Usually, the reserved bits are indicated in the Registry:
> 
>       +---------------+---------------------------------+-----------+
>       | RA Option Bit | Description                     | Reference |
>       +---------------+---------------------------------+-----------+
>       | 0             | L - On-link Flag                | [RFC4861] |
>       | 1             | A - Autonomous Address          | [RFC4861] |
>       |               |     Configuration Flag          |           |
>       | 2             | R - Router Address Flag         | [RFC6275] |
>       +---------------+---------------------------------+-----------+
> |      | 3-7           | Reserved1                       | [RFC6275] |
>       +---------------+---------------------------------+-----------+

Yes, that makes sense.


> 
>                                 Figure 2
> 
> Also, it is useful to indicate the Registration Procedure in this section,
> such as:
> 
> Registration Procedure(s)
>    Standards Action or IESG Approval
> 
> (IANA will ask for this, and may ask for the explicit registry name, too)

We already have text like that under figure 2:

   The assignment of new flags in the PIO option header require
   standards action.

   The registry for these flags should be added to:
   http://www.iana.org/assignments/icmpv6-parameters


> 
> Nits:
> Updates: 4861  (and also 6275, by establishing the ND PIO Flag registry, right?)

Yes, I guess you could say that. Good to have the meta date anyway.

> 
> Introduction: two explicit section references would have been helpful, such as
> https://tools.ietf.org/html/rfc4861#section-4.6.2
> https://tools.ietf.org/html/rfc6275#section-7.2
> as both are long RFCs.

OK, will add.

Best regards,
Ole