Re: [Int-area] I-D Action: draft-ietf-intarea-tunnels-05.txt

Joe Touch <touch@isi.edu> Tue, 28 March 2017 19:41 UTC

Return-Path: <touch@isi.edu>
X-Original-To: int-area@ietfa.amsl.com
Delivered-To: int-area@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 2FF95129564 for <int-area@ietfa.amsl.com>; Tue, 28 Mar 2017 12:41:36 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.901
X-Spam-Level:
X-Spam-Status: No, score=-6.901 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_HI=-5, RP_MATCHES_RCVD=-0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id zv6NaUXrxJ58 for <int-area@ietfa.amsl.com>; Tue, 28 Mar 2017 12:41:34 -0700 (PDT)
Received: from vapor.isi.edu (vapor.isi.edu [128.9.64.64]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 8068B12955F for <int-area@ietf.org>; Tue, 28 Mar 2017 12:41:34 -0700 (PDT)
Received: from [128.9.160.211] (mul.isi.edu [128.9.160.211]) (authenticated bits=0) by vapor.isi.edu (8.13.8/8.13.8) with ESMTP id v2SJc5ix019426 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NOT); Tue, 28 Mar 2017 12:38:06 -0700 (PDT)
To: "Templin, Fred L" <Fred.L.Templin@boeing.com>
References: <149062888196.30638.8369941985115982808@ietfa.amsl.com> <f5ab0422-fd49-9082-147b-8312e974de7e@isi.edu> <4d2a86f4948c4dc49ab3b0729743d028@XCH15-06-08.nw.nos.boeing.com>
Cc: "int-area@ietf.org" <int-area@ietf.org>
From: Joe Touch <touch@isi.edu>
Message-ID: <583e59d2-f846-6cd6-8e15-f3a0888889ac@isi.edu>
Date: Tue, 28 Mar 2017 12:38:05 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0
MIME-Version: 1.0
In-Reply-To: <4d2a86f4948c4dc49ab3b0729743d028@XCH15-06-08.nw.nos.boeing.com>
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 7bit
X-ISI-4-43-8-MailScanner: Found to be clean
X-MailScanner-From: touch@isi.edu
Archived-At: <https://mailarchive.ietf.org/arch/msg/int-area/SZNSmo5Tmtm8vDHpunBKkO6w7l0>
Subject: Re: [Int-area] I-D Action: draft-ietf-intarea-tunnels-05.txt
X-BeenThere: int-area@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: IETF Internet Area Mailing List <int-area.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/int-area>, <mailto:int-area-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/int-area/>
List-Post: <mailto:int-area@ietf.org>
List-Help: <mailto:int-area-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/int-area>, <mailto:int-area-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 28 Mar 2017 19:41:36 -0000

Hi, Fred,

Thanks for the feedback. Let's assume I incorporate the nits/bugs and I
address any clarifications when I get to them in detail, so I'll jump
only to the deeper issues below...

Joe


On 3/28/2017 9:36 AM, Templin, Fred L wrote:
> Here are my comments for this draft. Mostly editorial, but some substantial.
> I also noticed several indications that further work was needed in some
> sections. Will those sections be worked before publication?
As per Mark's presentation, it depends on whether this doc stays
informational (as was adopted) or becomes BCP (as we think it probably
should).

If informational, much of Section 5 will be removed. If not, then that
will be fleshed out.

> 10) Section 4.2.1, bulleted list toward bottom of section, tunnel
>    "atom" is a very strange word to me. Tunnel "cell"?

The concept of an atomic packet was defined in RFC 6864. This is derived
from that. Cell would be introducing a new term, one that is overloaded
with ATM, which we want to avoid.

> 13) Section 4.2.2, final sentence is incorrect. RFC4821-style MTU
>     probing cannot be used by tunnels due to ECMP because the probe
>     packets may take a different path than the data packets. That
>     is why AERO no longer uses RFC4821 probing.

Regarding ECMP issues, I think we need to wrap this issue up. Here's
what I propose:

- point out that ECMP causes problems with PMTUD (and can cause problems
with PLMTUD).
- an interface has two choices:
    - keep track of PMTU based on other packet context (flowID or next
header info)
    - merge PMTU feedback, taking the MIN of reported values

There's no magic here. It's a lot like multicast - either keep track in
a way that you *think* correlates to the different PMTU feedback or take
the MIN.

The current doc does need a scrub to make this point clearly and
consistently.

> 14) Section 4.2.2, should include discussion of how common tunnel
>     specs regard EMTU_R and the tunnel atom (cell) size. Namely,
>     many common specs assume larger EMTU_R and tunnel cell sizes
>     than are guaranteed by RFC1122 and RFC2460. That means that
>     these tunnel specs are relying on operational assurances of
>     larger values than the IPv4 and IPv6 minimums. Since they are
>     already published as standards, this document needs to explain
>     how they are able to operate without fragmentation.
I'll see what I can do to add that point; I'm hoping to not need to cite
dozens of examples, though.

> 17) Section 4.2.3 cites RFC4821, but PLPMTUD cannot be used by
>     tunnels due to ECMP.
I disagree; it can, but the system needs to either take the MIN or have
a way to decouple discovered PMTUs in way that can be trusted to
reasonably correspond to the ECMP splitting.

> 20) Section 4.3.3, fourth paragraph, "A multipoint tunnel MUST
>     have support for broadcast and multicast" - I think this
>     would be better as a "SHOULD". RFC2529 and AERO support
>     multicast, but RFC5214 does not yet it is widely deployed.

Multicast or its equivalent. Otherwise, you can't support IPv6
multicast, which is a required capability of IPv6.
> 23) Section 5.1, first sub-bullet under "Tunnels must obey core IP
>     requirements", Are you meaning to talk about IPv4 DF=1?

Yes, and that should be made more explicit. Also honoring the EMTU_R
limits until told otherwise.

-----