[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.
------------------------------------------------------------------