Re: [tcpm] draft-gont-tcpm-tcp-seq-validation

Rui Paulo <rpaulo@apple.com> Wed, 11 September 2013 21:43 UTC

Return-Path: <rpaulo@apple.com>
X-Original-To: tcpm@ietfa.amsl.com
Delivered-To: tcpm@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id A510521E80A5 for <tcpm@ietfa.amsl.com>; Wed, 11 Sep 2013 14:43:24 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -10.599
X-Spam-Level:
X-Spam-Status: No, score=-10.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, RCVD_IN_DNSWL_HI=-8]
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 AMXsF-QeCMuJ for <tcpm@ietfa.amsl.com>; Wed, 11 Sep 2013 14:43:18 -0700 (PDT)
Received: from mail-out.apple.com (bramley.apple.com [17.151.62.49]) by ietfa.amsl.com (Postfix) with ESMTP id 0C3C021E8050 for <tcpm@ietf.org>; Wed, 11 Sep 2013 14:43:18 -0700 (PDT)
MIME-version: 1.0
Content-type: text/plain; charset="windows-1252"
Received: from relay3.apple.com ([17.128.113.83]) by mail-out.apple.com (Oracle Communications Messaging Server 7u4-23.01 (7.0.4.23.0) 64bit (built Aug 10 2011)) with ESMTP id <0MSZ00HM8DMQG222@mail-out.apple.com> for tcpm@ietf.org; Wed, 11 Sep 2013 14:43:12 -0700 (PDT)
X-AuditID: 11807153-b7fa56d000007d7a-c5-5230e3f0ea90
Received: from sesame.apple.com (sesame.apple.com [17.128.115.128]) (using TLS with cipher RC4-MD5 (128/128 bits)) (Client did not present a certificate) by relay3.apple.com (Apple SCV relay) with SMTP id 01.F2.32122.0F3E0325; Wed, 11 Sep 2013 14:43:12 -0700 (PDT)
Received: from rui-macbook-pro.apple.com (rui-macbook-pro.apple.com [17.193.13.39]) by sesame.apple.com (Oracle Communications Messaging Server 7u4-24.01 (7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTPSA id <0MSZ00BUMDO03B60@sesame.apple.com> for tcpm@ietf.org; Wed, 11 Sep 2013 14:43:12 -0700 (PDT)
From: Rui Paulo <rpaulo@apple.com>
In-reply-to: <5230DAD2.2090506@gont.com.ar>
Date: Wed, 11 Sep 2013 14:41:41 -0700
Content-transfer-encoding: quoted-printable
Message-id: <608E91E8-7F47-477E-A7FA-6EC192FEC145@apple.com>
References: <CFDFBEFD-5C52-4DD2-B87F-ED94A0495CFE@apple.com> <5230DAD2.2090506@gont.com.ar>
To: Fernando Gont <fernando@gont.com.ar>
X-Mailer: Apple Mail (2.1809)
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrCLMWRmVeSWpSXmKPExsUi2FDcoPvhsUGQwfK3lhbbTs5ncmD0WLLk J1MAYxSXTUpqTmZZapG+XQJXxo5d25gLLnNVbDl+gLmBcSVHFyMnh4SAicSbl58ZIWwxiQv3 1rN1MXJxCAn0MUmcmDuREcLZwSRx7NpvoAwHB7OAnsT9i1ogDbxA5uPGqawgtrCAmcTxY/OY QWw2ASWJZ30n2EFsTgFticam/0wgNouAqsSj511gcWaBeok9b/8yQdjaEk/eXWCFmGkjceD1 e7A5QgKxEtv7m8FsEQENiXkLN7NCHCorsWP/EqYJjAKzEC6aheSiWUimLmBkXsUoUJSak1hp rJdYUJCTqpecn7uJERx2hcE7GP8sszrEKMDBqMTD2zHLIEiINbGsuDL3EKMEB7OSCG/7BKAQ b0piZVVqUX58UWlOavEhRmkOFiVx3rJrQCmB9MSS1OzU1ILUIpgsEwenVAPjZKXK0NPrnz/4 7rH6tIN9/7+L58S/mOrr57pOijfYzHZA/esMOdbD7dvrZ/XunHnW2dr34Ee7EN2Lt7UForYG 7oxhe8PqqdK6Zdf3Gj+3CRIhn+OyPzUpTn4aeG72tqP3O6M3J/cZSXCtEljAddEpeJ9XgEaa fMS2mcmOZrItKzTny57lvDpHiaU4I9FQi7moOBEAmwAvojcCAAA=
Cc: tcpm-chairs@tools.ietf.org, Fernando Gont <fgont@si6networks.com>, tcpm@ietf.org, David Borman <David.Borman@quantum.com>
Subject: Re: [tcpm] draft-gont-tcpm-tcp-seq-validation
X-BeenThere: tcpm@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: TCP Maintenance and Minor Extensions Working Group <tcpm.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tcpm>, <mailto:tcpm-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/tcpm>
List-Post: <mailto:tcpm@ietf.org>
List-Help: <mailto:tcpm-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tcpm>, <mailto:tcpm-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 11 Sep 2013 21:43:25 -0000

On 11 Sep 2013, at 14:04, Fernando Gont <fernando@gont.com.ar> wrote:

> Hi, Rui,
> 
> Thanks so much for the review! Please find my comments inline...
> 
> On 09/11/2013 02:08 PM, Rui Paulo wrote:
>> * Section 3.4 says " (Failed) Simultaneous Connection Termination" in
>> the TCP time sequence ASCII
> 
> mm.. yes.. anything I'm missing?

It should say "Simultaneous Window Probes”.


>> * Section 4.1.  TCP sequence number validation -> text is repeated
> 
> Yes. I guess the question here is what style we should use to do the update.
> 
> In other documents I authored in the past, I was asked to quote the text
> to be replaced, and then the text it should be replaced with -- that's
> why you see "duplicate" text.
> 
> Two possible improvements are:
> 
> * Indent the quoted text (although the indentation will mean that the
> quoted text won't look the same as in RFC793, because the length of each
> line would increase as a result of indentation, and hence they'd have to
> be rewrapped), or,
> 
> * Try to quote only the specific sentences and tables to be replaced --
> although this would make the text/spec harder to read.

OK, I understand it now. How about underlining the differences somehow or putting “=>” marks next to the areas that changed?

--
Rui Paulo