Re: [trill] Tsvart early review of draft-ietf-trill-over-ip-10

Magnus Westerlund <magnus.westerlund@ericsson.com> Fri, 02 February 2018 09:29 UTC

Return-Path: <magnus.westerlund@ericsson.com>
X-Original-To: trill@ietfa.amsl.com
Delivered-To: trill@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id E887312FA98 for <trill@ietfa.amsl.com>; Fri, 2 Feb 2018 01:29:42 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.32
X-Spam-Level:
X-Spam-Status: No, score=-4.32 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-0.001] autolearn=unavailable autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=ericsson.com
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 mSaTwU5SIAgA for <trill@ietfa.amsl.com>; Fri, 2 Feb 2018 01:29:40 -0800 (PST)
Received: from sessmg23.ericsson.net (sessmg23.ericsson.net [193.180.251.45]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 0085312FA90 for <trill@ietf.org>; Fri, 2 Feb 2018 01:29:39 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; d=ericsson.com; s=mailgw201801; c=relaxed/simple; q=dns/txt; i=@ericsson.com; t=1517563777; h=From:Sender:Reply-To:Subject:Date:Message-ID:To:CC:MIME-Version:Content-Type: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=47iTi6WKK92rqudn0L6an8pXOSS0DpgfhXTbx8EToIg=; b=HvVd7eRFRHtMaWjX0A4F27YjLX2UKf23DsNvfVnZQbP53oY53hSHTQUVvtHLYA9d TCRA59xxHg5c9BltOXV5bIZOusLGZ7r3ofRsvUmADnoo1/Qq6jPnfzyUqRyESviS MXiyTL5DxjAoAc/D0ZsJx/a7DwmH0kbxYveFV2GabS8=;
X-AuditID: c1b4fb2d-b35ff70000007932-d6-5a742f817563
Received: from ESESSHC020.ericsson.se (Unknown_Domain [153.88.183.78]) by sessmg23.ericsson.net (Symantec Mail Security) with SMTP id 15.46.31026.18F247A5; Fri, 2 Feb 2018 10:29:37 +0100 (CET)
Received: from [147.214.160.85] (153.88.183.153) by smtps.internal.ericsson.com (153.88.183.78) with Microsoft SMTP Server (TLS) id 14.3.352.0; Fri, 2 Feb 2018 10:29:36 +0100
From: Magnus Westerlund <magnus.westerlund@ericsson.com>
To: Donald Eastlake <d3e3e3@gmail.com>
CC: "tsv-art@ietf.org" <tsv-art@ietf.org>, "trill@ietf.org" <trill@ietf.org>, "draft-ietf-trill-over-ip.all@ietf.org" <draft-ietf-trill-over-ip.all@ietf.org>
References: <149754795560.13109.17521244075940607817@ietfa.amsl.com> <CAF4+nEG-28weDot9R9Z4-05PX1tzBoKZSOHu8BJY2GiRzOv0nA@mail.gmail.com> <52E4A8FC978E0241AE652516E24CAF0029AC2251@ESESSMB103.ericsson.se> <CAF4+nEEhaY+gtyjhVN1uzwgJ8m5oy1VU3urdH_hh-2KYV+NXLQ@mail.gmail.com> <CAF4+nEEaJr7RwAaQx59fTvAhh0qy1NRqPx4HREvzGPRHqdx++w@mail.gmail.com>
Message-ID: <e4cb17c2-188a-4201-8803-34437e38c36b@ericsson.com>
Date: Fri, 2 Feb 2018 10:29:29 +0100
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.5.2
MIME-Version: 1.0
In-Reply-To: <CAF4+nEEaJr7RwAaQx59fTvAhh0qy1NRqPx4HREvzGPRHqdx++w@mail.gmail.com>
Content-Type: multipart/signed; protocol="application/pkcs7-signature"; micalg=sha-256; boundary="------------ms020701080501040005050005"
X-Originating-IP: [153.88.183.153]
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrJIsWRmVeSWpSXmKPExsUyM2K7n26jfkmUwYYWWYuD2zUtPn+dxm7x fvJ2NotZexaxOLB47Jx1l91jyZKfTAFMUVw2Kak5mWWpRfp2CVwZe2/OZyp4dZSxYssnzwbG +QsZuxg5OSQETCTONDSydjFycQgJHGaU2He3mQ3C2cQo8e17FztIFZuAhcTNH41ACQ4OYQF7 iSvLxEDCIgJqEq+XL2ABqWcWWMgo8Xb9MXaI5rtMElNmv2YCqeIFaji7rxtsHYuAikTfkzVg cVGBGIkFzYeYIWoEJU7OfMICYnMKBEqcevCHEWJqN6PE9/sfwJqFBLQlGpo6WCHuVpK4Pu86 ywRGgVlI+mch6wFJMAuESdx4vIgVwhaXaPqyEso2k5i3+SEzhK0tsWzhayCbA8hWk1jWqoQq DGJbS8z4dZANwlaUmNL9kB3CNpV4ffQj1CojiXd7GtkXMPKsYhQtTi0uzk03MtZLLcpMLi7O z9PLSy3ZxAiMxINbfuvuYFz92vEQowAHoxIPb7JmSZQQa2JZcWXuIUYVoDmPNqy+wCjFkpef l6okwvt1Q3GUEG9KYmVValF+fFFpTmrxIUZpDhYlcd6TnrxRQgLpiSWp2ampBalFMFkmDk6p BkZXJ1+WFKvGlkOxn/wyTlRp5z/041Rb8/4P2ze+OYoKi7Z2M0xW/Zk4R+WnvfCkewXdD+Oe JOw60uJwJjLgkNTE5MvRGbF7bixgnnqmUs+3QEtBs/PqLO8nxgInw3iZSrqOSU2+0arMvFid 79Hrb9sOGZ64m5s552LWdaY0Q72+7U7v9DfpLVJiKc5INNRiLipOBADG77OyzAIAAA==
Archived-At: <https://mailarchive.ietf.org/arch/msg/trill/6aHy49KrNlNX1GRTAK-BykOjGGg>
Subject: Re: [trill] Tsvart early review of draft-ietf-trill-over-ip-10
X-BeenThere: trill@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "Developing a hybrid router/bridge." <trill.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/trill>, <mailto:trill-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/trill/>
List-Post: <mailto:trill@ietf.org>
List-Help: <mailto:trill-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/trill>, <mailto:trill-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 02 Feb 2018 09:29:43 -0000

Hi Donald,

I have reviewed the changes and my comments. Please see inline for my 
feedback. I will remove text not necessary to provide context. It is 
almost ready now. There are two areas where I think there needs to be 
some further improvements, DSCPs and TCP encapsulation. But the needed 
changes are relatively straightforward.


Den 2018-01-31 kl. 06:25, skrev Donald Eastlake:
> Hi Magnus,
>
> It has been a while but the just posted version -12 is intended to
> resolve your comments except those related to middle boxes. (The TRILL
> WG has decided middle boxes will be out of scope for this draft.)
>
>
>
> On Thu, Jun 29, 2017 at 10:17 PM, Donald Eastlake<d3e3e3@gmail.com>  wrote:
>> On Tue, Jun 27, 2017 at 1:13 PM, Magnus Westerlund
>> <magnus.westerlund@ericsson.com>  wrote:
>> Den 2017-06-26 kl. 02:07, skrev Donald Eastlake: 
>>> On Thu, Jun 15, 2017 at 1:32 PM, Magnus Westerlund
>>> <magnus.westerlund@ericsson.com>  wrote:
>>>
>>>
>>> Diffserv usage
>>> --------------
>>>
>>> Section 4.3:

The new text from -12:

    The default TRILL priority and DEI to DSCP mapping, which may be
    configured per TRILL over IP port, is an follows. Note that the DEI
    value does not affect the default mapping and, to provide a
    potentially lower priority service than the default priority 0,
    priority 1 is considered lower priority than 0. So the priority
    sequence from lower to higher priority is 1, 0, 2, 3, 4, 5, 6, 7, as
    it is in [802.1Q].

       TRILL Priority  DEI  DSCP Field (Binary/decimal)
       --------------  ---  -----------------------------
                   0   0/1  001000 / 8
                   1   0/1  000010 / 0
                   2   0/1  010000 / 16
                   3   0/1  011000 / 24
                   4   0/1  100000 / 32
                   5   0/1  101000 / 40
                   6   0/1  110000 / 48
                   7   0/1  111000 / 56

    The above all follow the recommended DSCP values from [RFC2474]
    except for the placing of priority 1 below priority 0, as specified
    in [802.1Q], and for the DSCP value of 000010 binary for low effort
    as recommended in [LEphb].

I don't really understand this change. Now you have priority 1 which 
points to the proposed new mapping for Lower than Best Effort, being the 
least prioritized. Then priority 0 intended to be more prioritized is 
CS1 which may still be lower than best effort (thus basically same as 
prio 1), or end up being receiving a more prioritized PHB than best 
effort. Why isn't 0 mapped to best effort (000000)? That appears that it 
would provide a more consistent behaviour.





>>> MTU and Fragmentation
>>> ---------------------

With the changes introduced and the no middlebox applicability it looks 
like the issues are resolved.


>>> Zero Checksum:
>>> --------------

The IPv6 zero checksum section (5.4.2) looks good.

>>>
>>> TCP Encapsulation issue
>>> -----------------------
>>>

So the TCP encapsulation section (5.6) is significantly improved, but 
some comments:


"All packets in a particular TCP stream SHOULD use the same DSCP
codepoint or packet re-ordering may occur."

	

	
	
	

I think that SHOULD is a MUST. You get no benefit from trying to send 
different packets in a TCP connection with different DSCPs. It will 
really only result in reordering, and additional retransmissions. Also 
the TCP stacks Head of line blocking will also not really allow any 
received payload to be released early. Thus, only downsides and now gain 
to use different DSCPs for a single connection.

"It is RECOMMEDED that
    at least two TCP connections be provided, for example one for
    priority 6 and 7 traffic and one for priority 0 through 5 taffice, in
    order to insure that urgent control traffic, including control
    traffic related to establishing and maintaining adjacency, is not
    significantly delayed by lower priority traffic."

I think this is fine, there is one potential issue here, but likely not 
a significant. If the high priority traffic is very sparse, then the TCP 
connection marked with a higher priority DSCP may have a very small 
congestion window. So when the high priority traffic is to be sent it 
will in fact be delayed more by the TCP stacks congestion control than 
the lower priority that has a larger window established. However, that 
is only really an issue if the high priority traffic is multiple packets 
that comes in burst. Also, if there is on path contention the lower 
priority traffic may still not make it even if sent earlier, so not 
using a high priority DSCP is not really an option either.

I think the IP/TCP/Trill and TCP header figures are simply non-relevant 
here. I think they can give the impression that a receiving node can 
process each TCP packet individually rather than running a full TCP 
implementation here. What is relevant is the framing format that delimit 
each Trill packet sent by TCP.

+--------+-------- // ----+ | Length | TRILL packet   |
       +--------+----- // -------+

The above figure is a bit confusing, I assume the intention is to 
indicate previous Trill packet with framing and the following one. I 
think you be more explicit about that as an explanation.


          If the
          initial 2 bytes of the TRILL packet are not one of these
          Ethertypes, then the receiver assumes that framing
          synchronization has been lost and MUST close that TCP
          connection.

Yes, this is very important text. I would recommend that it is moved 
into its own paragraph rather than hidden in a figure explanation. I am 
also missing the specification what to do when one have closed the 
connection. I assume that it is to have either part re-initiate. But, is 
it the closing party (receiver) that should do this, or the sending 
part? As TCP connections are bi-directional, but I assume that these are 
used only unidirectionally, and thus it matters who should establish 
them in relation to signalling? Congestion Control

>>>
>>> Flow and ECMP
>>> -------------
>>>

Issues resolved in the more limited applicability scope.

>>>
>>> NAT and TRILL over IP:

I think it is fine with the WG saying that NATs are out of scope for 
Trill. However, I really think you need to say this upfront in the 
document that this specification relies on that there are no NAT in the 
path between the trill nodes using this specification. And be explicit 
if there would be NATs then things will not work as intended.

Cheers

Magnus Westerlund

----------------------------------------------------------------------
Media Technologies, Ericsson Research
----------------------------------------------------------------------
Ericsson AB                 | Phone  +46 10 7148287
Torshamnsgatan 23           | Mobile +46 73 0949079
SE-164 80 Stockholm, Sweden | mailto:magnus.westerlund@ericsson.com
----------------------------------------------------------------------