Hi Dave,
Sorry for a late reply addressing your comments. Please, see inline.
Thanks,
Nabil
From: "Black, David"
<david(_dot_)black(_at_)emc(_dot_)com<mailto:david(_dot_)black(_at_)emc(_dot_)com>>
Date: Monday, August 20, 2012 8:57 PM
To: "dinmohan(_at_)hotmail(_dot_)com<mailto:dinmohan(_at_)hotmail(_dot_)com>"
<dinmohan(_at_)hotmail(_dot_)com<mailto:dinmohan(_at_)hotmail(_dot_)com>>,
"Bitar, Nabil N"
<nabil(_dot_)n(_dot_)bitar(_at_)one(_dot_)verizon(_dot_)com<mailto:nabil(_dot_)n(_dot_)bitar(_at_)one(_dot_)verizon(_dot_)com>>,
Sagessi <sajassi(_at_)cisco(_dot_)com<mailto:sajassi(_at_)cisco(_dot_)com>>,
"gen-art(_at_)ietf(_dot_)org<mailto:gen-art(_at_)ietf(_dot_)org>"
<gen-art(_at_)ietf(_dot_)org<mailto:gen-art(_at_)ietf(_dot_)org>>
Cc: "Black, David"
<david(_dot_)black(_at_)emc(_dot_)com<mailto:david(_dot_)black(_at_)emc(_dot_)com>>,
"pwe3(_at_)ietf(_dot_)org<mailto:pwe3(_at_)ietf(_dot_)org>"
<pwe3(_at_)ietf(_dot_)org<mailto:pwe3(_at_)ietf(_dot_)org>>,
"ietf(_at_)ietf(_dot_)org<mailto:ietf(_at_)ietf(_dot_)org>"
<ietf(_at_)ietf(_dot_)org<mailto:ietf(_at_)ietf(_dot_)org>>
Subject: Gen-ART review of draft-ietf-pwe3-mpls-eth-oam-iwk-06
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>.
Please resolve these comments along with any other Last Call comments you may
receive.
Document: draft-ietf-pwe3-mpls-eth-oam-iwk-06
Reviewer: David L. Black
Review Date: August 20, 2012
IETF LC End Date: August 20, 2012
Summary:
This draft is basically ready for publication, but has nits that
should be fixed before publication.
<NB> Thanks.
This draft covers defect behavior for Ethernet pseudowires,
including defect state mapping and PE defect reporting behavior.
The draft is generally in good shape. I found a few minor nits.
1) The draft uses a lot of acronyms - while each acronym appears to
be expanded on first use, an additional section near the start of the
draft listing all of them would be helpful.
<NB> Done.
2) There's a typo in the first paragraph of section 2:
covers the following Ethernet OAM (Opertaions, Administration and
Opertaions -> Operations.
<NB> Thanks. Done.
3) The following normative reference is incomplete - please add additional
information that will enable a reader to locate the referenced document:
[MEF16] "Ethernet Local Management Interface", MEF16, January 2006.
[MEF16] "Ethernet Local Management Interface", Metro Ethernet Forum Technical
Specification MEF16, January 2006.
<NB> changed it to :
4) idnits 2.12.13 did not like the pagination:
== The page length should not exceed 58 lines per page, but there was 22
longer pages, the longest (page 1) being 61 lines
<NB> That will be fixed.
Thanks,
--David
----------------------------------------------------
David L. Black, Distinguished Engineer
EMC Corporation, 176 South St., Hopkinton, MA 01748
+1 (508) 293-7953 FAX: +1 (508) 293-7786
david(_dot_)black(_at_)emc(_dot_)com<mailto:david(_dot_)black(_at_)emc(_dot_)com>
Mobile: +1 (978) 394-7754
----------------------------------------------------