Re: 6MAN WG Last Call: <draft-ietf-6man-udpchecksum-02>

Marshall Eubanks <marshall.eubanks@gmail.com> Mon, 21 May 2012 18:01 UTC

Return-Path: <marshall.eubanks@gmail.com>
X-Original-To: ipv6@ietfa.amsl.com
Delivered-To: ipv6@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 6172921F85C7 for <ipv6@ietfa.amsl.com>; Mon, 21 May 2012 11:01:44 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -103.599
X-Spam-Level:
X-Spam-Status: No, score=-103.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, RCVD_IN_DNSWL_LOW=-1, USER_IN_WHITELIST=-100]
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 7wM0tVb1vGOx for <ipv6@ietfa.amsl.com>; Mon, 21 May 2012 11:01:43 -0700 (PDT)
Received: from mail-lpp01m010-f44.google.com (mail-lpp01m010-f44.google.com [209.85.215.44]) by ietfa.amsl.com (Postfix) with ESMTP id 7347721F859B for <ipv6@ietf.org>; Mon, 21 May 2012 11:01:43 -0700 (PDT)
Received: by lagv3 with SMTP id v3so4256368lag.31 for <ipv6@ietf.org>; Mon, 21 May 2012 11:01:42 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; bh=f7gmwzjyJZi8scj3HxG2k+TBEjxk5s+X8KIaEjHUpZs=; b=QQWmX6WgnUpCyJGLts9spA4EPs/a3363/QWd5B1LiHaFczeIYXylA/VOlkjCZBwwqa d+cJ5IWe9Ptw2OEChXrseI6cFp4umxMtvI+HCDPQtBU799FncUXlnCtEYhKVaQEth7jD G2/BRJiN8Uqha6rl+qVQ7b77z6FDu53z+OFcZqxGb5ych5wkOboEHGKi478KwqLSoDol NjfQH0M/2S0I1JTh4fWFtnvfkFR+/RKnrHeQHhwZUp4GkaLghVm4+JEAf1uBVG6EAZJk CCPhuJVmOE2Pub6NU1YYV8eaPtGAMSdLtvZ7hL+OP/vVLTEJtMADWJZJVCSvgCKUjDAd aT7A==
MIME-Version: 1.0
Received: by 10.112.88.34 with SMTP id bd2mr8268501lbb.33.1337623302417; Mon, 21 May 2012 11:01:42 -0700 (PDT)
Received: by 10.112.56.13 with HTTP; Mon, 21 May 2012 11:01:42 -0700 (PDT)
In-Reply-To: <4FA8CB45.1020309@ericsson.com>
References: <036E1A9D-88D9-4B1B-A7B1-FF4A624C5E13@gmail.com> <4FA8CB45.1020309@ericsson.com>
Date: Mon, 21 May 2012 14:01:42 -0400
Message-ID: <CAJNg7V+jzQbkUDd1nm2xp1jwF1mQYpjUx9JNWAwcSwKA0ZFL1w@mail.gmail.com>
Subject: Re: 6MAN WG Last Call: <draft-ietf-6man-udpchecksum-02>
From: Marshall Eubanks <marshall.eubanks@gmail.com>
To: Magnus Westerlund <magnus.westerlund@ericsson.com>
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: quoted-printable
Cc: "ipv6@ietf.org Mailing List" <ipv6@ietf.org>, Bob Hinden <bob.hinden@gmail.com>, "draft-ietf-6man-udpchecksums@tools.ietf.org" <draft-ietf-6man-udpchecksums@tools.ietf.org>
X-BeenThere: ipv6@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: "IPv6 Maintenance Working Group \(6man\)" <ipv6.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ipv6>, <mailto:ipv6-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ipv6>
List-Post: <mailto:ipv6@ietf.org>
List-Help: <mailto:ipv6-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ipv6>, <mailto:ipv6-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 21 May 2012 18:01:44 -0000

Dear Magnus;

On Tue, May 8, 2012 at 3:29 AM, Magnus Westerlund
<magnus.westerlund@ericsson.com> wrote:
> Hi,
>
> I know the WG last call has closed. But I reviewed it anyway and I have
> found some nits and things which the WG chairs anyway will stumble on in
> their ID checklist processing of the document. I also have some more
> substantial comments on how this is written. I am sorry that I am late,
> but I simple missed the WG last call.
>
> I do support this document going forward but I do think it needs to be
> updated prior to submission to the AD.
>
> 1) Abstract contains bracketed references. I would suggest replacing
> RFC2460[RFC2460] with the documents title and RFC number without brackets.
>

The brackets come from xml2rfc. This nit is will be done.

> 2) Document header does not contain "Updates:RFC2460"

OK, will be fixed.

>
> 3) Section 1, " RFC 2460[RFC2460], " I think this should be "Internet
> Protocol, Version 6 (IPv6) Specification [RFC2460],"
>

Isn't this just # 1 ? Or am I missing something?
> 4. Section 5
>     "However, some protocols, such as lightweight tunneling
>      protocols that use UDP as a tunnel encapsulation, MAY omit
>      computing the UDP checksum of the encapsulating UDP header and set
>      it to zero, subject to the constraints described in
>      [I-D.ietf-6man-udpzero]."
>
> I am a bit worried about making [I-D.ietf-6man-udpzero] part of the
> specification text. Especially something that I at least consider being
> normative definition of constraints. Those I think should be part of
> this document. I think the pointers earlier in the document is
> sufficient to establish that part of the inclusion of the constraints in
> this document are based on whats in our document.

Here is the problem with
>      [I-D.ietf-6man-udpzero]."

The intended reference is to _the current document_. Remember, this
text is part of

 This item should be taken out of the bullet list and should be
   modified as follows:

where "this item" is [RFC2460] Section 8.1, 4th bullet

What I was worried about was that someone would take our instructions
literally, and create a mashup of 2560 and the current document, as
that is what we tell them to do. Then, you don't want to just say "as
in Section X", as that is ambiguous. Nor do you want
to say "Section X, of this document," for the same reason. Ideally, if
this document was RFC YYYY, you would say "Section X of RFC YYYY."

However, can a document normatively reference itself ? That might
excite Douglas Hofstadter, but it seems dubious to me.

It does occur to me that this must have been dealt with previously -
do you know of previous solutions ?

>
> 5. Section 6. "It is now 2011." Can the authors confirm if there has
> been no more analysis and maybe indicate the current year.

Will do.

>
> In general when I read this document it reads to much like a discussion
> paper rather than a normative specification. I think it is late in the
> process and do not require any changes now. However, I have raised this
> earlier (on the 00 wg version) and the authors have not addressed this.
>

I actually did change a bunch of text based on that. I agree it is a
bit late to do more.

Marshall

> Cheers
>
> Magnus Westerlund
>
> ----------------------------------------------------------------------
> Multimedia Technologies, Ericsson Research EAB/TVM
> ----------------------------------------------------------------------
> Ericsson AB                | Phone  +46 10 7148287
> Färögatan 6                | Mobile +46 73 0949079
> SE-164 80 Stockholm, Sweden| mailto: magnus.westerlund@ericsson.com
> ----------------------------------------------------------------------
>