Re: [Gen-art] Gen-ART review of draft-ietf-lisp-multicast-08

Dino Farinacci <dino@cisco.com> Thu, 06 October 2011 04:52 UTC

Return-Path: <dino@cisco.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 1308321F8C65 for <gen-art@ietfa.amsl.com>; Wed, 5 Oct 2011 21:52:44 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.235
X-Spam-Level:
X-Spam-Status: No, score=-3.235 tagged_above=-999 required=5 tests=[AWL=-0.636, BAYES_00=-2.599]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id hTR22TD5E+ah for <gen-art@ietfa.amsl.com>; Wed, 5 Oct 2011 21:52:43 -0700 (PDT)
Received: from mtv-iport-2.cisco.com (mtv-iport-2.cisco.com [173.36.130.13]) by ietfa.amsl.com (Postfix) with ESMTP id 0E51721F8C5D for <gen-art@ietf.org>; Wed, 5 Oct 2011 21:52:43 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=dino@cisco.com; l=3754; q=dns/txt; s=iport; t=1317876953; x=1319086553; h=subject:mime-version:from:in-reply-to:date:cc: content-transfer-encoding:message-id:references:to; bh=2ARICYXQdBO6p0AGGX4r6iwfCgto/3wuxrUM6128XdI=; b=N0cJsbM3wlBJ0RK7nRmAtRzMJhGP6Sw4gi2nwx3Xzrn39LvfcTw2KMLQ 52y5LxwDf7ApHC/gPHZ/oS+58y1fs4Eri4VaCNXvf1ktGUwGflgzpuzQA lkqvDDrjYIB0ot+xFNOILlq35BRBATwjZFAeE1Q9HhOTi57n4ZGVI0T6h 8=;
X-IronPort-AV: E=Sophos;i="4.68,495,1312156800"; d="scan'208";a="6221058"
Received: from mtv-core-4.cisco.com ([171.68.58.9]) by mtv-iport-2.cisco.com with ESMTP; 06 Oct 2011 04:55:53 +0000
Received: from sjc-vpn4-254.cisco.com (sjc-vpn4-254.cisco.com [10.21.80.254]) by mtv-core-4.cisco.com (8.14.3/8.14.3) with ESMTP id p964tqrs026919; Thu, 6 Oct 2011 04:55:52 GMT
Mime-Version: 1.0 (Apple Message framework v1244.3)
Content-Type: text/plain; charset="us-ascii"
From: Dino Farinacci <dino@cisco.com>
In-Reply-To: <AE31510960917D478171C79369B660FA0E091162A4@MX06A.corp.emc.com>
Date: Wed, 05 Oct 2011 21:55:50 -0700
Content-Transfer-Encoding: quoted-printable
Message-Id: <F9019FFF-F769-47B3-AAB0-EFF42A4B77E7@cisco.com>
References: <AE31510960917D478171C79369B660FA0E091162A4@MX06A.corp.emc.com>
To: "<kathleen.moriarty@emc.com>" <kathleen.moriarty@emc.com>
X-Mailer: Apple Mail (2.1244.3)
X-Mailman-Approved-At: Thu, 06 Oct 2011 05:14:33 -0700
Cc: gen-art@ietf.org, David Meyer <dmm@cisco.com>, draft-ietf-lisp-multicast.all@tools.ieft.org, Stig Venaas <stig@cisco.com>
Subject: Re: [Gen-art] Gen-ART review of draft-ietf-lisp-multicast-08
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 06 Oct 2011 04:52:44 -0000

Kathleen, thanks for the comments. I have made changes to reflect all your editorial comments. I will be posting a draft-ietf-lisp-multicast-09.txt with yours and Ralph's comments.

Dino

On Oct 3, 2011, at 8:23 PM, <kathleen.moriarty@emc.com> <kathleen.moriarty@emc.com> 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-lisp-multicast-08
> Reviewer: Kathleen Moriarty
> Review Date: 3 October 2011
> IETF LC End Date: 28 September 2011
> IESG Telechat date: (if known)
> 
> Summary:  The document is ready with nits.  I am sorry about the late review!
> 
> Major issues:
> 
> Minor issues:
> 
> Nits/editorial comments:
> The last paragraph of section 3 introduces the use of the term 'oif-list' within a definition for 'Unicast Encapsulated PIM Join/Prune Message'.  I think it would be helpful to define this term.
> 
> Section 4, consider breaking the first sentence of #6 into two sentences:
> "When a packet is originated by the multicast host in the source
>     site, it will flow to one or more ITRs which will prepend a LISP
>     header by copying the group address to the outer destination
>     address field and insert its own locator address in the outer
>     source address field."
> 
> Section 5, 2nd paragraph: Recommend changing from:
> "In a LISP site, packets
> are originated from hosts using their allocated EIDs, those addresses
> are used to identify the host as well as where in the site's topology
> the host resides but not how and where it is attached to the
> Internet."
> To: "In a LISP site, packets originate from hosts using their allocated EIDs.  EID addresses
> are used to identify the host as well as where in the site's topology
> the host resides, but not how and where it is attached to the
> Internet."
> 
> Section 7, IGMP section, second sentence, add a comma:
> To: "One being that they are link-
>    local and not used over site boundaries and second, they advertise
>    group addresses that don't need translation."
> 
> Section 7: PIM-SSN - consider breaking this into a couple of sentences to make it easier to read.
> "In this case, there is a small
>    modification to the operation of the PIM protocol (but not to any
>    message format) to support taking a Join/Prune message originated
>    inside of a LISP site with embedded addresses from the EID
>    namespace and converting them to addresses from the RLOC namespace
>    when the Join/Prune message crosses a site boundary."
> 
> Section 7: PIM-Bidir Section - consider breaking the following into a couple of sentences:
> "When using
>    Bidir-PIM for inter-domain multicast routing, it is recommended to
>    use staticly configured RPs so core routers think the Bidir group
>    is associated with an ITR's RLOC as the RP address and site
>    routers think the Bidir group is associated with the site resident
>    RP with an EID address."
> 
> Section 9.2: This section is the first place where 'you' is used.  The writing style changes, you may want to rewrite and make it consistent with the rest of the document.  There is a 'you' in 9.3 as well and 'we' is used in section 11.
> 
> Section 12: Should the 'must' in the first paragraph be in caps?
> "Mtrace functionality must be consistent with unicast traceroute
> functionality where all hops from multicast receiver to multicast
> source are visible."
> 
> 
> 
> 
> 
> 
>