Re: [Gen-art] Gen-ART Telechat review of draft-ietf-tsvwg-sctp-chunk-flags-01.txt

Suresh Krishnan <suresh.krishnan@ericsson.com> Wed, 06 October 2010 19:56 UTC

Return-Path: <suresh.krishnan@ericsson.com>
X-Original-To: gen-art@core3.amsl.com
Delivered-To: gen-art@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 7D4463A71AC for <gen-art@core3.amsl.com>; Wed, 6 Oct 2010 12:56:01 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.515
X-Spam-Level:
X-Spam-Status: No, score=-102.515 tagged_above=-999 required=5 tests=[AWL=0.084, BAYES_00=-2.599, USER_IN_WHITELIST=-100]
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 fZqdhxYK-yiO for <gen-art@core3.amsl.com>; Wed, 6 Oct 2010 12:56:00 -0700 (PDT)
Received: from imr4.ericy.com (imr4.ericy.com [198.24.6.8]) by core3.amsl.com (Postfix) with ESMTP id EF0553A6FE1 for <gen-art@ietf.org>; Wed, 6 Oct 2010 12:55:59 -0700 (PDT)
Received: from eusaamw0711.eamcs.ericsson.se ([147.117.20.178]) by imr4.ericy.com (8.14.3/8.14.3/Debian-9.1ubuntu1) with ESMTP id o96KCP8g001648; Wed, 6 Oct 2010 15:12:27 -0500
Received: from [142.133.10.113] (147.117.20.213) by eusaamw0711.eamcs.ericsson.se (147.117.20.179) with Microsoft SMTP Server id 8.2.234.1; Wed, 6 Oct 2010 15:56:55 -0400
Message-ID: <4CACD432.5060808@ericsson.com>
Date: Wed, 06 Oct 2010 15:55:30 -0400
From: Suresh Krishnan <suresh.krishnan@ericsson.com>
User-Agent: Thunderbird 2.0.0.24 (X11/20100411)
MIME-Version: 1.0
To: Michael Tuexen <tuexen@fh-muenster.de>
References: <4CAC093E.3060805@ericsson.com> <274054BC-13AE-4A31-A623-37114B5ED002@fh-muenster.de>
In-Reply-To: <274054BC-13AE-4A31-A623-37114B5ED002@fh-muenster.de>
Content-Type: text/plain; charset="us-ascii"; format="flowed"
Content-Transfer-Encoding: 7bit
Cc: General Area Review Team <gen-art@ietf.org>, "draft-ietf-tsvwg-sctp-chunk-flags.all@tools.ietf.org" <draft-ietf-tsvwg-sctp-chunk-flags.all@tools.ietf.org>
Subject: Re: [Gen-art] Gen-ART Telechat review of draft-ietf-tsvwg-sctp-chunk-flags-01.txt
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/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, 06 Oct 2010 19:56:01 -0000

Hi Michael,
   Thanks for the quick response. Please see comments inline.

On 10-10-06 01:50 PM, Michael Tuexen wrote:
> Hi Suresh,
> 
> thank you very much for the review. See my comments in-line.
> 
> Best regards
> Michael
> 
> On Oct 6, 2010, at 7:29 AM, Suresh Krishnan wrote:
> 
>> I have been selected as the General Area Review Team (Gen-ART)
>> reviewer for this draft (for background on Gen-ART, please see
>> http://www.alvestrand.no/ietf/gen/art/gen-art-FAQ.html).
>>
>> Please wait for direction from your document shepherd
>> or AD before posting a new version of the draft.
>>
>> Document: draft-ietf-tsvwg-sctp-chunk-flags-01.txt
>> Reviewer: Suresh Krishnan
>> Review Date: 2010/10/05
>> IESG Telechat date: 2010/10/07
>>
>> Summary: This draft is almost ready for publication as a Proposed Standard but it has two minor issues.
>>
>> Minor
>> =====
>>
>> * Introduction
>>
>> The justification for the draft is a bit vague. It states the following as a justification
>>
>> "Without publication
>>   of this document, the only way to have done this would have been to
>>   obsolete [RFC4960], which is not appropriate."
>>
>> I am not sure how the authors reached such a conclusion. The protocol extension documents could just update RFC4960 in order to achieve the same result.Can you clarify or delete?
> We can remove it... no problem.
> The reason for the document is the following:
> SCTP is an extensible protocol. Without that ID, some specification which define an
> extension need actually to update the base specification. With the ID, the extension
> can just define the extension without touching the base specification. So if someone
> wants to implement the base specification, it can no so with completely ignoring the
> extension.

It is the word "obsolete" that threw me off in the sentence. It has a 
very specific meaning in RFCs. I see that you meant they need to 
"Update" RFC4960 each  time. Please go ahead remove the text if you are 
fine with doing so.


> 
> Does this make sense to you?
>> * Section 3.1
>>
>> Even though this is under the IANA considerations section, the instructions seem to be aimed not at IANA but at protocol extension designers. Where does the required documentation need to be? In the relevant draft or in an IANA registry. The only IANA instruction I can see is the sentence beginning with "IANA creates for each new chunk type a registration table for the chunk flags for this type."
> Section 3.1 is the updated section 14.1 of RFC 4960. This is explained in Section 3:
> 
>    Section 3.1 describes the updated procedure for chunk type
>    registration and replaces Section 14.1 of [RFC4960].  Section 3.2
>    adds a new procedure for chunk flag registration to the updated
>    section 14.1 of [RFC4960].
> 
>    IANA is requested to create an SCTP Chunk Flag registry.  The initial
>    contents of the registry should be assigned using the values
>    specified in Section 3.3.
> 
> So what we expect from IANA when this ID is approved is specified in section 3.3.

Makes sense. As I said, I had no issues with Section 3.2 and 3.3. They 
work great. Did you see the mail from Russ about proposed text for 
Section 3? That would resolve my concern and would make it clearer for 
IANA what they are expected to do.

Thanks
Suresh