RE: [RPRWG] Opening of CRG Comment Period on D3.0a - Leon's comments - Update
Attached are the editorial fixes (not all covered by comments) of the OAM
clause.
Leon
Clause 12
- Page 408, line 2: Double full stop after the word "network"
- Page 409, line 34: sesThreshold is bold and all other variables aren't
- Page 409, line 34: Missing period before "to declare"
- Page 412, line 49: according with comments 525 and 526 (they refer other
places but similar issue) replace "The contents and number of bytes in this
field.." with: The contents in this field..."
- Figure 12.6: Something wrong with the fonts
- Page 416, line 27: Replace "Table 6.1. (See.." with: "Table 6.1 (see.."
- Page 419, line 10: according with comment 556 (refers to the previous line
but similar issue) replace: "previously" with: "all previously"
- Replace Table 12.6 with DVJ write up. Delete 12.4.1.6 and add all
necessary variables, definitions and routines to 12.4.5.
- Page 409, line 36 and page 415 line 24: Replace "Annex L" with: "Annex K"
John's comments
Clause 12
p407: Change "Provide mechanism" to "Provide a mechanism".
p408: Change "network.." to "network.".
p409: Change "Q_OAM_REQ" to correct font size.
p409: Add Q_RX_ECHO_REQ, Q_RX_ECHO_RSP, Q_RX_FLUSH, Q_RX_ORG, and Q_TX_RS.
p409: Remove bold from "sesThreshold:" and remove ":".
p409: Change "(see 12.6.1)to" to "(see 12.6.1) to".
p410: RingletSelect() is no longer used. Remove it.
p411: Change "will be encoded" to "is encoded". (I apologize for a bad
copy/paste on the original comment that asked for this with the wrong
wording.)
p412: Change all instances of "8" in Figures 12.5, 12.6, and 12.7 to "4".
p414: Change "RingletSelection()" to "DefaultRinglet()".
p416: Change ".)." to ".)".
p417: Add Q_RX_ECHO_REQ and Q_TX_RS.
p417: Add Dequeue() and Enqueue().
p417: RingletSelect() is no longer used. Remove it.
p418: Change "Q_OAM_ECHO_REQ" to "Q_RX_ECHO_REQ".
p419: Change ".)." to ".)".
p422: Add Q_RX_ECHO_REQ, Q_RX_ECHO_RSP, Q_RX_FLUSH, Q_RX_ORG, and Q_TX_RS.
p422: Add Dequeue() and Enqueue().
p423: Remove hyphen from "FlushRequest-Frame(txFrame);".
p423: Add missing period, as directed by Comment 568.
p424: Change "= CT_OAM_ECHO_REQ" to "== CT_OAM_ECHO_REQ" (unless superceded
by following comment).
p424: Instead of pulling from Q_RX_MAC, pull from Q_RX_ECHO_REQ,
Q_RX_ECHO_RSP, Q_RX_FLUSH, and Q_RX_ORG. Merge Table 12.4 into Table 12.6,
moving the associated text as well.
p427: Change both instances "Clause 11" to "Clause 12" and correct the
references to use Frame cross-references.
p428: Change "Clause 11" to "Clause 12" and correct the reference to use a
Frame cross-reference.
p428: Move asterisks to the right side of the predicates.
Clause 13:
p431: Change "viewed as "containing" the MIB for that layer" to "viewed as
each implementing the MIB for its respective layer", as directed by Comment
593.
p431: Delete the last sentence, as directed by Comment 594.
p432: Correct Heading3 style, as directed by Comment 595.
p432: Change "The value of the mib_attribute." to "The value of the
mib_attribute returned by the MLME_GET.request.", as directed by Comment
598.
p432: Change "The status of the request." to "The status of the request
returned as a result of the MLME_GET.request.", as directed by Comment 599.
p433: Correct Heading3 style, as directed by Comment 597.
p433: Change "The status of the request (SUCCESS or FAIL)" to "The status of
the request returned as a result of the MLME_SET.request.", as directed by
Comment 600. Also, add two lines of values as done for status for
MLME_GET.request.
p433: Change "from" to "in", as directed by Comment 602.
p434: Correct cross-references in Table 13.1.
p434: Remove the period after "operational state", as directed by Comment
605.
p435: Change "Annex E" to "Annex D" and correct the reference to use a Frame
cross-reference.
p435: Add two lines of values as done for status for MLME_GET.request.
p436: Add period after "frame-content-dependent information", as directed by
Comment 613.
p437: Why are half of the MIB attributes in italics?
p438: Change "he agent" to "The agent".
p438: Change "align with the wall clock" to "aligns with the wall clock".
p438: Do we have a value yet for RFC xxxx in 13.3.3.2? Remove Editor's Note.
p439: Change "Clause 12" to "Clause 13" and correct the reference to use a
Frame cross-reference.
p440: Change "Clause 13" to use a Frame cross-reference.
p440: Add an asterisk after LME2.
p440: Change "Table13.2 and Table 13.3" to use Frame cross-references.
p440: Correct table border.
Annex K:
p694: Remove Editor's Note in K.2.3.
cls12_OAM.pdf