Hi Thomas, I have incorporated your comments into rev11 of the document.
Cheers, Ali On 12/8/17, 1:45 AM, "Thomas Morin" <[email protected]> wrote: Ali, A few nits, sorry for not answering earlier: > > I added the following paragraph to section 6 for clarification: > “When a PE advertises multiple supported encapsulations, it MUST > advertise encapsulations that use the same EVPN procedures including > procedures associated with split-horizon filtering described in > section 8.3.1. For example, VxLAN and NvGRE (or MPLS and MPLS over > GRE) encapsulations use the same EVPN procedures and thus a PE can > advertise both of them and can support either of them or both of them > simultaneously. However, a PE MUST NOT advertise VxLAN and MPLS You should say "VxLAN/NVGRE" here to be consistent with the rest of the document that treats VXLAN and NVGRE equally. > encapsulations together because a) MPLS field of EVPN routes is set - s/MPLS field/the MPLS field/ - "(a)" instead of "a)" would be a bit more readable > to either a MPLS label for a VNI but not both and b) some of EVPN - s/a MPLS label/an MPLS label/ - s/for a VNI/or a VNI/ - s/some of EVPN/some EVPN/ (?) > procedures (such as split-horizon filtering) are different for > VxLAN/NvGRE and MPLS encapsulations.” Thanks, -Thomas _______________________________________________ BESS mailing list [email protected] https://www.ietf.org/mailman/listinfo/bess
