[Gen-art] Gen-ART Last Call review of draft-ietf-geopriv-held-measurements-07

Suresh Krishnan <suresh.krishnan@ericsson.com> Wed, 08 May 2013 04:06 UTC

Return-Path: <suresh.krishnan@ericsson.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 CE3B721F8C55 for <gen-art@ietfa.amsl.com>; Tue, 7 May 2013 21:06:07 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -101.855
X-Spam-Level:
X-Spam-Status: No, score=-101.855 tagged_above=-999 required=5 tests=[AWL=0.745, BAYES_00=-2.599, USER_IN_WHITELIST=-100]
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 O1Zov0gBUXXc for <gen-art@ietfa.amsl.com>; Tue, 7 May 2013 21:06:01 -0700 (PDT)
Received: from usevmg21.ericsson.net (usevmg21.ericsson.net [198.24.6.65]) by ietfa.amsl.com (Postfix) with ESMTP id B552921F866E for <gen-art@ietf.org>; Tue, 7 May 2013 21:06:01 -0700 (PDT)
X-AuditID: c6180641-b7f906d000003e3f-6b-5189cf28565e
Received: from EUSAAHC003.ericsson.se (Unknown_Domain [147.117.188.81]) by usevmg21.ericsson.net (Symantec Mail Security) with SMTP id B7.DB.15935.82FC9815; Wed, 8 May 2013 06:06:00 +0200 (CEST)
Received: from eusaamw0712.eamcs.ericsson.se (147.117.20.181) by EUSAAHC003.ericsson.se (147.117.188.81) with Microsoft SMTP Server (TLS) id 14.2.328.9; Wed, 8 May 2013 00:05:59 -0400
Received: from [164.48.125.47] (147.117.20.214) by smtps-am.internal.ericsson.com (147.117.20.181) with Microsoft SMTP Server (TLS) id 8.3.279.1; Wed, 8 May 2013 00:05:59 -0400
Message-ID: <5189CE88.2070902@ericsson.com>
Date: Wed, 08 May 2013 00:03:20 -0400
From: Suresh Krishnan <suresh.krishnan@ericsson.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130329 Thunderbird/17.0.5
MIME-Version: 1.0
To: General Area Review Team <gen-art@ietf.org>, draft-ietf-geopriv-held-measurements.all@tools.ietf.org
X-Enigmail-Version: 1.5.1
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: 7bit
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrBJMWRmVeSWpSXmKPExsUyuXRPoK7G+c5Ag3e96hZ3tl5ht7j66jOL A5PHkiU/mTy+XP7MFsAUxWWTkpqTWZZapG+XwJXxuO83U8FHzoppG/pYGxj/sHcxcnBICJhI nOx17WLkBDLFJC7cW8/WxcjFISRwlFFi159uRghnN6PE387v7CBVQgJbGCWuTZIHaeYV0Jbo vVEBEmYRUJH4t/IQG4jNBjRzw87PTCC2qECYxKr1y5hBbF4BQYmTM5+wgNgiAjkSi6bfB6sX FvCRuHXsOiPEEZISW160g61iFtCTmHK1hRHClpfY/nYOM8QJmhJb13xnncAoMAvJ2FlIWmYh aVnAyLyKkaO0OLUsN93IcBMjMPSOSbA57mBc8MnyEKM0B4uSOG8iV2OgkEB6YklqdmpqQWpR fFFpTmrxIUYmDk6pBkb/x0ELJD4F1SsEzNvan3Zzs++ML45TuRJPv3Y51W1m+vgB19tjp5om +cfETZ9n6ebCyGM74fqbPDHR92cyZhybIPa84OXigmdOB+PMnohbcXjo7fXNipwVrfJRyvjL k5/dE7fdi3pfuWlzho3NjINtV66Uxk8Rt7i+fIKnAM/hs9u+71sRVtSsxFKckWioxVxUnAgA /uvFGwsCAAA=
Subject: [Gen-art] Gen-ART Last Call review of draft-ietf-geopriv-held-measurements-07
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: Wed, 08 May 2013 04:06:07 -0000

I am the assigned Gen-ART reviewer for
draft-ietf-geopriv-held-measurements-07

For background on Gen-ART, please see the FAQ at
<http://www.alvestrand.no/ietf/gen/art/gen-art-FAQ.html>.

Please resolve these comments along with any other Last Call comments
you may receive.

Summary: This draft is almost ready for publication as a Proposed
Standard, but I have a few minor comments that you may wish to address.

* Section 5.2

- The Interface-Id option is the DHCPv6 equivalent of the circuit
identifier defined in RFC3046. Please add a reference to Section 22.18
of RFC3315 that describes this option.

- Is there any specific reason that the giaddr is being specified using
the IPv4-mapped IPv6 address format? From my reading giaddr is of type
bt:ipAddressType and it allows specification of both IPv4 and IPv6
addresses natively.

* Section 8.3 Page 44

IPv4-compatible addresses have been deprecated in the IPv6 addressing
architecture (RFC4291) and need not be supported here.

* Section 8.7 Page 53

I think there may be an off-by-one error here.

 <xs:maxInclusive value="268435456"/>

Shouldn't this be

 <xs:maxInclusive value="268435455"/>

so that the largest value will fit in 28 bits?

Thanks
Suresh