[apps-discuss] AppsDir reivew of draft-ietf-6man-impatient-nud

["Murray S. Kucherawy" <[e-mail]>]

I have been selected as the Applications Area Directorate (appsdir) reviewer
for this draft.  (For background on appsdir, please see
http://trac.tools.ietf.org/area/app/trac/wiki/ApplicationsAreaDirectorate).

Please resolve these comments along with any other Last Call comments you may
receive.  Please wait for direction from your document shepherd or AD
before posting a new version of the draft.

Document: draft-ietf-6man-impatient-nud
Title: Neighbor Unreachability Detection is too impatient
Reviewer: Murray S. Kucherawy
Review Date: May 8, 2013
IETF Last Call Date: ends May 9, 2013
IESG Telechat Date: May 9, 2013

Summary: This document is almost ready for publication on the Standards
Track.  The technology described in it seems sound overall.  However, the
document is in some need of an editorial pass as I found myself tripping
over several expressions and such.  So, rather than providing a technical
review (which doesn't seem to be needed), my comments below are entirely
stylistic or grammatical in the hopes that you'll get to AUTH48 a little
faster.

MAJOR ISSUES:
None.


MINOR ISSUES:
These are all editorial concerns that caused me to stumble while reading.
I think they're more important to fix, to help future readers, than the ones
I listed under "NITS".

Section 1, Introduction:

"These short can be appropriate" should be "This short timeout can be
appropriate" or some such.

The "For example... " sentence is a fragment.  Suggest adding to the end:
", then a short timeout functions well."

"alternative neighbor;" should be "alternative neighbor,"

The second and third paragraphs don't need to be separate.  I suggest merging
them.

Add a comma right after "As a result" and "For IPv4".

Section 3, Protocol Updates:

For the expression MAX_UNICAST_SOLICIT/MAX_MULTICAST_SOLICIT, is that "or" or
"division"?  I suggest not using the slash but rather using text to say what
you mean there.

Change "it SHOULD" to "they SHOULD" since you're talking about a plural
("implementations").

Fourth paragraph, add a comma after "above".

I suggest a section reference for the conceptual model mentioned in paragraph
five, not just a document reference.

Change "But in the UNREACHABLE state" to "However, in the UNREACHABLE state,"

In the sixth paragraph you talk about "we" for the first time.  The style
should be consistent, so use "implementations" instead of "we".

In paragraph 7, add a comma after "Thus".

In paragraph 10, change the second sentence to:

	UNREACHABLE is added to the current list of STALE, PROBE, or
	DELAY.

Change "That needs to be replaced by" to "The following table replaces the
State table at Section [xxx] of RFC4861." and remove the original table.

Change "clamped at" to "limited to".

Change "then it makes sense to stop" to "then implementations MAY discontinue".

In the last paragraph, please be more specific than "sooner or later".

Section 4, Example Algorithm:

Change "that allows it" to "that allow it" (you're talking about multiple
configuration values this time).

In the second paragraph, please express the equation using a figure or other
illustration rather than making it part of the prose.

Third paragraph, add a comma after "transmissions".
of course.  Same in the following two paragraphs.

Add a comma after "With the above recommended values".

Add a comma after "In the language of the state machine in [RFC4861]", and
another after "Thus".

In "in a link-layer address i.e., the case when", add a comma after "address".

Change "you would get the same behavior as in [RFC4861]" to "the same
behavior as in [RFC4861] would be produced".

Use a figure or illustration for the equation in the last paragraph as well.

Section 7, IANA Considerations:

Include an RFC Editor note to remove the section prior to publication.


NITS: 
These are all minor things, pet peeves and the like.  The RFC Editor will
probably make them too, but I won't complain if you don't fix them.  :-)

Title should be "Neighbor Unreachability Detection Is Too Impatient" (note
capitalization).

In the abstract, "3" should be "three".  Also, "whether or not" should be just
"whether".

In paragraph four of Section 1, s/which/that/.

In paragraph five of Section 1, s/it means that/it means/

In Section 3, change "A node MAY garbage collect" to "A node MAY perform
garbage collection".

In Section 4, first sentence, s/which/that/.

Also in Section 4, in the fourth paragraph: For numbers lower than ten,
use the English word rather than the numeral (e.g. "five attempts",
"one second", "three seconds", etc.).  You don't need to do this for your
configuration settings, however.