From: Suresh Krishnan <suresh.krishnan@ericsson.com>
To: "draft-ietf-l2vpn-ipls.all@tools.ietf.org"
 <draft-ietf-l2vpn-ipls.all@tools.ietf.org>, General Area Review Team
 <gen-art@ietf.org>
Thread-Topic: Gen-ART Telechat review of draft-ietf-l2vpn-ipls-15.txt
Thread-Index: Ac/zLmQdeCKg/APAQESZx7Qbqc92fw==
Date: Wed, 29 Oct 2014 04:11:21 +0000
Message-ID: <E87B771635882B4BA20096B589152EF6289044A9@eusaamb107.ericsson.se>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Archived-At: http://mailarchive.ietf.org/arch/msg/gen-art/KMLd6EDcOAZ-zEqGvc2w9LCuBog
Subject: [Gen-art] Gen-ART Telechat review of draft-ietf-l2vpn-ipls-15.txt
Precedence: list

I have been selected as the General Area Review Team (Gen-ART)=0A=
reviewer for this draft (for background on Gen-ART, please see=0A=
http://www.alvestrand.no/ietf/gen/art/gen-art-FAQ.html).=0A=
=0A=
Please wait for direction from your document shepherd=0A=
or AD before posting a new version of the draft.=0A=
=0A=
Document: draft-ietf-l2vpn-ipls-15.txt=0A=
Reviewer: Suresh Krishnan=0A=
Review Date: 2013/10/28=0A=
IESG Telechat date: 2013/10/30=0A=
=0A=
=0A=
Summary: This document is ready for publication as a Historic RFC, but I =
=0A=
have some comments you may wish to address.=0A=
=0A=
* Section 7.1=0A=
=0A=
The Length field of the IP address TLV in case the IP address is IPv6 is =
=0A=
not defined. It should be defined to be 18 (2 bytes for address family =0A=
and 16 bytes for the IPv6 address)=0A=
=0A=
=0A=
* Address family reference=0A=
=0A=
The IANA registry at =0A=
http://www.iana.org/assignments/address-family-numbers/address-family-numbe=
rs.xhtml =0A=
is a much better reference for address families than the currently =0A=
referred RFC3232 that does not contain any of the relevant information.=0A=
=0A=
Thanks=0A=
Suresh=0A=
=0A=
=0A=
=0A=