Re: [Gen-art] Gen-ART Telechat review of draft-ietf-ippm-type-p-monitor-02.txt

Suresh Krishnan <suresh.krishnan@ericsson.com> Thu, 22 October 2015 04:03 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 (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D9A1C1B36E9; Wed, 21 Oct 2015 21:03:11 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.201
X-Spam-Level:
X-Spam-Status: No, score=-4.201 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001] autolearn=ham
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 CsFNLkEGRPY0; Wed, 21 Oct 2015 21:03:09 -0700 (PDT)
Received: from usevmg21.ericsson.net (usevmg21.ericsson.net [198.24.6.65]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id ACCE81B2D90; Wed, 21 Oct 2015 21:03:09 -0700 (PDT)
X-AuditID: c6180641-f792c6d00000686a-b6-5627f33ce047
Received: from EUSAAHC003.ericsson.se (Unknown_Domain [147.117.188.81]) by usevmg21.ericsson.net (Symantec Mail Security) with SMTP id A8.88.26730.C33F7265; Wed, 21 Oct 2015 22:19:09 +0200 (CEST)
Received: from EUSAAMB107.ericsson.se ([147.117.188.124]) by EUSAAHC003.ericsson.se ([147.117.188.81]) with mapi id 14.03.0248.002; Thu, 22 Oct 2015 00:03:08 -0400
From: Suresh Krishnan <suresh.krishnan@ericsson.com>
To: Gregory Mirsky <gregory.mirsky@ericsson.com>, "draft-ietf-ippm-type-p-monitor.all@ietf.org" <draft-ietf-ippm-type-p-monitor.all@ietf.org>, General Area Review Team <gen-art@ietf.org>
Thread-Topic: Gen-ART Telechat review of draft-ietf-ippm-type-p-monitor-02.txt
Thread-Index: AdELrnQtmghXe3H1SiKtOvVwCaDy+w==
Date: Thu, 22 Oct 2015 04:03:07 +0000
Message-ID: <E87B771635882B4BA20096B589152EF63A9CF18F@eusaamb107.ericsson.se>
References: <E87B771635882B4BA20096B589152EF63A9CA979@eusaamb107.ericsson.se> <7347100B5761DC41A166AC17F22DF1122190D57D@eusaamb103.ericsson.se>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [147.117.188.11]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFrrPLMWRmVeSWpSXmKPExsUyuXRPoK7tZ/Uwg+4z3BbHj2ZZXH31mcWB yWPJkp9MAYxRXDYpqTmZZalF+nYJXBn9808xFxyXqfj6didbA+Mk8S5GTg4JAROJDy0tjBC2 mMSFe+vZuhi5OIQEjjJKvJtxjAXCWc4o8ePVSjaQKjagjg07PzOBJEQEDjBKHNy7HSwhLOAr 8ffROjBbRCBA4sP8xawQtp7EqRVTmUFsFgFVieYJl4FqODh4geon/rGEWDCRUWL1tU52kBpG oDO+n1rDBGIzC4hL3HoynwniPAGJJXvOM0PYohIvH/9jhbCVJD7+ns8OUa8jsWD3JzYIW1ti 2cLXYPW8AoISJ2c+YZnAKDILydhZSFpmIWmZhaRlASPLKkaO0uLUstx0I8NNjMBgPybB5riD ccEny0OMAhyMSjy8CbPUw4RYE8uKK3MPMUpwMCuJ8P6z1AgT4k1JrKxKLcqPLyrNSS0+xCjN waIkzjtvxv1QIYH0xJLU7NTUgtQimCwTB6dUA+PSe7PmM/JVRgu67Ju/za7cZ7pO5u91laeX JHiEld/uNuF/2v++RNJB5+hcq8h0maWbq9SfCFtfidZj+qCVLqZvo7HliFTl4tWnLjyXmyYj OPP/hUyzpRfOdul7dy4J1vVV/Ohjonr1dX7ktiOzWhlVFy/TeVMZZl41P0Esvv3s16NsBSUl vUosxRmJhlrMRcWJALXhdG1yAgAA
Archived-At: <http://mailarchive.ietf.org/arch/msg/gen-art/mXPWRx8KuIoYHck_0MqcasozKiE>
Subject: Re: [Gen-art] Gen-ART Telechat review of draft-ietf-ippm-type-p-monitor-02.txt
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.15
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: Thu, 22 Oct 2015 04:03:12 -0000

Hi Greg,
   Thanks for addressing my comments quickly. Your proposed changes sound 
good to me.

Cheers
Suresh

On 10/21/2015 04:38 PM, Gregory Mirsky wrote:
> Hi Suresh,
> thank you for the most careful review and very helpful comments. Please find my answers in-line and tagged by GIM>>.
>
> 	Regards,
> 		Greg
>
> -----Original Message-----
> From: Suresh Krishnan [mailto:suresh.krishnan@ericsson.com]
> Sent: Tuesday, October 20, 2015 8:13 PM
> To: draft-ietf-ippm-type-p-monitor.all@ietf.org; General Area Review Team
> Subject: Gen-ART Telechat review of draft-ietf-ippm-type-p-monitor-02.txt
>
> 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 wait for direction from your document shepherd or AD before posting a new version of the draft.
>
> Document: draft-ietf-ippm-type-p-monitor-02.txt
> Reviewer: Suresh Krishnan
> Review Date: 2015/10/20
> IESG Telechat date: 2015/10/22
>
> Summary: The draft is almost ready for publication as a Proposed Standard but I do have some comments that you may wish to address.
>
> Minor
> =====
>
> * MBZ is not expanded. I understand this should expand to "Must Be Zero" and it MUST be set to zero by senders and MUST be ignored by receivers. It makes sense to add this to the terminology section or before first use.
> GIM>> MBZ is used only in figures that reflect updates to formats defined in RFC 5357. None of fields defined in RFC 5357 referenced in this proposal and their identification in the figures is the same as in RFC 5357. I think it may be confusing to those familiar with RFC 5357 to see "Must Be Zero" instead of MBZ in figures.
>
> * Please cite as references RFC2474 for the DSCP field and RFC3168 for ECN.
> GIM>> Thank you, will add references and send diff for review.
>
> * Section 2.2.1:
>
> "the first six bits of the Differentiated Service field"
>
> Not sure if this "first" qualification is required as RFC2474 defines the DSCP field to be *exactly* 6 bits long. I have a similar issue with the word "following" in the definition of the ECN field as they are two separate fields.
> GIM>> Al suggested re-wording that, in my view, makes it clear and unambiguous:
>     o  the six (least-significant) bits of the Differentiated Service
>        field MUST be copied from received Session-Sender test packet into
>        Sender DSCP (S-DSCP) field;
>
> * Section 2.2.2: Figure 4 seems to be incomplete and it has no mention of either DSCP or ECN. Is this correct? Probably would also explain where the 28 byte padding requirement comes from.
> GIM>> Figure 4 reflects impact of DSCP and ECM Monitoring on test packet transmitted by a Session-Sender that supports RFC 6038. RFC 6038 states that in order to support Symmetrical Size and/or Reflects Octets modes Session-Sender must append at least 27 octet-long Packet Padding. Because the DSCP and ECM Monitoring extension requires Session-Reflector to copy additional octet, the minimal size of Packet Padding to support RFC 6038 must be 28 octets.
>
> Editorial
> =========
>
> * Section 2.2.1
>
> s/MUST extracts/MUST extract/
> GIM>> Done.
>
> Thanks
> Suresh
>
>