Hi Rob, Thanks for your review. Please see below. On 7/7/20, 3:41 AM, "Robert Wilton via Datatracker" <nore...@ietf.org> wrote:
Robert Wilton has entered the following ballot position for draft-ietf-lisp-gpe-16: No Objection When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html for more information about IESG DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-lisp-gpe/ ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- Hi, I found this document easy to read and understand. A couple of minor comments: 4.3. UDP Checksum By default, UDP checksum MUST be used when LISP-GPE is transported over IPv6. A tunnel endpoint MAY be configured for use with zero UDP checksum if additional requirements in Section 4.3.1 are met. This paragraph could probably be moved/merged into section 4.3.1? [FM] good catch. We have moved it there in rev-17 4.4. DSCP, ECN and TTL When a LISP-GPE router performs Ethernet encapsulation, the inner header 802.1Q [IEEE.802.1Q_2014] VLAN Identifier (VID) MAY be mapped to, or used to determine the LISP Instance IDentifier (IID) field. This text doesn't appear to be related to DSCP, ECN or TTL. Perhaps tweak the title of this section to cover this text? [FM] Good point. Changed to " DSCP, ECN, TTL, and 802.1Q". Thanks, Fabio Regards, Rob _______________________________________________ lisp mailing list lisp@ietf.org https://www.ietf.org/mailman/listinfo/lisp