[ippm] AD Review: draft-ietf-ippm-twamp-session-cntrl-03

Lars Eggert <lars.eggert@nokia.com> Fri, 26 February 2010 13:55 UTC

Return-Path: <lars.eggert@nokia.com>
X-Original-To: ippm@core3.amsl.com
Delivered-To: ippm@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 771283A87B2 for <ippm@core3.amsl.com>; Fri, 26 Feb 2010 05:55:09 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.57
X-Spam-Level:
X-Spam-Status: No, score=-6.57 tagged_above=-999 required=5 tests=[AWL=0.029, BAYES_00=-2.599, RCVD_IN_DNSWL_MED=-4]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id SL5ux7soBoFB for <ippm@core3.amsl.com>; Fri, 26 Feb 2010 05:55:08 -0800 (PST)
Received: from mgw-mx09.nokia.com (smtp.nokia.com [192.100.105.134]) by core3.amsl.com (Postfix) with ESMTP id 97C6E3A87B1 for <ippm@ietf.org>; Fri, 26 Feb 2010 05:55:08 -0800 (PST)
Received: from vaebh106.NOE.Nokia.com (vaebh106.europe.nokia.com [10.160.244.32]) by mgw-mx09.nokia.com (Switch-3.3.3/Switch-3.3.3) with ESMTP id o1QDuXs8021361 for <ippm@ietf.org>; Fri, 26 Feb 2010 07:57:22 -0600
Received: from vaebh104.NOE.Nokia.com ([10.160.244.30]) by vaebh106.NOE.Nokia.com with Microsoft SMTPSVC(6.0.3790.3959); Fri, 26 Feb 2010 15:56:56 +0200
Received: from mgw-sa01.ext.nokia.com ([147.243.1.47]) by vaebh104.NOE.Nokia.com over TLS secured channel with Microsoft SMTPSVC(6.0.3790.3959); Fri, 26 Feb 2010 15:56:55 +0200
Received: from mail.fit.nokia.com (esdhcp030222.research.nokia.com [172.21.30.222]) by mgw-sa01.ext.nokia.com (Switch-3.3.3/Switch-3.3.3) with ESMTP id o1QDur0K012524 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO) for <ippm@ietf.org>; Fri, 26 Feb 2010 15:56:55 +0200
From: Lars Eggert <lars.eggert@nokia.com>
X-Virus-Status: Clean
X-Virus-Scanned: clamav-milter 0.95.3 at fit.nokia.com
Content-Type: multipart/signed; boundary="Apple-Mail-12--1024898690"; protocol="application/pkcs7-signature"; micalg="sha1"
Date: Fri, 26 Feb 2010 14:56:33 +0100
Message-Id: <53E0E1D0-3472-4DED-8A81-845E29B07132@nokia.com>
To: IETF IPPM WG <ippm@ietf.org>
Mime-Version: 1.0 (Apple Message framework v1077)
X-Mailer: Apple Mail (2.1077)
X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.2.3 (mail.fit.nokia.com [0.0.0.0]); Fri, 26 Feb 2010 15:56:43 +0200 (EET)
X-OriginalArrivalTime: 26 Feb 2010 13:56:56.0003 (UTC) FILETIME=[8EA1F930:01CAB6EB]
X-Nokia-AV: Clean
Subject: [ippm] AD Review: draft-ietf-ippm-twamp-session-cntrl-03
X-BeenThere: ippm@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: IETF IP Performance Metrics Working Group <ippm.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/ippm>, <mailto:ippm-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ippm>
List-Post: <mailto:ippm@ietf.org>
List-Help: <mailto:ippm-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ippm>, <mailto:ippm-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 26 Feb 2010 13:55:09 -0000

Summary: Almost ready. Issues below are straightforward to address - either revise the document or provide me with an RFC Editor Note.

Section 1., paragraph 4:
>    Implementers of this feature may also wish to implement the "Reflect
>    Octets" feature, described in [I-D.ietf-ippm-twamp-reflect-octets].

  Suggest to add something like "once the RFC has been published."


Section 1., paragraph 6:
>    This memo is intended to be an update to the TWAMP RFC.

  Then you need an "Updates: 5357" header in the boilerplate. It would
  also be good to clarify in the text that that means that this
  extensions becomes mandatory to implement for all TWMP implementations
  (i.e., it is non-optional.) Oh, and add a reference to RFC5357 to this
  statement.


Section 3.7., paragraph 2:
>    If this feature is adopted, the last ???? bits of the Modes 32-bit

  Do you mean "used" instead of "adopted"? Also, ???? needs to be
  clarified.


Section 8.2., paragraph 2:
>    [x]        "".

  Remove.