Re: [Gen-art] Gen-ART LC review of draft-ietf-mext-nemo-v4traversal-06.txt

Brian E Carpenter <brian.e.carpenter@gmail.com> Mon, 01 December 2008 20:17 UTC

Return-Path: <gen-art-bounces@ietf.org>
X-Original-To: gen-art-archive@optimus.ietf.org
Delivered-To: ietfarch-gen-art-archive@core3.amsl.com
Received: from [127.0.0.1] (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id C73023A682E; Mon, 1 Dec 2008 12:17:28 -0800 (PST)
X-Original-To: gen-art@core3.amsl.com
Delivered-To: gen-art@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 334573A698F for <gen-art@core3.amsl.com>; Mon, 1 Dec 2008 12:17:28 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.569
X-Spam-Level:
X-Spam-Status: No, score=-2.569 tagged_above=-999 required=5 tests=[AWL=0.030, BAYES_00=-2.599]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id RKAt9sjuNpPu for <gen-art@core3.amsl.com>; Mon, 1 Dec 2008 12:17:27 -0800 (PST)
Received: from ti-out-0910.google.com (ti-out-0910.google.com [209.85.142.190]) by core3.amsl.com (Postfix) with ESMTP id AE0F53A682E for <gen-art@ietf.org>; Mon, 1 Dec 2008 12:17:26 -0800 (PST)
Received: by ti-out-0910.google.com with SMTP id a6so1852629tib.25 for <gen-art@ietf.org>; Mon, 01 Dec 2008 12:17:22 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:message-id:date:from :organization:user-agent:mime-version:to:cc:subject:references :in-reply-to:content-type:content-transfer-encoding; bh=6BFINcOYxGiKBxuL38AedZ/8bYnECaBVnl8lFGK4N60=; b=Rb3zual1Gso/pZzCY7J/kuZG9Kift7gmWvPrOm6KeGpaCjWbNrOq+7vFyFtKGDE3BN gc8TMfPsXXhn3JGZYM8kziNVazieT9ctRgSnYpEaCceYywazGS2AlIm3SQKbjof7atmu 47qnPEr/xVJD26NyjFrNtgtUbLSql9oKM8sNo=
DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:organization:user-agent:mime-version:to:cc :subject:references:in-reply-to:content-type :content-transfer-encoding; b=LLA9hpNWv3ZsY49tw2K2or2acIU+GELjDEr40loNTin+qGSwv7O5OBTTZZwxeByQOT 7xBoNVCgS/IudvlKPZ2ustNrAF6JgAtciWIgwb+JC0YPPSgAJEpzrXn7t4p3xUsKbFXN 5Df2j1F66D0+rT5FGFMMPSS5aGxykuNGPDf3s=
Received: by 10.110.43.16 with SMTP id q16mr16931361tiq.2.1228162641801; Mon, 01 Dec 2008 12:17:21 -0800 (PST)
Received: from ?10.1.1.4? (118-93-184-220.dsl.dyn.ihug.co.nz [118.93.184.220]) by mx.google.com with ESMTPS id y3sm1698336tia.6.2008.12.01.12.17.17 (version=SSLv3 cipher=RC4-MD5); Mon, 01 Dec 2008 12:17:20 -0800 (PST)
Message-ID: <49344647.6080405@gmail.com>
Date: Tue, 02 Dec 2008 09:17:11 +1300
From: Brian E Carpenter <brian.e.carpenter@gmail.com>
Organization: University of Auckland
User-Agent: Thunderbird 2.0.0.6 (Windows/20070728)
MIME-Version: 1.0
To: Hesham Soliman <hesham@elevatemobile.com>
References: <C559F09B.A7C6%hesham@elevatemobile.com>
In-Reply-To: <C559F09B.A7C6%hesham@elevatemobile.com>
Cc: draft-ietf-mext-nemo-v4traversal@tools.ietf.org, mext-chairs@tools.ietf.org, General Area Review Team <gen-art@ietf.org>, Jari Arkko <jari.arkko@piuha.net>
Subject: Re: [Gen-art] Gen-ART LC review of draft-ietf-mext-nemo-v4traversal-06.txt
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/pipermail/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>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Sender: gen-art-bounces@ietf.org
Errors-To: gen-art-bounces@ietf.org

Hi Hesham, see in line. Especially on the GRE point, I think
there's still an open issue.

On 2008-12-01 21:50, Hesham Soliman wrote:
> Hi Brian,
> 
> Thanks for the review and apologies for the late response. I'll only respond
> to comments that need further discussions or include your questions. I won't
> include comments that I'll address as you requested.
> 
>> 3.3.2.  Foreign Network Supports IPv4 Only
>>
>>    If the mobile node is in a foreign network that only supports IPv4,
>>    it needs to detect whether a NAT is in its communication path to the
>>    home agent.  This is done while exchanging the binding update and
>>    acknowledgement messages as shown later in this document.  NAT
>>    detection is needed for the purposes of the signaling presented in
>>    this specification.  A mobile node SHOULD NOT assume that its IPv4
>>    address is globally unique if a NAT device was not detected.
> 
> 
> 
> Don't understand the last sentence. How, then, does the node choose to
> enter the "3.3.2.1.  Foreign Network Supports IPv4 Only (Public Addresses)"
> logic?
> 
> => Perhaps it was my clumsy way of saying "just because there is no NAT
> between a MN and the HA, doesn't mean your address is globally unique",
> which is true of course. But I'm not sure it's a useful comment. So I can
> remove that. 

Yes, I think in this case deletion is the best clarification :-)

> 
>>    F
>>
>>       When set, this flag indicates a request for forcing UDP
>>       encapsulation regardless of whether a NAT is present on the path
>>       between the mobile node and the home agent.
> 
> 
> 
> In what circumstances is this set? (Same question for 4.2.2.)
> 
> => If the MN is configured by a policy that requires it to always use UDP
> encapsulation, or if it tried normal IP in IP and it didn't work. Presumably
> the latter happened because the firewall didn't allow IP in IP.

OK. I think it would be clearer for the reader if you add this information.

> 
>>    IPv4 Home Address
>>
>>       The IPv4 home address that the home agent will use in the binding
>>       cache entry.  This could be a public or private address.
> 
> 
> 
> But if it's private, it's of no value to the mobile, which cannot send
> to it. Surely the mobile (and possibly its correspondent node) needs to
> know the public address of the NAT that hides the private home address?
> 
> => This is the home address of the mobile node. It doesn't send to it. I
> think you must've thought it was the home agent's address?

I certainly wasn't thinking quite clearly. Of course, it's the correspondent
node that can never send directly to the private home address; the
correspondent node will only ever know the NATted home address.

> 
>> 4.2.2.  The NAT Detection Option
> 
>>    This option is sent from the home agent to the mobile node to
>>    indicate whether a NAT was in the path.  This option MAY also include
>>    a suggested NAT binding refresh time for the mobile node.
> 
> 
> 
> The home agent has even less chance than the mobile of guessing the NAT's
> timeout. So why is this refresh time a useful addition to the protocol,
> rather than just defining the mobile node's default refresh as NATKATIMEOUT?
> 
> => It was suggested by people to allow home agent's that are usually within
> the same admin domain to configure longer periods based on administrative
> configuration. This is for networks like 3GPP networks where the mobile node
> is usually within the same admin domain.

OK, as long as there is a reason for the extra complication.

> 
> 
>> 5.1.  Tunelling Formats
>>    The following value is assigned to the Type field, other values may
>>    be assigned in the future:
>>
>>       1 GRE
> 
> 
> 
> I'm puzzled by the choice of acronym. GRE refers to RFC2784 which has its
> own encapsulation header format.
> 
> => I know, it's the same GRE. People wanted to put GRE encapsulation in this
> specification.

Well in that case there needs to be a reference to RFC2784 and perhaps
RFC2890. I'm quite confused, because there is nothing in the text that
refers to GRE encapsulation - the only specific encapsulation mentioned
is RFC2473 and RFC4213, and neither of those use GRE as far as I know.
You can't use the same TLV to indicate both GRE and non-GRE encapsulations.

> 
>>    To accommodate traffic that uses Explicit Congestion Notification
>>    (ECN), it is RECOMMENDED that the ECN information is copied between
>>    the inner and outer header as defined in [RFC3168].
> 
> 
> 
> OK, but why not also suggest that the DSCP be copied, as defined in
> RFC2983 (which is Informational)?
> 
> => Ok, but I'll use 4213, which obsoletes 2983.

Wrong! It obsoletes 2893. Very confusing...

> 
>> 5.2.  NAT Detection
>>    Upon receiving the binding acknowledgement with the NAT detection
>>    option, the mobile node sets the tunnel to the home agent to UDP
>>    encapsulation.  Hence, all future packets to the home agent are
>>    tunneled in UDP and IPv4.  For all tunneled IPv6 packets, the source
>>    address in the IPv6 header is the mobile node's IPv6 home address and
>>    the destination address is the correspondent node's IPv6 address.
>>    All tunneled IPv4 packets will contain the mobile node's IPv4 home
>>    address in the source address field of the inner IPv4 packet and the
>>    correspondent node's IPv4 address in the destination address field.
> 
> 
> 
> Where did the correspondent node's IPv4 address come from?
> 
> => Well, the mobile node is talking to a correspondent node. The address
> could come from anywhere. This is simply an illustration of how traffic is
> routed from a correspondent node to a mobile node and vice versa.

OK, but I believe this is almost the only place that the correspondent
node is mentioned, so it came as a bit of a surprise.

> 
>>    A mobile node MUST always tunnel binding updates in UDP when located
>>    in an IPv4-only network.
> 
> 
> 
> How does the mobile know that it's in an IPv4-only network? (Also, there's
> a missing verb in that sentence.)
> 
> => Ok about the verb. The mobile node knows because it didn't receive any
> IPv6 information from the router, i.e. Router advertisements.

I think you should say a few words about that (i.e. that you assume that
the stack provides a status indicator of some kind for this, since this
is not a part of MIP6 itself).

    Brian
_______________________________________________
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art