Re: [Gen-art] Russ's Discs on draft-ietf-pim-port [Was: Gen-ART Telechat review of draft-ietf-pim-port-09.txt]

Suresh Krishnan <suresh.krishnan@ericsson.com> Mon, 28 November 2011 18:19 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 B68C821F899F for <gen-art@ietfa.amsl.com>; Mon, 28 Nov 2011 10:19:07 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -104.41
X-Spam-Level:
X-Spam-Status: No, score=-104.41 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, DATE_IN_FUTURE_12_24=2.189, RCVD_IN_DNSWL_MED=-4, 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 4bXdCapXtPQo for <gen-art@ietfa.amsl.com>; Mon, 28 Nov 2011 10:19:07 -0800 (PST)
Received: from imr4.ericy.com (imr4.ericy.com [198.24.6.9]) by ietfa.amsl.com (Postfix) with ESMTP id 2F83A21F88B7 for <gen-art@ietf.org>; Mon, 28 Nov 2011 10:19:07 -0800 (PST)
Received: from eusaamw0712.eamcs.ericsson.se ([147.117.20.181]) by imr4.ericy.com (8.14.3/8.14.3/Debian-9.1ubuntu1) with ESMTP id pASIIxGG028315; Mon, 28 Nov 2011 12:19:01 -0600
Received: from [142.133.10.111] (147.117.20.214) by smtps-am.internal.ericsson.com (147.117.20.181) with Microsoft SMTP Server (TLS) id 8.3.137.0; Mon, 28 Nov 2011 13:18:58 -0500
Message-ID: <4ED521FE.5080401@ericsson.com>
Date: Tue, 29 Nov 2011 13:18:38 -0500
From: Suresh Krishnan <suresh.krishnan@ericsson.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:7.0.1) Gecko/20110929 Thunderbird/7.0.1
MIME-Version: 1.0
To: Stig Venaas <svenaas@cisco.com>
References: <00d401ccabbe$216a6ae0$643f40a0$@olddog.co.uk> <4ED1860F.3080501@ericsson.com> <4ED3CBF1.9050303@cisco.com>
In-Reply-To: <4ED3CBF1.9050303@cisco.com>
X-Enigmail-Version: 1.3.3
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: 7bit
Cc: "adrian@olddog.co.uk" <adrian@olddog.co.uk>, 'General Area Review Team' <gen-art@ietf.org>, "draft-ietf-pim-port.all@tools.ietf.org" <draft-ietf-pim-port.all@tools.ietf.org>, 'Stig Venaas' <stig@venaas.com>
Subject: Re: [Gen-art] Russ's Discs on draft-ietf-pim-port [Was: Gen-ART Telechat review of draft-ietf-pim-port-09.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: Mon, 28 Nov 2011 18:19:07 -0000

Hi Stig,


On 11/28/2011 12:59 PM, Stig Venaas wrote:
>> Please note that the Connection ID AFI in the PORT Hello Option does not
>> need to match the address family of PIM Hello message that carries it.
>> e.g. an IPv6 PIM Hello message could contain a PORT Hello Option with a
>> TCP Connection ID AFI of 1 (IPv4).
> 
> While I am OK with this text, wouldn't it be better to add this to 3.1
> and 3.2 where we formally define the hello options?
> 
> My proposal would be:
> 
> End of 3.1:
> 
> OLD:
>     TCP Connection ID AFI:   The AFI value to describe the address-family
>        of the address of the TCP Connection ID field.  When this field is
>        0, a mechanism outside the scope of this document is used to
>        obtain the addresses used to establish the TCP connection.
> 
> NEW:
>     TCP Connection ID AFI:   The AFI value to describe the address-family
>        of the address of the TCP Connection ID field. Note that this
>        value does not need to match the address-family of the PIM Hello
>        message that carries it. When this field is 0, a mechanism
>        outside the scope of this document is used to obtain the
>        addresses used to establish the TCP connection.
> 
> End of 3.2:
> 
> OLD:
>     SCTP Connection ID AFI:   The AFI value to describe the address-
>        family of the address of the SCTP Connection ID field.  When this
>        field is 0, a mechanism outside the scope of this document is used
>        to obtain the addresses used to establish the SCTP connection.
> 
> NEW:
>     SCTP Connection ID AFI:   The AFI value to describe the address-
>        family of the address of the SCTP Connection ID field. Note that
>        this value does not need to match the address-family of the PIM
>        Hello message that carries it. When this field is 0, a mechanism
>        outside the scope of this document is used to obtain the
>        addresses used to establish the SCTP connection.
> 
> What do you think?

This works great.

Thanks
Suresh