Re: Gen-ART review of draft-ietf-softwire-dslite-deployment-06

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thanks David! I will apply your changes to the next version.

On 10/17/12 9:11 PM, "Black, David" <david.black@xxxxxxx> wrote:

>Yiu,
>
>Thank you for your responses - most of them look fine, and in
>particular anything that I don't comment on here is acceptable to me.
>
>> [YL] In 2.2, we will add:
>> 
>> "Note that reassembly at Tunnel Exit-Point is resource
>> 	intensive. A large number of B4 may terminate on the same AFTR. If
>>many B4
>> 	fragment the packets, it would increase sufficient load to the AFTR for
>> 	reassembly. We recommend the operator should increase the MTU size of
>>the IPv6
>> 	network between B4 and AFTR to avoid fragmentation."
>
>I would change "is" to "may be" in the first line.
>
>> >[5] Section 2.8 discusses IPv4 accounting at the AFTR, but notes that
>> >"the AFTR does not have detailed customer identity information."  Does
>> >that create a theft of service possibility?  If so, that possibility
>> >should be mentioned as a security consideration.  Also, Section 2.8
>> >ought to be expanded with a sentence or two explaining why the AFTR
>> >does not have that identity information, and in particular to explain
>> >whether and why it is unreasonable in some or all cases to expect
>> >that customer identity information be provided to the AFTR as part
>> >of provisioning each customer's softwire.
>> 
>> [YL] Good catch. Actually I believe AFTR "does" have both IPv4 and IPv6
>>to
>> identify the customer. I suggest we remove
>> 
>> "but it will potentially introduce some additional complexity because
>>    the AFTR does not have detailed customer identity information."
>
>That's definitely an easy way to address that issue, and it's fine with
>me.
>
>> >Section 2.3 on MTU Considerations could usefully mention MTU discovery
>> >techniques, as possibilities for customer IPv4 traffic to adapt to a
>> >smaller IPv4 MTU.  If this is done, it would be useful to mention
>> >RFC 4821 in addition to the mention of RFC 1191 in RFC 6333.
>> 
>> [YL] We would consider RFC 4821. However, the challenge is I don't have
>> information how many current OS support RFC 4821. Since DS-lite is a
>> transition technology, it would be unreasonable to ask the OS company to
>> implement RFC 4821 for DS-lite.
>
>That's ok - this was a nit.
>
>> >- Are one or both types of logging recommended?
>> 
>> [YL] We tempted to recommend source-specific log. However, some
>>regulators
>> require to use both and the regulations vary country from country, this
>>is
>> why we left it open and let the operators to decide.
>
>Please add a version of your explanation to the draft.
>
>> >In Section 2.7, I'm having a hard time understanding which traffic is
>> >intended to be subject to the Outgoing Policies and the Incoming
>>Policies.
>> >Expanding each of those two bullets to explain what traffic is subject
>> >to each class of policies would help.
>> 
>> [YL] Does this help?
>> 
>> Outgoing Policies apply to packets originating from B4 to IPv4 network.
>> Incoming Policies apply to packets originating from IPv4 network to B4.
>
>Yes, that's clearer, although the B4 is not the network endpoint for any
>of that traffic.  I suggest:
>
>Outgoing Policies apply to packets originating from behind the B4 with
>a destination on the IPv4 network.
>
>Incoming Policies apply to packets originating from the IPv4 network to
>a destination behind the B4.
>
>Thanks,
>--David
>
>
>> -----Original Message-----
>> From: Lee, Yiu [mailto:Yiu_Lee@xxxxxxxxxxxxxxxxx]
>> Sent: Wednesday, October 17, 2012 8:46 PM
>> To: Black, David; roberta.maglione@xxxxxxxxxxxxxxxx;
>>carlw@xxxxxxxxxxxxx;
>> christian.jacquenet@xxxxxxxxxx; mohamed.boucadair@xxxxxxxxxx;
>>gen-art@xxxxxxxx
>> Cc: softwires@xxxxxxxx; ietf@xxxxxxxx; Ralph Droms
>> Subject: Re: Gen-ART review of draft-ietf-softwire-dslite-deployment-06
>> 
>> Hi David,
>> 
>> Thanks very much for review the draft. Comments inline.
>> 
>> Thanks,
>> Yiu
>> 
>> On 10/15/12 7:10 PM, "Black, David" <david.black@xxxxxxx> wrote:
>> 
>> >I am the assigned Gen-ART reviewer for this draft. For background on
>> >Gen-ART, please see the FAQ at
>> ><http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.
>> >
>> >Please resolve these comments along with any other Last Call comments
>> >you may receive.
>> >
>> >Document: draft-ietf-softwire-dslite-deployment-06
>> >Reviewer: David L. Black
>> >Review Date: October 15, 2012
>> >IETF LC End Date: October 16, 2012
>> >IESG Telechat Date: October 25, 2012
>> >
>> >Summary:
>> >
>> >This draft is on the right track but has open issues, described in the
>> >review.
>> >
>> >This is a generally well-written draft that expands considerably on
>>the brief
>> >deployment considerations for DS-Lite in Appendix A of RFC 6333.  The
>>draft
>> >is clear and reasonably well-written, and a significant improvement on
>>that
>> >RFC 6333 Appendix, although an understanding of RFC 6333 is necessary
>>to
>> >understand this draft, which seems completely reasonable.
>> >
>> >The B4 and AFTR acronyms are clever - kudos to whomever came up with
>> >those.
>> >
>> >I found five open issues, all of which are minor.
>> >
>> >Minor Issues:
>> >
>> >[1] Ironically, the first issue is that something should be said about
>> >the relationship of this draft to Appendix A of RFC 6333.  It probably
>> >suffices to say that this draft expands on the material in that
>>Appendix,
>> >as I see no need to specify that this draft updates RFC 6333 solely to
>> >change non-normative material in an Appendix.
>> 
>> [YL] In "Overview", we will add:
>> 
>> "This document expands Appendix A of RFC6333 and discusses various
>> DS-Lite deployment considerations for operators."
>> 
>> >
>> >[2] The MTU increase technique in Section 2.2 is a "may", which is
>> >consistent with Section 5.3 of RFC 6333.  OTOH, Section 2.2 of this
>> >draft should also discuss the AFTR resource exhaustion concern that
>> >described in Section 6.3 of RFC 6333, as that is a potential
>> >operational reason for an ISP to increase MTU size (e.g., if customer
>> >sourcing of large IPv4 packets is not sufficiently rare).
>> 
>> [YL] In 2.2, we will add:
>> 
>> "Note that reassembly at Tunnel Exit-Point is resource
>> 	intensive. A large number of B4 may terminate on the same AFTR. If
>>many B4
>> 	fragment the packets, it would increase sufficient load to the AFTR for
>> 	reassembly. We recommend the operator should increase the MTU size of
>>the IPv6
>> 	network between B4 and AFTR to avoid fragmentation."
>> 
>> >
>> >[3] Section 2.7 refers to "the AFTR's internal interface".  There may
>>be
>> >two internal interfaces, the physical IPv6 interface (outer header) and
>> >the tunnel's IPv4 interface (inner header).  The text should be
>>clarified
>> >to indicate which interface is involved - it appears that the AFTR's
>> >physical IPv6 interface is intended in this section.
>> 
>> [YL] We replace "internal" to "IPv6"
>> 
>> >
>> >[4] Section 2.7 cites RFC 6333 for use of DHCPv6 with DS-Lite.  RFC
>>6334
>> >should be cited - either in addition to or instead of RFC 6333.
>> 
>> [YL] Fixed
>> 
>> >
>> >[5] Section 2.8 discusses IPv4 accounting at the AFTR, but notes that
>> >"the AFTR does not have detailed customer identity information."  Does
>> >that create a theft of service possibility?  If so, that possibility
>> >should be mentioned as a security consideration.  Also, Section 2.8
>> >ought to be expanded with a sentence or two explaining why the AFTR
>> >does not have that identity information, and in particular to explain
>> >whether and why it is unreasonable in some or all cases to expect
>> >that customer identity information be provided to the AFTR as part
>> >of provisioning each customer's softwire.
>> 
>> [YL] Good catch. Actually I believe AFTR "does" have both IPv4 and IPv6
>>to
>> identify the customer. I suggest we remove
>> 
>> "but it will potentially introduce some additional complexity because
>>    the AFTR does not have detailed customer identity information."
>> 
>> >
>> >Nits:
>> >
>> >Section 2.3 on MTU Considerations could usefully mention MTU discovery
>> >techniques, as possibilities for customer IPv4 traffic to adapt to a
>> >smaller IPv4 MTU.  If this is done, it would be useful to mention
>> >RFC 4821 in addition to the mention of RFC 1191 in RFC 6333.
>> 
>> [YL] We would consider RFC 4821. However, the challenge is I don't have
>> information how many current OS support RFC 4821. Since DS-lite is a
>> transition technology, it would be unreasonable to ask the OS company to
>> implement RFC 4821 for DS-lite.
>> 
>> >
>> >Section 2.4 should define "lawful intercept".  It would be helpful to
>> >cite a reference for this concept if something appropriate can be
>>found.
>> 
>> [YL] We will find whether there is any reference to this concept. If we
>> can't find any, we would add an explanation in the draft.
>> 
>> >
>> >Section 2.5:
>> >- Are one or both types of logging recommended?
>> 
>> [YL] We tempted to recommend source-specific log. However, some
>>regulators
>> require to use both and the regulations vary country from country, this
>>is
>> why we left it open and let the operators to decide.
>> 
>> >- Last paragraph on p.4, "timestamped log" is not a good verb phrase.
>> >	"maintain a timestamped log of" would be a better replacement.
>> 
>> [YL] Fixed
>> 
>> >- Last paragraph in section, where is the batch file sent?
>> 
>> [YL] We will add:
>> 
>> "The files may be compressed before transferring to a repository server
>> 	to better utilize bandwidth and storage."
>> 
>> 
>> >
>> >In Section 2.7, I'm having a hard time understanding which traffic is
>> >intended to be subject to the Outgoing Policies and the Incoming
>>Policies.
>> >Expanding each of those two bullets to explain what traffic is subject
>> >to each class of policies would help.
>> 
>> [YL] Does this help?
>> 
>> Outgoing Policies apply to packets originating from B4 to IPv4 network.
>> Incoming Policies apply to packets originating from IPv4 network to B4.
>> 
>> 
>> 
>> >
>> >Section 3.2.2.2 uses 192.0.0.2/32 as an example IP address for the
>> >B4.  That section should also cross-reference Section 5.7 on RFC 6333
>> >on IP address assignment to B4s, as other IP addresses may be used.
>> 
>> [YL] Added the cite.
>> 
>> >
>> >idnits 2.12.13 found a tiny nit - draft-ietf-pcp-base is now at
>> >version 28.
>> 
>> [YL] Fixed.
>> 
>> >
>> >Thanks,
>> >--David
>> >----------------------------------------------------
>> >David L. Black, Distinguished Engineer
>> >EMC Corporation, 176 South St., Hopkinton, MA  01748
>> >+1 (508) 293-7953             FAX: +1 (508) 293-7786
>> >david.black@xxxxxxx        Mobile: +1 (978) 394-7754
>> >----------------------------------------------------
>> >
>> >

<<attachment: smime.p7s>>


[Index of Archives]     [IETF Annoucements]     [IETF]     [IP Storage]     [Yosemite News]     [Linux SCTP]     [Linux Newbies]     [Fedora Users]