[Int-dir] INTDIR Review of draft-ietf-nvo3-bfd-geneve-12

[Donald Eastlake <[e-mail]>]

I am an assigned INT directorate reviewer for
<draft-ietf-nvo3-geneve-12.txt>. These comments were written primarily
for the benefit of the Internet Area Directors. Document editors and
shepherd(s) should treat these comments just like they would treat
comments from any other IETF contributors and resolve them along with
any other Last Call comments that have been received. For more details
on the INT Directorate, see
https://datatracker.ietf.org/group/intdir/about/
<https://datatracker.ietf.org/group/intdir/about/>.

Based on my review, if I was on the IESG I would ballot this document
as DISCUSS. I have the following DISCUSS/ABSTAIN level issues:

- I do not understand the second half of the last paragraph of Section
1. It says: "BFD for Geneve MUST be used within a TMCE unless BFD is
congestion controlled." But then seems to specify that it be
congestion controlled inside a TMCE. Would it be simpler to say that
BFD for Geneve must always be congestion controlled, if that is what
is intended?

- The wording in Section 4.1 first paragraph seems confusing and
incomplete. (I believe this has been covered in other reviews.)

- In the first paragraph of Section 6: How can it be that both "Geneve
provides security" and "Geneve does not have any inherent security
mechanisms" ?

The following are other issues I found with this document that SHOULD
be corrected before publication:

- In section 4, the Inner Ethernet Header MAC addresses are in the
wrong order. The Destination MAC comes first, followed by the Source
MAC in an Ethernet header, the opposite of IP.

The following are minor issues (typos, misspelling, minor text
improvements) with the document:

- Given the prominence of "tunnels" in the one sentence abstract, I
think it would be good to use that word in the first paragraph of the
Introduction. Possibly: "... an overlay network of tunnels by
decoupling ..."

- Section 1, last line of first paragraph on page 3: payload -> payloads

- Section 4.1, first paragraph: "Protocol Type" -> "Ethertype"

- Section 5, last line: that -> when

- Section 6, "not low" -> "high"

Thanks,
Donald
===============================
 Donald E. Eastlake 3rd   [phone] (cell)
 2386 Panoramic Circle, Apopka, FL 32703 USA
 [e-mail]