[trill] WG LC for draft-ietf-trill-ia-appsubtlv-03 -
gayle noble <windy_1@skyhighway.com> Mon, 01 June 2015 05:52 UTC
Return-Path: <windy_1@skyhighway.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 BD41E1A8894 for <trill@ietfa.amsl.com>; Sun, 31 May 2015 22:52:02 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.909
X-Spam-Level:
X-Spam-Status: No, score=-1.909 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, T_RP_MATCHES_RCVD=-0.01] 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 3rgEmINRjVXI for <trill@ietfa.amsl.com>; Sun, 31 May 2015 22:52:00 -0700 (PDT)
Received: from skyhighway.com (skyhighway.com [63.249.82.6]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 1DC0D1A8889 for <trill@ietf.org>; Sun, 31 May 2015 22:52:00 -0700 (PDT)
Received: from Firefly.skyhighway.com (dsl-63-249-88-160.static.cruzio.com [63.249.88.160]) by skyhighway.com with ESMTP id t515pwHu089632 for <trill@ietf.org>; Sun, 31 May 2015 22:51:58 -0700 (PDT)
Message-Id: <201506010551.t515pwHu089632@skyhighway.com>
X-Mailer: QUALCOMM Windows Eudora Version 7.1.0.9
Date: Sun, 31 May 2015 22:51:50 -0700
To: trill@ietf.org
From: gayle noble <windy_1@skyhighway.com>
In-Reply-To: <02d501d09a0a$371e7dc0$a55b7940$@ndzh.com>
References: <02d501d09a0a$371e7dc0$a55b7940$@ndzh.com>
Mime-Version: 1.0
Content-Type: multipart/mixed; boundary="=====================_1471306175==_"
Archived-At: <http://mailarchive.ietf.org/arch/msg/trill/_6AaJKW81__YA0YmDVtq8sCfmIo>
Subject: [trill] WG LC for draft-ietf-trill-ia-appsubtlv-03 -
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: <http://www.ietf.org/mail-archive/web/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: Mon, 01 Jun 2015 05:52:02 -0000
TRILL: Interface Addresses APPsub-TLV <draft-eastlake-trill-ia-appsubtlv-03.txt> errors: 1. Page 1 abstract and page 3 Introduction Forth sentence ['use" should be "used"] (as written) Such information could be use in some cases to synthesize responses to or by-pass the need for the Address Resolution Protocol (ARP), the IPv6 Neighbor Discovery (ND) protocol, or the flooding of unknown MAC addresses. (should be) Such information could be used in some cases to synthesize responses to or by-pass the need for the Address Resolution Protocol (ARP), the IPv6 Neighbor Discovery (ND) protocol, or the flooding of unknown MAC addresses. ----------------------------------------------------------------------------- 2. page 6 description of L bit ["Both D and L can one in the same" does not make sense. Was this supposed to mean "Both D and L can be set to one in the same IA APPsub-TLV. ??] (as written) L: Local flag: If L is one, the APPsub-TLV contains information learned ocally by observing ingressed frames. (Both D and L can one in the same IA APPsub-TLV.) (probably should be) L: Local flag: If L is one, the APPsub-TLV contains information learned locally by observing ingressed frames. (Both D and L can be one in the same IA APPsub-TLV.) (would read better as) L: Local flag: If L is one, the APPsub-TLV contains information learned locally by observing ingressed frames. (Both D and L can be set to one in the same IA APPsub-TLV.) ----------------------------------------------------------------------------- 3. page 6 description of N bit ["one in a ESADI LSP" should be "one in an ESADI LSP" and "contens" should be "contents" ] (as written) N: Notify flag: When a TRILL switch receives a new IA APPsub-TLV (one in a 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 contens of the APPsub-TLV for IP address to MAC address mappings. (probably should be) 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 IP address to MAC address mappings. ----------------------------------------------------------------------- 4. page 7 in Template: the fifth sentence (the last one) ["filed" should be "field"] (as written) If K is 255, the length of the Template filed is three bytes and the values of the second and third byte, considered as an unsigned integer in network byte order, are reserved to correspond to future specified ordered sets of AFNs. (should be) If K is 255, the length of the Template field is three bytes and the values of the second and third byte, considered as an unsigned integer in network byte order, are reserved to correspond to future specified ordered sets of AFNs. ----------------------------------------------------------------------------- 5. page 12 section 3.4 Topology sub-sub-TLV first sentence [The last word "give" doesn't make any sense. Should this be "given"?] (as written) The presence of this sub-sub-TLV indicates that the interfaces given in the IA APPsub-TLV are reachable in the topology give. (should this be?) The presence of this sub-sub-TLV indicates that the interfaces given in the IA APPsub-TLV are reachable in the topology given. -------------------------------------------------------- 6. page 15 section 5.1 Additional AFN Number Allocation ["alocated" should be spelt "allocated"] (as written) IANA has alocated AFN numbers as follows: (should be) IANA has allocated AFN numbers as follows: ----------------------------------------------------------------- 7. page 19 paragraph at the bottom first sentence [The word "give" doesn't make any sense. Should this be "given"?] (as written) The Fixed Address sub-sub-TLV causes the IPv6/64 value give to be treated as if it occurred as a 4th entry inside each of the three Address Sets. (should this be?) The Fixed Address sub-sub-TLV causes the IPv6/64 value given to be treated as if it occurred as a 4th entry inside each of the three Address Sets. ============================ suggestions:: 1. page 6 It was not clear to me if 0 is the highest confidence or 254 is the highest confidence. Perhaps, it could explicitly say that confidence is described in [RFC6325]. Or maybe add something saying that larger values of confidence indicate higher confidence. ------------------------------------------------------------------
- [trill] WG LC for draft-ietf-trill-ia-appsubtlv-0… Susan Hares
- Re: [trill] WG LC for draft-ietf-trill-ia-appsubt… Linda Dunbar
- Re: [trill] WG LC for draft-ietf-trill-ia-appsubt… Mingui Zhang
- [trill] WG LC for draft-ietf-trill-ia-appsubtlv-0… gayle noble
- Re: [trill] WG LC for draft-ietf-trill-ia-appsubt… Donald Eastlake
- Re: [trill] WG LC for draft-ietf-trill-ia-appsubt… Donald Eastlake
- Re: [trill] WG LC for draft-ietf-trill-ia-appsubt… Liyizhou