[Gen-art] Gen-art LC review of draft-ietf-manet-rfc6779bis-05

Elwyn Davies <elwynd@dial.pipex.com> Fri, 06 May 2016 13:30 UTC

Return-Path: <elwynd@dial.pipex.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 8797E12D5C9; Fri, 6 May 2016 06:30:27 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.599
X-Spam-Level:
X-Spam-Status: No, score=-102.599 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7, USER_IN_WHITELIST=-100] 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 6jor5-tCrNYe; Fri, 6 May 2016 06:30:23 -0700 (PDT)
Received: from b.painless.aa.net.uk (b.painless.aa.net.uk [IPv6:2001:8b0:0:30:5054:ff:fe5e:1643]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 8EC4312D5A7; Fri, 6 May 2016 06:30:23 -0700 (PDT)
Received: from 5.2.f.6.4.7.f.1.3.3.2.b.c.6.0.a.1.0.0.0.f.b.0.0.0.b.8.0.1.0.0.2.ip6.arpa ([2001:8b0:bf:1:a06c:b233:1f74:6f25]) by b.painless.aa.net.uk with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.77) (envelope-from <elwynd@dial.pipex.com>) id 1ayfpk-0004PG-Tg; Fri, 06 May 2016 14:30:21 +0100
To: General area reviewing team <gen-art@ietf.org>
From: Elwyn Davies <elwynd@dial.pipex.com>
Message-ID: <572C9C6D.9080509@dial.pipex.com>
Date: Fri, 06 May 2016 14:30:21 +0100
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.7.2
MIME-Version: 1.0
Content-Type: multipart/alternative; boundary="------------030207080706090402010703"
Archived-At: <http://mailarchive.ietf.org/arch/msg/gen-art/iB0mHMPY8onE0NIFseV14Gb6M-c>
Cc: draft-ietf-manet-rfc6779bis.all@ietf.org
Subject: [Gen-art] Gen-art LC review of draft-ietf-manet-rfc6779bis-05
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 06 May 2016 13:30:28 -0000

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 <​ 
http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq> 
<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq%3E>.

Document: draft-ietf-manet-rfc6779bis-05.txt
Reviewer: Elwyn Davies
Review Date: 2016/05/06
IETF LC End Date: 2016/05/16
IESG Telechat date: (if known) -

Summary: Ready with a couple of editorial nits.

Major issues: None

Minor issues: None

Nits/editorial comments:

The suggestions for the Abstract, s1 and s1.1 are intended to clarify 
the relationship to RFC 7466 in the introductory text (the later 
comments in the MIB itself are more than adequately clear about this!)
Abstract:
OLD:
    In particular, it
    describes objects for configuring parameters of the Neighborhood
    Discovery Protocol (NHDP) process on a router.
NEW:
    In particular, it
    describes objects for configuring parameters of the Neighborhood
    Discovery Protocol (NHDP) process on a router.  The extensions
    described in this document adds objects and values to support the
    NHDP optimisation described in RFC 7466.
END

s1:
OLD:
    In particular, it describes objects for configuring
    parameters of the Mobile Ad Hoc Network (MANET) Neighborhood
    Discovery Protocol (NHDP) [RFC6130] process on a router.
NEW:
    In particular, it describes objects for configuring
    parameters of the Mobile Ad Hoc Network (MANET) Neighborhood
    Discovery Protocol (NHDP) [RFC6130] process on a router.  The 
extensions
    described in this document adds objects and values to support the
    NHDP optimisation described in [RFC7466].
END
s1.1:
It might be worth adding a list of the changes since it is short and 
they are a bit buried:
I think they are:
  - Addition of objects nhdpIib2HopSetN2Lost and 
nhdpIfPerfCounterDiscontinuityTime.
  - Addition of extra value (notConsidered) to nhdp2HopNbrState.
  - Revised full compliance state.

s4:  We don't normally leave IPR statements in finished documents - 
Probably best to leave a RFC Editor instruction to delete the section 
before publication.

s7.3, para 2: The referent of 'this table' is not totally clear:
s/this table/the nhdpInterfaceTable/

s8, top of page 13 - DESCRIPTION below CONTACT INFO, last para:
OLD:
             This version of this MIB module is part of RFC 6779; see
             the RFC itself for full legal notices."
NEW:
             This version of this MIB module is part of RFC xxxx; see
             the RFC itself for full legal notices."

s10, para 1:  There are weasel words here:

> A fuller discussion of MANET network
>     management use cases and challenges will be provided elsewhere.
Has this now happened?  If so a reference would be desirable. Otherwise 
maybe
    A full discussion of MANET network
    management use cases and challenges is beyond the scope of this 
document..