[dhcwg] Comments on draft-farrer-dhc-shared-address-lease

Qi Sun <sunqi.csnet.thu@gmail.com> Fri, 18 October 2013 09:31 UTC

Return-Path: <sunqi.csnet.thu@gmail.com>
X-Original-To: dhcwg@ietfa.amsl.com
Delivered-To: dhcwg@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id A1A4C11E810B for <dhcwg@ietfa.amsl.com>; Fri, 18 Oct 2013 02:31:25 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.599
X-Spam-Level:
X-Spam-Status: No, score=-2.599 tagged_above=-999 required=5 tests=[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 p7Xs8500wE0z for <dhcwg@ietfa.amsl.com>; Fri, 18 Oct 2013 02:31:23 -0700 (PDT)
Received: from mail-pd0-x229.google.com (mail-pd0-x229.google.com [IPv6:2607:f8b0:400e:c02::229]) by ietfa.amsl.com (Postfix) with ESMTP id C413C21F9CE9 for <dhcwg@ietf.org>; Fri, 18 Oct 2013 02:31:22 -0700 (PDT)
Received: by mail-pd0-f169.google.com with SMTP id q10so2943027pdj.14 for <dhcwg@ietf.org>; Fri, 18 Oct 2013 02:31:22 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:content-type:content-transfer-encoding:subject:date:message-id :cc:to:mime-version; bh=1YCHI+9Lrz6q8eKk3HNQhMOH/GVgr9dLVIkRPMNyWIk=; b=E221FVzQB23dT5WXSR+QGZ4naIxCTK70dgIJ3QPiFg59xR26Vi1UJGTTeAIhL16GH+ Ookx/aoZy+4q3CkAVyL94F9MvS+y6vZTx/etEz2rSE5A2U4cgWWQK3V55wmxlw4UnDLT o0b0sMmZmCrDfWSgQpIm8BkU/u9fqaK+iMe2aPTZUx30fzepyUS1YBmclh3Uat3o9Hpt AHzvd2BaJ0NUog+YFRuMgWWUBJpRua/rIsOaL3ZO6lHGWgdY7YPPoJ5ZTFEyiwwwIZ5e MIoqQov/LejHWkWWWfdn7/+QamPZqByQp99S3vs4Ly58iaDaN+ko01w1Xtb57RPR7h74 8gbA==
X-Received: by 10.67.24.7 with SMTP id ie7mr2562749pad.112.1382088682074; Fri, 18 Oct 2013 02:31:22 -0700 (PDT)
Received: from [192.168.199.122] ([166.111.68.231]) by mx.google.com with ESMTPSA id rv9sm1183070pbc.4.2013.10.18.02.31.20 for <multiple recipients> (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Fri, 18 Oct 2013 02:31:21 -0700 (PDT)
From: Qi Sun <sunqi.csnet.thu@gmail.com>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: quoted-printable
Date: Fri, 18 Oct 2013 17:31:17 +0800
Message-Id: <29A6DD83-8E2B-45AF-8C60-802FDEDEBC5C@gmail.com>
To: draft-farrer-dhc-shared-address-lease@tools.ietf.org
Mime-Version: 1.0 (Apple Message framework v1085)
X-Mailer: Apple Mail (2.1085)
Cc: "<dhcwg@ietf.org> WG" <dhcwg@ietf.org>
Subject: [dhcwg] Comments on draft-farrer-dhc-shared-address-lease
X-BeenThere: dhcwg@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: <dhcwg.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/dhcwg>, <mailto:dhcwg-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/dhcwg>
List-Post: <mailto:dhcwg@ietf.org>
List-Help: <mailto:dhcwg-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/dhcwg>, <mailto:dhcwg-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 18 Oct 2013 09:31:26 -0000

Dear authors,

I read this document and have some comments on it. Please see whether the comments are useful for the document.

Best Regards,
Qi

<Comments>

1. General comments:

1) The port set option is expected to be used by a various of mechanisms besides DHCPv4 over DHCPv6, like DHCPv4 over IPv6, DHCPv4 over Softwire, etc. If we define this option restricted to the context of DHCPv4 over DHCPv6, then in v4configuration draft DHCPv4 over IPv6 and DHCPv4 over Softwire are not supposed to support the port set allocation, which is not true.
2) Section 7 is discussing another topic. IMHO, it would be clearer to put it into another draft. 

2. Some detailed comments:
1) sec 3.1
The sentence of 'The following message flow is transported within the DHCPv6 OPTION_BOOTP_MSG message' indicates that the described message flow is within the context of DHCPv4 over DHCPv6. So IMHO, it would be not necessary to say again those DHCPv4 messages are within BOOTPREQUESTV6/BOOTREPLYV6 messages.

2) sec 3.1, bullet 1
The client MAY insert a non-zero value in the PSID-Len field
       within OPTION_PORTPARAMSV4 to indicate the preferred size of the
       restricted port range allocation to the unified server.
=> ... to indicate to the 4o6 Server the preferred size of the restricted port range allocation. 

3) sec 3.2
..., with the client's requested port set being included in the DHCPDISCOVER message.
DHCPDISCOVER => DHCPREQUEST

4) sec 4
I think not all the functions are expected to be a DHCP client's. The client can configure a NAPT within a restricted port range. The rest should be done by the NAPT functionalities.

5) sec 6
"This is to maintain compatibility with existing implementations."
There are various implementation of the port set currently. This sentence is not necessary here, IMHO.

client MUST use apply this PSID to the interface... => ... MUST apply ... (remove use)

6) sec 7
   To provide this function, the DHCPV4oDHCPv6 client MAY implement
   OPTION_DHCPV4_O_DHCPV6_SADDR (defined below).  This option is
   included by the client within OPTION_BOOTP_MSG messages and is used
   alongside the DHCPv4 request process

OPTION_DHCPV4_O_DHCPV6_SADDR is a DHCPv6 option. This option should not be included in the OPTION_BOOTP_MSG option, which is used for encapsulating DHCPv4 messages. The two options are the same level, IMHO. 

3. Some nits:
1) Unified Server -> 4o6 Server
2) sec 3.1, bullet 1
transported within the DHCPv6 BOOTPRELAYV6 message => BOOTPREQUESTV6 message

</Comments>