[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