Review of draft-ietf-softwire-multicast-prefix-option-11

Sheng Jiang <jiangsheng@huawei.com> Tue, 10 January 2017 03:55 UTC

Return-Path: <jiangsheng@huawei.com>
X-Original-To: ietf@ietf.org
Delivered-To: ietf@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id DE716129A59; Mon, 9 Jan 2017 19:55:01 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Sheng Jiang <jiangsheng@huawei.com>
To: ops-dir@ietf.org
Subject: Review of draft-ietf-softwire-multicast-prefix-option-11
X-Test-IDTracker: no
X-IETF-IDTracker: 6.40.3
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <148402050186.25046.4223816824977657511.idtracker@ietfa.amsl.com>
Date: Mon, 09 Jan 2017 19:55:01 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/ietf/JkQFnU37TTrCpzNH1H--3NnSmns>
Cc: softwires@ietf.org, ietf@ietf.org, draft-ietf-softwire-multicast-prefix-option.all@ietf.org
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.17
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 10 Jan 2017 03:55:02 -0000

Reviewer: Sheng Jiang
Review result: Has Nits

Summary: This draft is almost ready for publication as a standard
track RFC.

Major issues:

Minor issues:

“the specification of a DHCPv6 option that could be used to discover
   unicast PREFIX64s in environments where multicast is not enabled.
   Such side effect conflicts with the recommendation documented in
   Section 6 of [RFC7051].”

It is unclear how the Section 6 of RFC7051 relevant with the content
above. It would be necessary to quote particular content of RFC7051
and give necessary analysis.

Nits:

“the Pv4 multicast address is inserted in the last 32 bits of the
IPv4-embedded IPv6
   multicast address.”

Pv4//IPv4