Re: [Softwires] RtgDir review: draft-ietf-softwire-dslite-multicast-14.txt

<mohamed.boucadair@orange.com> Tue, 24 January 2017 07:08 UTC

Return-Path: <mohamed.boucadair@orange.com>
X-Original-To: softwires@ietfa.amsl.com
Delivered-To: softwires@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 44FDA129577; Mon, 23 Jan 2017 23:08:22 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -5.818
X-Spam-Level:
X-Spam-Status: No, score=-5.818 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-3.199, SPF_PASS=-0.001, UNPARSEABLE_RELAY=0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id c-saGkgdK8PX; Mon, 23 Jan 2017 23:08:21 -0800 (PST)
Received: from relais-inet.orange.com (mta239.mail.business.static.orange.com [80.12.66.39]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id A9B0B126CD8; Mon, 23 Jan 2017 23:08:20 -0800 (PST)
Received: from opfedar07.francetelecom.fr (unknown [xx.xx.xx.9]) by opfedar26.francetelecom.fr (ESMTP service) with ESMTP id 00F5D1C07DD; Tue, 24 Jan 2017 08:08:19 +0100 (CET)
Received: from Exchangemail-eme2.itn.ftgroup (unknown [xx.xx.31.10]) by opfedar07.francetelecom.fr (ESMTP service) with ESMTP id C2F53C0056; Tue, 24 Jan 2017 08:08:18 +0100 (CET)
Received: from OPEXCLILMA3.corporate.adroot.infra.ftgroup ([fe80::60a9:abc3:86e6:2541]) by OPEXCLILM5C.corporate.adroot.infra.ftgroup ([fe80::4bd:9b2b:3651:6fba%19]) with mapi id 14.03.0319.002; Tue, 24 Jan 2017 08:08:18 +0100
From: mohamed.boucadair@orange.com
To: Stig Venaas <stig@venaas.com>
Thread-Topic: RtgDir review: draft-ietf-softwire-dslite-multicast-14.txt
Thread-Index: AQHSbDe0hVADs41lCEykxiP/w7snr6E0YD9ggBLDx0+AACRx4A==
Date: Tue, 24 Jan 2017 07:08:17 +0000
Message-ID: <787AE7BB302AE849A7480A190F8B933009DE8E65@OPEXCLILMA3.corporate.adroot.infra.ftgroup>
References: <CAHANBtLtJxyoAdMMd05UfQK=5hE55CDpusjFycGTyB0s3T3-ig@mail.gmail.com> <787AE7BB302AE849A7480A190F8B933009DE3299@OPEXCLILMA3.corporate.adroot.infra.ftgroup> <CAHANBt+kSGCiZDHsgoMvhZ2-hK5AxVk6xKEUz7SO9V-QjNu3OQ@mail.gmail.com> <D66827DC-BBF5-47C9-A756-16F464F63B8B@comcast.com> <CAHANBtJLg5efELR+kPqmeUF_LwV_dFUoUr-wknJBUZnNOEXksA@mail.gmail.com> <CAHANBt+Z2vaV8jsu0Y_WGtvueKgBBbtm0BUPTOU5kuo7yvwJhg@mail.gmail.com> <787AE7BB302AE849A7480A190F8B933009DE8997@OPEXCLILMA3.corporate.adroot.infra.ftgroup> <CAHANBt+cRZqMULhfo22hrr1vJMDFUm7igRLbR+=fCC5rHEThYg@mail.gmail.com>
In-Reply-To: <CAHANBt+cRZqMULhfo22hrr1vJMDFUm7igRLbR+=fCC5rHEThYg@mail.gmail.com>
Accept-Language: fr-FR, en-US
Content-Language: fr-FR
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.168.234.1]
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/softwires/LBJ_CAjzyP9CfhwFM7mpSAu__eU>
Cc: "rtg-ads@ietf.org" <rtg-ads@ietf.org>, "draft-ietf-softwire-dslite-multicast.all@ietf.org" <draft-ietf-softwire-dslite-multicast.all@ietf.org>, "softwires@ietf.org" <softwires@ietf.org>, "Lee, Yiu" <Yiu_Lee@comcast.com>, "rtg-dir@ietf.org" <rtg-dir@ietf.org>
Subject: Re: [Softwires] RtgDir review: draft-ietf-softwire-dslite-multicast-14.txt
X-BeenThere: softwires@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: softwires wg discussion list <softwires.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/softwires>, <mailto:softwires-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/softwires/>
List-Post: <mailto:softwires@ietf.org>
List-Help: <mailto:softwires-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/softwires>, <mailto:softwires-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 24 Jan 2017 07:08:22 -0000

Hi Stig, 

Works for me. 

The changes are implemented copy. I will wait for other reviews before submitting a new revision.

Thank you.

Cheers,
Med

> -----Message d'origine-----
> De : Stig Venaas [mailto:stig@venaas.com]
> Envoyé : mardi 24 janvier 2017 05:56
> À : BOUCADAIR Mohamed IMT/OLN
> Cc : Lee, Yiu; rtg-ads@ietf.org; draft-ietf-softwire-dslite-
> multicast.all@ietf.org; rtg-dir@ietf.org; softwires@ietf.org
> Objet : Re: RtgDir review: draft-ietf-softwire-dslite-multicast-14.txt
> 
> Hi
> 
> On Mon, Jan 23, 2017 at 2:51 AM,  <mohamed.boucadair@orange.com> wrote:
> > Hi Stig,
> >
> > Please see inline.
> >
> > Cheers,
> > Med
> >
> >> -----Message d'origine-----
> >> De : Stig Venaas [mailto:stig@venaas.com]
> >> Envoyé : mercredi 18 janvier 2017 19:27
> >> À : Lee, Yiu
> >> Cc : BOUCADAIR Mohamed IMT/OLN; rtg-ads@ietf.org; draft-ietf-softwire-
> >> dslite-multicast.all@ietf.org; rtg-dir@ietf.org; softwires@ietf.org
> >> Objet : Re: RtgDir review: draft-ietf-softwire-dslite-multicast-14.txt
> >>
> >> Hi
> >>
> >> The good version looks pretty good. But I noticed one more thing that
> >> you might want to change. I'll leave it to you.
> >>
> >> 8.1.1.  The MLD Querier is Co-Located with the mAFTR
> >>
> >>    The mAFTR can embed the MLD Querier function (as well as the PIMv6
> >>    DR) for optimization purposes.
> >>
> >> I'm wondering if you should say IPv6 DR or PIMv6 DR instead. As you
> >> write after the heading, it must be co-located with the DR as well.
> >> Normally the querier would be the same router, but the querier could
> >> be another router on the link.
> >
> > [Med] Do you suggest to replace in the title "MLD Querier" with "PIMv6
> DR"?
> 
> Yes, or I guess IPv6 DR is better. You used the term IPv4 DR below.
> 
> I'm not sure if you have it in your document, I know I should check,
> but it would be good somewhere to say that DR is the Designated Router
> as defined in RFC 7761.
> 
> Stig
> 
> >>
> >> Then we have this section:
> >>
> >> 8.1.2. The DR is Co-Located with the mAFTR
> >>
> >> In the text you talk about the DR connected to the IPv4 source. Hence
> >> it is the first hop DR, and IPv4 DR. I'm wondering if it is worth
> >> saying IPv4 DR or something in the title to distinguish it from the
> >> other DR. Basically we have first hop DR and last hop DR, or if you
> >> like IPv4 DR and IPv6 DR.
> >>
> >
> > [Med] Makes sense to be explicit here. I changed DR to IPv4 DR.
> >
> >> I'm leaving it to you to decide. This is the only remaining issue I
> >> can find. There is a typo at the end of Appendix B though. It say
> >> "Noet".
> >
> > [Med] Fixed it. Thanks.
> >
> >>
> >> Stig
> >>
> >>
> >> On Fri, Jan 13, 2017 at 12:33 PM, Stig Venaas <stig@venaas.com> wrote:
> >> > Sounds good, I'll still try to have another look when you post the
> >> > next revision.
> >> >
> >> > Stig