Reviewer: Gorry Fairhurst Review result: Ready with Nits This document has been reviewed as part of the transport area review team's ongoing effort to review key IETF documents. These comments were written primarily for the transport area directors, but are copied to the document's authors and WG to allow them to address any issues raised and also to the IETF discussion list for information. When done at the time of IETF Last Call, the authors should consider this review as part of the last-call comments they receive. Please always CC tsv-art@xxxxxxxx if you reply to or forward this review. This document specifies procedure updates for broadcast, unknown unicast, and multicast (BUM) traffic in Ethernet VPNs (EVPN), including selective multicast, and provider tunnel segmentation. There is a normative reference to draft-ietf-bess-evpn-igmp-mld-proxy, which normatively refers to this spec. This draft focusses on topics beneath the transport layer. It does not appear to introduce specific transport protocol concerns. 1. A common concern for transports using tunnels is the topic of fragmentation and packet size discovery. This is not mentioned, and it could be useful to point to the relevant section of a spec (I note that path "segmentation" in this draft relates to something different, in RFC 7524). 2. A general comment is that the draft section 1 states "It is expected that audience is familiar with EVPN and MVPN concepts and terminologies", I suggest it would none-the-less be very helpful to: * Include appropriate RFC references to where these terms are defined (.e.g. RFC7432?); * Check all the abbreviations and either define each in section 1 or simply expand on first use in this document; 3. Section 8. Describes a temporary IANA assignment, which I presume publication of this draft confirms? I expect an IANA note to this effect would help the IANA Team. -- last-call mailing list last-call@xxxxxxxx https://www.ietf.org/mailman/listinfo/last-call
