Re: [Lwip] [Fwd: New Version Notification for draft-gomez-lwig-tcp-constrained-node-networks-01.txt]

"Scharf, Michael (Nokia - DE)" <michael.scharf@nokia.com> Tue, 14 March 2017 23:01 UTC

Return-Path: <michael.scharf@nokia.com>
X-Original-To: lwip@ietfa.amsl.com
Delivered-To: lwip@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 186BA129B8D; Tue, 14 Mar 2017 16:01:08 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.798
X-Spam-Level:
X-Spam-Status: No, score=-2.798 tagged_above=-999 required=5 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-2.796, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=nokia.onmicrosoft.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 WmYO8s6A1qu2; Tue, 14 Mar 2017 16:01:07 -0700 (PDT)
Received: from EUR02-VE1-obe.outbound.protection.outlook.com (mail-eopbgr20121.outbound.protection.outlook.com [40.107.2.121]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 97AAB129440; Tue, 14 Mar 2017 16:01:06 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nokia.onmicrosoft.com; s=selector1-nokia-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=81k3soxzrh5gBEd2YMRIW5WJE+Sj4K/rRF9OcBYANz4=; b=SJarKUOH2vQANGbceshLIo4utdqJj6ySNU0IJknoZSvqZ4UPJXKQUeau6LLPjLZdtatQiICuTHxAqfqE2ccogAYioat081+sRgqq5B42i80g7OLSuupfpew3cTl8qJDBc7wpFQ8FIjoC8Y16Vl4M//wS/zEkqSUlsS6WnwlmWtw=
Received: from AM5PR0701MB2547.eurprd07.prod.outlook.com (10.173.92.15) by AM5PR0701MB2548.eurprd07.prod.outlook.com (10.173.92.16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.977.5; Tue, 14 Mar 2017 23:01:04 +0000
Received: from AM5PR0701MB2547.eurprd07.prod.outlook.com ([10.173.92.15]) by AM5PR0701MB2547.eurprd07.prod.outlook.com ([10.173.92.15]) with mapi id 15.01.0977.010; Tue, 14 Mar 2017 23:01:04 +0000
From: "Scharf, Michael (Nokia - DE)" <michael.scharf@nokia.com>
To: Carles Gomez Montenegro <carlesgo@entel.upc.edu>
CC: "lwip@ietf.org" <lwip@ietf.org>, "tcpm@ietf.org Extensions" <tcpm@ietf.org>, "jon.crowcroft@cl.cam.ac.uk" <jon.crowcroft@cl.cam.ac.uk>
Thread-Topic: [Lwip] [Fwd: New Version Notification for draft-gomez-lwig-tcp-constrained-node-networks-01.txt]
Thread-Index: AQHSM3N8XbK/bgtr5USLkNUwp0et86DPv4NwgMBbiwCABaUBhg==
Date: Tue, 14 Mar 2017 23:01:04 +0000
Message-ID: <AM5PR0701MB2547E2DF7DC2E987D5E6B23593240@AM5PR0701MB2547.eurprd07.prod.outlook.com>
References: <563cd49c505340a559cded05282a3dc6.squirrel@webmail.entel.upc.edu> <655C07320163294895BBADA28372AF5D48B68E11@FR712WXCHMBA15.zeu.alcatel-lucent.com>, <31c0efd99c203beeb3115f5900341326.squirrel@webmail.entel.upc.edu>
In-Reply-To: <31c0efd99c203beeb3115f5900341326.squirrel@webmail.entel.upc.edu>
Accept-Language: en-US, de-DE
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: ietf.org; dkim=none (message not signed) header.d=none;ietf.org; dmarc=none action=none header.from=nokia.com;
x-originating-ip: [25.160.207.132]
x-microsoft-exchange-diagnostics: 1; AM5PR0701MB2548; 7:2kBfVvhBpV5tOIqz9j7EjBGpKYDpfsRpwT18TkD3t+ZKiQir4rbWuK1JTFVIGfJbemB7MQnvDYCMbkoj4G/fqFLeYGJoCaig7JYuz18qicWQ3+NKQw2tGlG937RLevAe7tQ+QLGxJEMsuhDGVyHtFIvUl6qtA6DfAz5OOosGIz6DA3zl/21bMaGMLrmHDlX/A9wK8r2hVMIbmtIz1lb7OFsYNXIXpjxm7JFa5ceVdB6nOKlcHAT9aP5dDzyUF9OLBH4UAvJqrbjrGWOZXSx4Jnv/tn1/5pxBOAE1+FIAhg8Ev62Sd2Pgfb4W/U44GbI8jT1KuY+CrhSX0GOEGxzVnA==
x-ms-office365-filtering-correlation-id: 3a6e2803-8cb5-4776-9ea0-08d46b2dfddc
x-ms-office365-filtering-ht: Tenant
x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(48565401081); SRVR:AM5PR0701MB2548;
x-microsoft-antispam-prvs: <AM5PR0701MB25485B38AECCAD24BD42A14D93240@AM5PR0701MB2548.eurprd07.prod.outlook.com>
x-exchange-antispam-report-test: UriScan:;
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(6040375)(601004)(2401047)(5005006)(8121501046)(3002001)(10201501046)(6055026)(6041248)(20161123558025)(20161123562025)(20161123555025)(20161123564025)(20161123560025)(6072148); SRVR:AM5PR0701MB2548; BCL:0; PCL:0; RULEID:; SRVR:AM5PR0701MB2548;
x-forefront-prvs: 02462830BE
x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(6009001)(39450400003)(39410400002)(39840400002)(39860400002)(39850400002)(86362001)(122556002)(33656002)(2900100001)(9686003)(77096006)(229853002)(54906002)(99286003)(55016002)(6436002)(6506006)(8676002)(25786008)(81166006)(2171002)(38730400002)(110136004)(6246003)(53936002)(15650500001)(3660700001)(3280700002)(4326008)(189998001)(2906002)(230783001)(3846002)(102836003)(6116002)(2950100002)(6916009)(305945005)(8936002)(74316002)(7736002)(5660300001)(66066001)(7696004)(54356999)(76176999)(50986999); DIR:OUT; SFP:1102; SCL:1; SRVR:AM5PR0701MB2548; H:AM5PR0701MB2547.eurprd07.prod.outlook.com; FPR:; SPF:None; MLV:sfv; LANG:en;
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nokia.com
X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Mar 2017 23:01:04.1165 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5d471751-9675-428d-917b-70f44f9630b0
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM5PR0701MB2548
Archived-At: <https://mailarchive.ietf.org/arch/msg/lwip/z5T-rKLzFSAUKTsc8TX-FYwLmPA>
Subject: Re: [Lwip] [Fwd: New Version Notification for draft-gomez-lwig-tcp-constrained-node-networks-01.txt]
X-BeenThere: lwip@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Lightweight IP stack <lwip.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/lwip>, <mailto:lwip-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/lwip/>
List-Post: <mailto:lwip@ietf.org>
List-Help: <mailto:lwip-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/lwip>, <mailto:lwip-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 14 Mar 2017 23:01:08 -0000

Hi Carles,

Thanks a lot for the update. The version -02 indeed reads better from my point of view. Also making this document informational is IMHO appropriate.

I still have one comment on Section 4.7.  "TCP options":

   A TCP implementation needs to support options 0, 1 and 2 [RFC793].  A
   TCP implementation for a constrained device that uses a single-MSS
   TCP receive or transmit window size may not benefit from supporting
   the following TCP options: Window scale [RFC1323], TCP Timestamps
   [RFC1323], Selective Acknowledgements (SACK) and SACK-Permitted
   [RFC2018].  Other TCP options should not be used, in keeping with the
   principle of lightweight operation.

   Other TCP options should not be supported by a constrained device, in
   keeping with the principle of lightweight implementation and
   operation.

I think the last two sentences are still not perfect even if the RFC 2119 language is now removed. This sort of statement can easily get outdated, since there continues to be innovation on TCP extensions and they typically use options. I think a more future-proof wording would be something of the form

  "At the time of publication, lightweight TCP implementations on constrained devices do not have to use other TCP options."

As a side note, the inconsistency regarding the TFO option still exist in this wording. And the last two sentences also have quite a bit of repetition. In addition, it is not clear to me whether the different wording ("use" vs. "support") in the last two sentences is intentional. For some TCP options, it is up to the active opener to decide whether to send the option, and it could refrain from doing so by default even if the option is actually "supported" (i.e., implemented in the stack). This may be a reasonable TCP stack configuration in some deployment scenarios, e.g., if window scale is not needed for sensor data but would help for firmware updates.

Thanks

Michael