Re: [dtn] working group last call on draft-ietf-dtn-bpbis

"Burleigh, Scott C (312B)" <scott.c.burleigh@jpl.nasa.gov> Fri, 20 January 2017 22:54 UTC

Return-Path: <scott.c.burleigh@jpl.nasa.gov>
X-Original-To: dtn@ietfa.amsl.com
Delivered-To: dtn@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D2149129530 for <dtn@ietfa.amsl.com>; Fri, 20 Jan 2017 14:54:05 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -7.42
X-Spam-Level:
X-Spam-Status: No, score=-7.42 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-3.199, SPF_PASS=-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 gk2fyEA4M24y for <dtn@ietfa.amsl.com>; Fri, 20 Jan 2017 14:54:04 -0800 (PST)
Received: from mail.jpl.nasa.gov (smtp.jpl.nasa.gov [128.149.139.105]) (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 2C9C712952F for <dtn@ietf.org>; Fri, 20 Jan 2017 14:54:04 -0800 (PST)
Received: from mail.jpl.nasa.gov (ap-ehub-sp02.jpl.nasa.gov [128.149.137.149]) by smtp.jpl.nasa.gov (Sentrion-MTA-4.3.1/Sentrion-MTA-4.3.1) with ESMTP id v0KMs2o6020211 (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256 bits) verified NO); Fri, 20 Jan 2017 14:54:02 -0800
Received: from AP-EMBX-SP10.RES.AD.JPL ([169.254.1.111]) by ap-ehub-sp02.RES.AD.JPL ([fe80::dd85:7b07:1e36:7e3c%15]) with mapi id 14.03.0319.002; Fri, 20 Jan 2017 14:54:02 -0800
From: "Burleigh, Scott C (312B)" <scott.c.burleigh@jpl.nasa.gov>
To: "Templin, Fred L" <Fred.L.Templin@boeing.com>, dtn <dtn@ietf.org>
Thread-Topic: [dtn] working group last call on draft-ietf-dtn-bpbis
Thread-Index: AQHSaCGmzxdqqnp0pkKUzwuCATCJUKEsTCiAgBZCZID//3y/YA==
Date: Fri, 20 Jan 2017 22:54:02 +0000
Message-ID: <A5BEAD028815CB40A32A5669CF737C3B612FEED2@ap-embx-sp10.RES.AD.JPL>
References: <44B4919D-4283-4FDD-91E5-1EE5288D50AC@viagenie.ca> <E2450792-7FEE-4ECE-902E-1F7BDBF42BBD@viagenie.ca> <33dcf870891646fcb895f85f9449d0b6@XCH15-06-08.nw.nos.boeing.com>
In-Reply-To: <33dcf870891646fcb895f85f9449d0b6@XCH15-06-08.nw.nos.boeing.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [128.149.137.26]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Source-Sender: scott.c.burleigh@jpl.nasa.gov
X-AUTH: Authorized
Archived-At: <https://mailarchive.ietf.org/arch/msg/dtn/5imJlBqrSlgQeHe5oJuVwg6Gp0s>
Subject: Re: [dtn] working group last call on draft-ietf-dtn-bpbis
X-BeenThere: dtn@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: "Delay Tolerant Networking \(DTN\) discussion list at the IETF." <dtn.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/dtn>, <mailto:dtn-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/dtn/>
List-Post: <mailto:dtn@ietf.org>
List-Help: <mailto:dtn-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/dtn>, <mailto:dtn-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 20 Jan 2017 22:54:06 -0000

Fred, some responses in-line below.

Scott

-----Original Message-----
From: dtn [mailto:dtn-bounces@ietf.org] On Behalf Of Templin, Fred L
Sent: Friday, January 20, 2017 2:27 PM
To: dtn <dtn@ietf.org>
Subject: Re: [dtn] working group last call on draft-ietf-dtn-bpbis

Hi, here are my review comments for draft-ietf-dtn-bpbis:

Fred Templin
fred.l.templin@boeing.com

---

1) Abstract, instead of "This Internet Draft", say: "This document" so that the text can carry
     forward into RFC publication.

Sure.

2) Page 4, figure 1 is split between two pages. Make it fit onto a single page.

Sure.

3) Page 4, figure 1, "Trans1" and "Trans3" are unexplained. Are they the same as "T1/T3"?
     Why not just call them "T1" and "T3"?

Yep, fine with me.

4) Page 6, "Partial payload" is it the case that payload fragmentation is limited to 2-fragment
     fragmentation, or can it be N-fragment? The definition makes it appear that only 2-fragment
    fragmentation is supported.

Bundles can be split up into an unlimited number of fragments, but the way to do that is to split a single bundle into two fragments, each of which is a full-fledged bundle - and since each fragment is a bundle it can then be split into two fragments, each of which is a bundle, and so on.  The rules for dividing one bundle into two fragments can be described exactly, in a fairly concise way.  The rules for dividing a single bundle directly into N bundles would be much more complicated, harder to grasp, and easier to get wrong, and they are really not necessary.  (The development of multicellular organisms is sort of a model here.)  But certainly I can add a sentence somewhere to the effect that fragments are bundles and are therefore amenable to further fragmentation, such that the net result is fragmentation of a single bundle into N fragments, N>= 2.

5) Page 6, figure 2 is split between two pages. Make it fit onto a single page.

Sure.

6) Section 3.2, page 14, the sentence: "However, in some environments there may be
    segments of the end-to-end path for which no reliable convergence-layer protocol
    is available; in such environments the use of reliable convergence-layer protocols
    wherever possible can reduce the incidence of data loss." - this seems to be saying
    two contradictory things. It basically seems says: "when no reliable convergence-layer
    protocol is available, use a reliable convergence-layer protocol" which is an
    oxymoron - perhaps a misunderstanding?

The writing here is bad.  The idea is that even though some segments of the end-to-end path can't be served by reliable CL protocols, you're still better off overall if you use reliable CL protocols on those segments that can be.  I will fiddle with this text to make it clearer.

7) Section 4.1.5.1, page 18, should there be an informative reference for "dtn:" and "ipn:"?

All we are doing in this section is saying what to do with various string values.  There happen to be some documents that discuss the schemes identified by those strings, and we could insert some informative references to those documents if they would be helpful to the reader.

8) Section 4.2.2, page 22, "Total Application Data Unit Length" only appears if the bundle is
    a fragment. But, does it mean that only "two-fragment" fragmented bundles are possible?

No, it doesn't mean that (see above).

9) Section 4.3.4, page 25, why does it need to include both a hop limit and hop count? Isn't
    it sufficient to just have a hop limit that decrements to 0?

A fair point, but having both fields actually does provide additional information - if you only decrement hop limit to zero you have no idea what value it started out with, which might be important for some routers.  That said, I'm fine either way.

10) Section 5.8, page 32, why is only "2-fragment" fragmentation supported? Why not
    "N-fragment"?

As discussed above, a bundle can be divided into any number of fragments.

11) Section 6.1.2, figure 6, page 42, change: "No known route destination from here."
    to: "No known route to destination from here."

Yes.

_______________________________________________
dtn mailing list
dtn@ietf.org
https://www.ietf.org/mailman/listinfo/dtn