[Gen-art] Gen-ART review of draft-ietf-softwire-4rd-08.txt

Christer Holmberg <christer.holmberg@ericsson.com> Mon, 06 October 2014 10:40 UTC

Return-Path: <christer.holmberg@ericsson.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id CF8051A1B85 for <gen-art@ietfa.amsl.com>; Mon, 6 Oct 2014 03:40:41 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.201
X-Spam-Level:
X-Spam-Status: No, score=-4.201 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001] autolearn=ham
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 kv8dKR7KTOWb for <gen-art@ietfa.amsl.com>; Mon, 6 Oct 2014 03:40:36 -0700 (PDT)
Received: from sesbmg23.ericsson.net (sesbmg23.ericsson.net [193.180.251.37]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 4466F1A1BAA for <gen-art@ietf.org>; Mon, 6 Oct 2014 03:40:35 -0700 (PDT)
X-AuditID: c1b4fb25-f791c6d00000617b-6b-543271a1ce40
Received: from ESESSHC003.ericsson.se (Unknown_Domain [153.88.253.124]) by sesbmg23.ericsson.net (Symantec Mail Security) with SMTP id 39.72.24955.1A172345; Mon, 6 Oct 2014 12:40:33 +0200 (CEST)
Received: from ESESSMB209.ericsson.se ([169.254.9.136]) by ESESSHC003.ericsson.se ([153.88.183.27]) with mapi id 14.03.0174.001; Mon, 6 Oct 2014 12:40:33 +0200
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: "gen-art@ietf.org" <gen-art@ietf.org>
Thread-Topic: Gen-ART review of draft-ietf-softwire-4rd-08.txt
Thread-Index: Ac/hTnVUKr9PcjLzTpiKE/sbK/Pltg==
Date: Mon, 06 Oct 2014 10:40:32 +0000
Message-ID: <7594FB04B1934943A5C02806D1A2204B1D46991F@ESESSMB209.ericsson.se>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [153.88.183.147]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFrrILMWRmVeSWpSXmKPExsUyM+Jvje7CQqMQg6ff9CyWzNnNYnH11WcW ByaPJUt+Mnl8ufyZLYApissmJTUnsyy1SN8ugSvj6GrLgj7xii1TbrI1ML4R7GLk5JAQMJH4 0juHBcIWk7hwbz1bFyMXh5DAUUaJM5f3sIIkhAQWM0rcf6baxcjBwSZgIdH9TxskLCKgKTF3 xVsmEJtZIF7i8Y7pjCAlwgKWEn0z3CBK7CSO/3zMAmHrSfR0r2MGsVkEVCTWfN0N1sor4Cux 9P9DsE2MQCd8P7UGaqS4xK0n85kgThOQWLLnPDOELSrx8vE/VpBVEgJKEtO2pkGU60gs2P2J DcLWlli28DUzxHhBiZMzn7BMYBSZhWTqLCQts5C0zELSsoCRZRWjaHFqcVJuupGxXmpRZnJx cX6eXl5qySZGYBwc3PJbdQfj5TeOhxgFOBiVeHgT9huGCLEmlhVX5h5ilOZgURLnXXhuXrCQ QHpiSWp2ampBalF8UWlOavEhRiYOTqkGRnmeMycVZ0uVFqevLvhy3fDryQkJTsLrHotOtd46 gXXe0yWNkU1/F/1kfLA1Q67wI8/rA9lbaxdzxHzf8DqjUvDM72c3mPO1Mh+9dvp936ZE/57d j5sXWsu712tbPNog6Hp08melRme9w+qeXTNrt8R2zDSOtw/b3i20MTj4s/v1k50tkze6FSqx FGckGmoxFxUnAgAdOFzdZAIAAA==
Archived-At: http://mailarchive.ietf.org/arch/msg/gen-art/A5wNwAbkEfJl5s7tQYFB6O61gdk
Cc: "draft-ietf-softwire-4rd.all@tools.ietf.org" <draft-ietf-softwire-4rd.all@tools.ietf.org>
Subject: [Gen-art] Gen-ART review of draft-ietf-softwire-4rd-08.txt
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 06 Oct 2014 10:40:42 -0000

I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>

Document:		draft-ietf-softwire-4rd-08.txt

Reviewer:                 	Christer Holmberg

Review Date:            	6 October 2014

IETF LC End Date:       	10 October 2014

IETF Telechat Date:      	16 October 2014

Summary:                         The document is well written, and almost ready for publication, but there are some editorial nits that I ask the authors to address before publishing.

Major Issues: None

Minor Issues: None

Editorial nits: None


QGEN_1:

In a number of places in the document you talk about "mesh topology" and "Hub&Spoke topology". Are those considered commonly known, or would it be useful to have a reference?


QABS_1:

The Abstract needs to be re-formulated. It seems to describe a problem, but does not really say anything about the scope of the document. Normally, after the problem statement, there would be a sentence starting with "This document defines blah blah blah...".


Q_1_1:

The first sentence says "For deployments of residual IPv4 service via IPv6 networks,". Is there a document defining "residual IPv4 service via IPv6 networks" which you could reference?

Q_1_2:

I would suggest to split the first paragraph into smaller paragraphs. Something like (note some minor editorial changes):

	"For deployments of residual IPv4 service via IPv6 networks, the need
   	for a stateless solution, i.e. one where no per-customer state is
   	needed in IPv4-IPv6 gateway nodes of the provider, is expressed in
   	[I-D.ietf-softwire-stateless-4v6-motivation]. This document specifies such a 
	solution, named "4rd" for IPv4 Residual Deployment.
   	
	Using the solution, IPv4 packets are transparently tunneled across IPv6 networks
  	 (reverse of 6rd [RFC5969] in which IPv6 packets are statelessly
   	tunneled across IPv4 networks). 

	While IPv6 headers are too long to be mapped into IPv4 headers (why 6rd requires 
	encapsulation of full IPv6 packets in IPv4 packets), IPv4 headers can be reversibly
   	translated into IPv6 headers in such a way that, during IPv6 domain
   	traversal, UDP packets having checksums and TCP packets are valid
   	IPv6 packets.  IPv6-only middle boxes that perform deep-packet-
   	inspection can operate on them, in particular for port inspection and
  	web caches."


Q4_1:

In section 4, the text lists a number of functions that a 4rd CE and a 4rd BR SHOULD follow.

However, e.g. in R-2 the text says:

	"CEs and BRs MUST be configured with the following Domain parameters:"

So, is R-2 a "MUST", or a "SHOULD"?

Perhaps you in section 4 should only list the functions, and for each function you then say whether it is SHOULD, MUST, or something else.


Regards,

Christer