Thread Links Date Links
Thread Prev Thread Next Thread Index Date Prev Date Next Date Index

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