Hi Sheng, Thank you for the review.
Please see inline. Cheers, Med > -----Message d'origine----- > De : Sheng Jiang [mailto:[email protected]] > Envoyé : mardi 10 janvier 2017 04:55 > À : [email protected] > Cc : [email protected]; [email protected]; draft-ietf-softwire-multicast- > [email protected] > Objet : Review of draft-ietf-softwire-multicast-prefix-option-11 > > Reviewer: Sheng Jiang > Review result: Has Nits > > Summary: This draft is almost ready for publication as a standard > track RFC. > > Major issues: > > Minor issues: > > “the specification of a DHCPv6 option that could be used to discover > unicast PREFIX64s in environments where multicast is not enabled. > Such side effect conflicts with the recommendation documented in > Section 6 of [RFC7051].” > > It is unclear how the Section 6 of RFC7051 relevant with the content > above. It would be necessary to quote particular content of RFC7051 > and give necessary analysis. > [Med] What about: OLD: Note that it was tempting to define three distinct DHCPv6 options, but that approach was not adopted because it has a side effect: the specification of a DHCPv6 option that could be used to discover unicast PREFIX64s in environments where multicast is not enabled. Such side effect conflicts with the recommendation documented in Section 6 of [RFC7051]. NEW: Note that it was tempting to define three distinct DHCPv6 options, but that approach was not adopted because it has a side effect: the specification of a DHCPv6 option that could be used to discover unicast PREFIX64s in environments where multicast is not enabled. Such side effect conflicts with the recommendation documented in Section 6 of [RFC7051]. As a reminder, that recommendation is to ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ to support the Well-Known DNS Name heuristic discovery-based method ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ for unicast-only environments. ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Better? > Nits: > > “the Pv4 multicast address is inserted in the last 32 bits of the > IPv4-embedded IPv6 > multicast address.” > > Pv4//IPv4 [Med] Fixed. _______________________________________________ Softwires mailing list [email protected] https://www.ietf.org/mailman/listinfo/softwires
