Re: [trill] draft-ietf-trill-ia-appsubtlv-05.txt - Shepherd's review

"Susan Hares" <shares@ndzh.com> Wed, 23 December 2015 16:29 UTC

Return-Path: <shares@ndzh.com>
X-Original-To: trill@ietfa.amsl.com
Delivered-To: trill@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 632AE1A1B64 for <trill@ietfa.amsl.com>; Wed, 23 Dec 2015 08:29:21 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -97.156
X-Spam-Level:
X-Spam-Status: No, score=-97.156 tagged_above=-999 required=5 tests=[BAYES_20=-0.001, DOS_OUTLOOK_TO_MX=2.845, USER_IN_WHITELIST=-100] autolearn=no
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 hx9HJ0BhV3Sz for <trill@ietfa.amsl.com>; Wed, 23 Dec 2015 08:29:20 -0800 (PST)
Received: from hickoryhill-consulting.com (hhc-web3.hickoryhill-consulting.com [64.9.205.143]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 77F081A1BB1 for <trill@ietf.org>; Wed, 23 Dec 2015 08:29:18 -0800 (PST)
X-Default-Received-SPF: pass (skip=loggedin (res=PASS)) x-ip-name=74.43.47.177;
From: "Susan Hares" <shares@ndzh.com>
To: "'Donald Eastlake'" <d3e3e3@gmail.com>
References: <00a801d0fd57$0c2c5b40$248511c0$@ndzh.com> <CAF4+nEGMdnjqgKyBgKksGfXBarEQ+RdzzGk++7iJtaO_LZu6Ug@mail.gmail.com>
In-Reply-To: <CAF4+nEGMdnjqgKyBgKksGfXBarEQ+RdzzGk++7iJtaO_LZu6Ug@mail.gmail.com>
Date: Wed, 23 Dec 2015 11:29:00 -0500
Message-ID: <005e01d13d9f$07b3e6e0$171bb4a0$@ndzh.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
X-Mailer: Microsoft Outlook 14.0
Thread-Index: AQK30vGeCoauv1aduj5PANj/8+s18wF8BzNYnP+3u9A=
Content-Language: en-us
X-Authenticated-User: skh@ndzh.com
Archived-At: <http://mailarchive.ietf.org/arch/msg/trill/-_1uigPg0-yZ7wXEjFfZpI45dQU>
Cc: 'Jon Hudson' <jon.hudson@gmail.com>, trill-chairs@tools.ietf.org, trill@ietf.org
Subject: Re: [trill] draft-ietf-trill-ia-appsubtlv-05.txt - Shepherd's review
X-BeenThere: trill@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "Developing a hybrid router/bridge." <trill.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/trill>, <mailto:trill-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/trill/>
List-Post: <mailto:trill@ietf.org>
List-Help: <mailto:trill-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/trill>, <mailto:trill-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 23 Dec 2015 16:29:21 -0000

Donald: 

Thank you for the changes.  These changes address all the comments I made in the shepherd review. 

Sue Hares 

-----Original Message-----
From: Donald Eastlake [mailto:d3e3e3@gmail.com] 
Sent: Sunday, December 20, 2015 1:06 PM
To: Susan Hares
Cc: trill@ietf.org; trill-chairs@tools.ietf.org; Jon Hudson
Subject: Re: draft-ietf-trill-ia-appsubtlv-05.txt - Shepherd's review

Hi Sue,

Thanks for the comments.

On Fri, Oct 2, 2015 at 5:12 PM, Susan Hares <shares@ndzh.com>; wrote:
> Shepherd’s review:  draft-ietf-trill-ia-appsubtlv-05
>
> Authors: Donald Eastlake and Yizhou Li
>
> Status: Major concern on complexity of AFN template concepts. This 
> concern has been expressed to authors who will revise the draft. The 
> text for the AFN Template is covered in Section 2 and section 3.

The draft has been simplified.

> Note: IANA previewed section 5, and agreed to it.
>
> Minor Technical issues: none
>
> Editorial issues
>
> #1 Abstract:
>
> Current
>    This document specifies a TRILL (Transparent Interconnection of
>    Lots of Links) IS-IS application sub-TLV that enables the
>    reporting by a TRILL switch of sets of addresses such that all of
>    the addresses in each set designate the same interface (port) and
>    the reporting for such a set of the TRILL switch by which it is
>    reachable.
>
> New suggested:
>    This document specifies a TRILL (Transparent Interconnection of
>    Lots of Links) IS-IS application sub-TLV that enables the
>    reporting by a TRILL switch of sets of addresses. Each set of
>    addresses reports addresses the same interface (port) and the
>    TRILL switch by which it reachable.

A change along those lines has been made.

> #2 – Section 2 – paragraph 1 p. 5
>
> Please enumerate or point to where the enumeration of the types of 
> addresses.

The current text lists IPv4, IPv6, and 48-bit MAC addresses. The following sentence has been added: "While those three types of address are likely to be the only types of interest, any address type for which an AFN (Address Family Number) has been assigned by IANA can be represented."

> #3 Section 2 – page 6
>
> Current:
>
>  Addr Sets End: The unsigned integer offset of the byte, within the  
> IA APPsub-TLV value part, of the last byte of the last Address  Set. 
> This will be the byte just before the first sub-sub-TLV if  any 
> sub-sub-TLVs are present (see Section 3). If this is equal to  Length, 
> there are no sub-sub-TLVs. If this is greater than Length or points to 
> before the end of the Template, the IA APPsub-TLV is corrupt and MUST 
> be discarded. This field is always two bytes in size.
>
> New
>
> Addr Sets End: The unsigned integer offset of the byte, within the IA 
> APPsub-TLV value part, of the last byte of the last Address  Set. This 
> byte will be the byte just before the first sub-sub-TLV if  any 
> sub-sub-TLVs are present (see Section 3). The processing is as follows:
>
> ·         If this is equal to  Length, there are no sub-sub-TLVs.
>
> ·         If this is greater than Length or points to before the end of the
> Template, the IA APPsub-TLV is corrupt and MUST be discarded.
>
> ·         Note: This field is always two bytes in size.

OK.

> #3 – page 7 paragraph 3
>
>         N: Notify flag: When a TRILL switch receives a new IA APPsub-
>             TLV (one in an ESADI-LSP fragment with a higher sequence
>             number or a new message of some other type) and the N bit is
>             one, the TRILL switch then checks the contents of the
>             APPsub-TLV for address sets including both an IP address and
>             a MAC address.  For each such address set it finds, a
>             gratuitous ARP [RFC826] or spontaneous Neighbor
>             Advertisement [RFC4861], depending on whether the IP address
>             is IPv4 or IPv6 respectively, may be sent. In both cases,
>             these are sent out all the ports of the TRILL switch
>             offering end station service and are in the VLAN or FGL of
>             the address set information, that is, are Appointed
>             Forwarder for the VLAN or for the VLAN to which the FGL
>             maps.
>
> What needs changing:
>
> Old: /and are in the VLAN or FGL of/
> New: /which are in the VLAN or FGL of/  - I believe the dependent “which”
> clause is more appropriate here.

OK.

A new version -06 has been posted.

Thanks,
Donald
=============================
 Donald E. Eastlake 3rd   +1-508-333-2270 (cell)
 155 Beaver Street, Milford, MA 01757 USA  d3e3e3@gmail.com