Document: draft-ietf-pim-p2mp-policy-ping Title: P2MP Policy Ping Reviewer: Linda Dunbar Review result: Has Nits Summary: This document is technically solid and aligns well with existing MPLS and SR OAM mechanisms. Some Nits though: Abstract: “via a controller-based mechanisms” → remove plural: “via a controller-based mechanism” Section 3.1: “The appropriate respond is sent back…” → should be: “The appropriate response is sent back…” Section 6 (Security): “Overall, the security needs for P2MP policy ping is same as [RFC8029].” — grammatical fix: “...are the same as...” Section 5 (IANA): “This sub-type value is assigned from the standards Action of range...” — is it better with : “...from the Standards Action range of ...” The following sentence is difficult to follow (Section 3.1.3): "For example, when a P2MP Policy Ping or Traceroute packet enters an Unicast SR domain, it MUST be processed on the two interconnecting Replication Segments, based on the Replication SID and its TTL value.". Best Regards, Linda Dunbar -- last-call mailing list -- last-call@xxxxxxxx To unsubscribe send an email to last-call-leave@xxxxxxxx
