Re: [Gen-art] Gen-Art review of draft-ietf-netext-access-network-option-10.txt

jouni korhonen <jouni.nospam@gmail.com> Sun, 06 May 2012 19:58 UTC

Return-Path: <jouni.nospam@gmail.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 54EB421F8549; Sun, 6 May 2012 12:58:59 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.599
X-Spam-Level:
X-Spam-Status: No, score=-3.599 tagged_above=-999 required=5 tests=[AWL=0.000, BAYES_00=-2.599, RCVD_IN_DNSWL_LOW=-1]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id fyYtqiaC1HuI; Sun, 6 May 2012 12:58:58 -0700 (PDT)
Received: from mail-wg0-f44.google.com (mail-wg0-f44.google.com [74.125.82.44]) by ietfa.amsl.com (Postfix) with ESMTP id 62C2A21F84DF; Sun, 6 May 2012 12:58:58 -0700 (PDT)
Received: by wgbdr13 with SMTP id dr13so3017735wgb.13 for <multiple recipients>; Sun, 06 May 2012 12:58:57 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=subject:mime-version:content-type:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to:x-mailer; bh=n5/9AYJo7nobC146FVKsqZdRH0862yyQbSeEnTbjgK0=; b=FlhSvRbpCJzN6ATFOElGhTCGoQR54ky8gk6nSn7rQrvdHzUa+OMrmzr37QJ6prJWBH n+08vwcBrpo+2D4WnbYcs0Hl5lrCdUlC6DNemYG/YL4b3y14wJTfgagMVZq1VhVJ8m83 Z2zVGoTrUqi3njXW2BqQopkJUEsE1i1tG1e+8pwNMWC/y8OYlx9GK+Xenzx4JCwIng2r Q6Fl1/sfsar01YEPHb1IDgJw9ns032DUGXtDuAiHAGbtOgLMw0pEVqBfNlOZI1zsvcdJ oXc5g4l+tRmS4PTrlBlSBP132GhiLc/hx9nENix4t4sUXm8OUOhLScIToIxCqQzXZfUh zhPA==
Received: by 10.180.77.4 with SMTP id o4mr29482396wiw.17.1336334337517; Sun, 06 May 2012 12:58:57 -0700 (PDT)
Received: from [188.117.15.106] ([188.117.15.106]) by mx.google.com with ESMTPS id k6sm16524411wiy.7.2012.05.06.12.58.53 (version=TLSv1/SSLv3 cipher=OTHER); Sun, 06 May 2012 12:58:55 -0700 (PDT)
Mime-Version: 1.0 (Apple Message framework v1084)
Content-Type: text/plain; charset="us-ascii"
From: jouni korhonen <jouni.nospam@gmail.com>
In-Reply-To: <E30B4753-60B8-470B-A1DF-E2ABD31D62E0@isode.com>
Date: Sun, 06 May 2012 22:58:52 +0300
Content-Transfer-Encoding: quoted-printable
Message-Id: <47440497-5003-4067-97D5-5426099AFECC@gmail.com>
References: <CADhm99oKj8Trcx5o37kg=xZ85ut1iT4xcog+mBO_GP1T4Xnxqg@mail.gmail.com> <01b401cd13d3$322bc950$96835bf0$@isode.com> <3B677B5C-CC21-4252-B343-C6C4A57BBE5B@isode.com> <021901cd1447$39b550f0$ad1ff2d0$@isode.com> <24541053-C0B1-47B3-B866-19E0909BB176@isode.com> <021f01cd1491$13104820$3930d860$@isode.com> <E30B4753-60B8-470B-A1DF-E2ABD31D62E0@isode.com>
To: Alexey Melnikov <alexey.melnikov@isode.com>
X-Mailer: Apple Mail (2.1084)
Cc: "Gen-art@ietf.org" <Gen-art@ietf.org>, "draft-ietf-netext-access-network-option.all@tools.ietf.org" <draft-ietf-netext-access-network-option.all@tools.ietf.org>, "iesg@ietf.org" <iesg@ietf.org>
Subject: Re: [Gen-art] Gen-Art review of draft-ietf-netext-access-network-option-10.txt
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.12
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: <http://www.ietf.org/mail-archive/web/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: Sun, 06 May 2012 19:58:59 -0000

Alexey,

Thanks for the review. See some initial comments inline.


On May 5, 2012, at 8:58 PM, Alexey Melnikov wrote:

> I am the assigned Gen-ART reviewer for this draft. For background on
> Gen-ART, please see the FAQ at
> <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.
> 
> Please resolve these comments along with any other Last Call comments
> you may receive.
> 
> Document: draft-ietf-netext-access-network-option-10.txt
> Reviewer: Alexey Melnikov
> Review Date: 5 May 2012
> IETF LC End Date: 9 May 2012
> IESG Telechat date: Unknown
> 
> Summary: This draft is ready as a Proposed Standard
> 
> Major issues: none
> 
> Minor issues:
> 
> Minor: [ANI] and [TS23003] seem to be Normative (as per their use in 3.1.1)

If it is OK from the RFC process point of view, we can put these
non-IETF references into the normative section.

> 
> Nits/editorials:
> 
> 1.  Introduction
> 
>    This document defines a new mobility option, the Access Network
>    Identifier (ANI) option and its sub-options for Proxy Mobile IPv6,
>    that can be used by the mobile access gateway to signal the access
>    network information to the local mobility anchor.  The specific
>    details on how the local mobility anchor uses this information are
>    out-of-scope for this document.  These mobility options are optional
>    and are not mandatory for the Proxy Mobile IPv6 protocol.
> 
> Nit: Last sentence: "optional" and "not mandatory" are the same thing on my book.

Proposal for new text:

   "These mobility options are optional for the Proxy Mobile IPv6 protocol."

> 
> Strictly speaking PEN numbers are not limited to 4 bytes. However you have a registry for types of identifiers, so a new value can be allocated for bigger-than-4-bytes PENs.

Right. So we could just remove the 4 octet length requirement and use a 
"natural" length indicated octet coding for the PENs. For example

ANI Length = 1 -> PENs 0-255
ANI Length = 2 -> PENs 0-65535
ANI Length = 3 -> PENs 0-16777216
...

That would be ok?


- Jouni