The MPLS WG Archive

Cell Relay Retreat>MPLS WG Archive>month:2003-Aug> msg00023



[Date Prev][Date Next][Thread Prev][Thread Next]  
  [Date Index][Thread Index][Author Index][Subject Index]

draft-ietf-mpls-in-ip-or-gre-01.txt to PS

  • From: Mark Duffy <mduffy@quarrytech.com>
  • Date: Mon, 11 Aug 2003 15:24:11 -0400
  • Cc: mpls@UU.NET, erosen@cisco.com

At 09:35 AM 8/11/2003 -0400, George Swallow wrote:
>I personally believe this document is ready for last call.  I'll give
>it a few days before we start the official last call to see if there
>are any objections.
>
>...George


Hi,
I sent some comments in response to the previous last call (?) in April, 
which have not been addressed.  They are included below.
Thanks, Mark


>X-Sender: mduffy@email
>X-Mailer: QUALCOMM Windows Eudora Version 5.2.0.9
>Date: Fri, 11 Apr 2003 17:38:33 -0400
>To: mpls@UU.NET,erosen@cisco.com
>From: Mark Duffy <mduffy@quarrytech.com>
>Subject: Re: MPLS WG Last Call draft-ietf-mpls-in-ip-or-gre-00.txt
>In-Reply-To: <200303311913.OAA05418@bifocal.cisco.com>
>Mime-Version: 1.0
>Content-Type: text/plain; charset="us-ascii"; format=flowed
>
>
>>This message begins an MPLS Workgroup last call on
>>
>>   Encapsulating MPLS in IP or GRE
>>     draft-ietf-mpls-in-ip-or-gre-00.txt
>
>Hi, I have a few questions/comments on the draft.
>
>1.  In section 2 there are protocol numbers 'X' and 'Y' called out.  I 
>presume these are to be replaced with values assigned by IANA.  I don't 
>know what the politics are by which values get assigned or denied but 
>since the 'Y' value for MPLS multicast "is for further study" perhaps IANA 
>should only be asked for an 'X' value at this time.  I should think a 
>request for one value is more likely to succeed than a request for two.
>
>
>2.  In sect 4.1, there is a clear implication that the "Tunnel MTU" 
>describes the maximum sized *MPLS packet* that can be encapsulated in the 
>IP or GRE packet.  Therefore I think the following paragraph is not quite 
>correct:
>
>    In some cases, the tunnel head receives, for encapsulation, an IP
>    packet, which it first encapsulates in MPLS and then encapsulates in
>    MPLS-in-IP or MPLS-in-GRE.  If the source of the IP packet is
>    reachable from the tunnel head, and if the result of this
>    encapsulation would be a packet whose size exceeds the Tunnel MTU,
>    then the tunnel head SHOULD use the Tunnel MTU value for the purposes
>    of fragmentation and PMTU discovery outside the tunnel.
>
>I think it should instead read something like:
>
>    In some cases, the tunnel head receives, for encapsulation, an IP
>    packet, which it first encapsulates in MPLS and then encapsulates in
>    MPLS-in-IP or MPLS-in-GRE.  If the source of the IP packet is
>    reachable from the tunnel head, and if the result of **the MPLS**
>    encapsulation would be a packet whose size exceeds the Tunnel MTU,
>    then the tunnel head SHOULD use the Tunnel MTU value, **less the size
>    of the added MPLS label stack,** for the purposes of fragmentation
>    and PMTU discovery outside the **MPLS LSPs and encapsulating** tunnel.
>
>
>3.  A few typos:
>
>In the last sentence of sect. 2 and the exact same text again at the last 
>sentence of sect. 3:  s/is the topmost packet of the decapsulated 
>packet/is the topmost label of the decapsulated packet/
>
>In sect 4.1  s/mpls packet be decapsulated/mpls packet can be decapsulated/
>
>In sect 8 s/RFC7915/RFC791/
>
>Thanks,
>Mark