[RPRWG] Minutes of Teleconference
Please email me with all errors / ommissions
(most likely attendees who I missed when the logged in
or off).
cheers,
mike
--
Michael Takefman tak@xxxxxxxxx
Distinguished Engineer, Cisco Systems
Chair IEEE 802.17 Stds WG
3000 Innovation Dr, Ottawa, Canada, K2K 3E8
voice: 613-254-3399 cell:613-220-6991
IEEE 802.17 Teleconference
Attendees:
M. Takefman, Bob Sultan, Robert Castellano, Harry Peng,
John Lemon, Stein Gjessing, Michael Allen, Necdet Uzun,
Tom Alexander, Don O'Connor, Tom Mathey, Jason Fan,
David James, Mark Holness
Open Issues Sync Up:
OAM etc:
Leon wants a motion to merge Tables 12.4 and 12.6
as per David's clause 12 write-up.
Leon has some editorial changes based on comments
from JLemon and Others. Given below
Subclause ordering changes from DVJ
- delayed to OAM ad-hoc
Leon received a DVJ file with a rewritten OAM
clause much discussion.
- delayed to OAM ad-hoc
Global:
RunTogether StateMachine Names.
Space Hyphens etc etc etc - no need for a motion from
the group since while it is editorial and there is no
license, this is something that IEEE editors do it for
free.
Introduction:
No particular issues.
HEC comment from Leon
Architecture:
A few issues with regard to resolutions from JohnLemon. Will be
discussed later in the meeting.
MAC:
Couple of problems: Forgot/missed a few changes that were mandated.
One non-authorized change that will be the subject of a motion.
A technical comment from Leon that has to be described.
PHY: some editorials
Fairness:
One issue concerning comment 287: units of the timestamp
Michael Allen brought up an issue on how a formula is
written that can cause a value to go negative and breaks the
calculation if it isn't done right.
DVJ new document:
Topology:
4 items to be brought up before the group for guidance.
3 editor's notes and
ComputeTotalUnreservedRate 10.4.2 - where to put it
10-9Row9 10-9row16
Specific Resolutions / Motions:
MOTIONS:
-------------------------------------------------------
Motion 1:
Move to remove 12.4.1.6 and replace 12.4.5 with text
from DVJ proposal Clause12_oam1.pdf clause 12.7.2
(OAM Frame Receive State Machine)
Move: Leon Bruckman
Second: David James
No Opposition
-------------------------------------------------------
Motion 2:
Move to accept all of the typographical errors detailed
below:
- Page 404, line 2: Double full stop (periods) after the word "network"
= remove one period
- Page 405, line 34: sesThreshold is bold and all other variables aren't
= remove bold, delete colon insert linefeed
- Page 405, line 34: Missing period before "to declare"
= add space before "to declare"
- Page 408, 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..."
= replace with The contents of this field"
= P407 Line 44 same change as above
= P408 line 9 same change as above
= P408 line 48 change "user_data, data[n]" with "userData" and remove italics from
brackets but not "n"
= p408 line 46 Figure 12.6 change "organization data" to userData
= p408 line 46 change "organization data" to userData
- Page 412, line 27: Replace "Table 6.1. (See.." with: "Table 6.1 (see.."
= change text in parans to "see Table 7.1"
- page 412 line 30: See also becomes see
= remove also
- Page 415, line 10: according with comment 556 (refers to the previous line
but similar issue) replace: "previously" with: "all previously"
= replace "previously" with: "all previously"
- Page 409, line 36 and page 415 line 24: Replace "Annex L" with: "Annex K"
= page 409 line 36 replace Annex L with Annex K use FrameMaker cross-references
= page 415 line 24 replace Annex L with Annex K use FrameMaker cross-references
- p403: Change "Provide mechanism" to "Provide a mechanism".
= change "Provide mechanism" to "Provide a mechanism"
- p404: Change "network.." to "network.".
= remove one period
- p405: Change "Q_OAM_REQ" to correct font size.
= change font
p405: Add Q_RX_ECHO_REQ, Q_RX_ECHO_RSP, Q_RX_FLUSH, Q_RX_ORG, and Q_TX_RS.
= Add Q_RX_ECHO_REQ, Q_RX_ECHO_RSP, Q_RX_FLUSH, Q_RX_ORG, and Q_TX_RS
= Editor to go through each definition in 12.2.2.1 and remove any that
are unused.
p406: RingletSelect() is no longer used. Remove it.
= p406 remove ringletselect
= p410 Table 12.3 change RingletSelection() to defaultRinglet()
= p409 remove ringletselect
= p413 remove ringletselect
p407: Change "will be encoded" to "is encoded".
= accepted
p408: Change all instances of "8" in Figures 12.5, 12.6, and 12.7 to "4"
= accepted
p413: Add Q_RX_ECHO_REQ and Q_TX_RS.
= P413 add Q_RX_ECHO_REQ and Q_TX_RS to 12.4.1.6.2
= P413 Editor to remove any defintion that 12.4.1.6.2 that is unused
p413: Add Dequeue() and Enqueue().
= p413 add Dequeue() and Enqueue() to 12.4.1.6.3
p415: Change ".)." to ".)".
p415 line 30 fix to .)
p415 line 34 fix to .)
p415 line 35 fix to .)
p418: Add Q_RX_ECHO_REQ, Q_RX_ECHO_RSP, Q_RX_FLUSH, Q_RX_ORG, and Q_TX_RS.
= accept
p419: Add Dequeue() and Enqueue().
= accept addition to 12.4.4.3
p419: Remove hyphen from "FlushRequest-Frame(txFrame);".
= line 30 change the keyword to nameplain
p419: Add missing period, as directed by Comment 568.
=p419 line 45 add period
p423: Change both instances "Clause 11" to "Clause 12" and correct the
references to use Frame cross-references.
= line 1 accept
= line 37 accept
p424: Change "Clause 11" to "Clause 12" and correct the reference to use a
Frame cross-reference.
= line 1 accept
p424: Move asterisks to the right side of the predicates.
= line 8, 10 accept
p430: Correct cross-references in Table 13.1.
= reject - defer to next round since it is a non trivial change
p431: Change "Annex E" to "Annex D" and correct the reference to use a Frame
cross-reference.
= accept
p431: Add two lines of values as done for status for MLME_GET.request.
= copy lines 40/41 from P428 paste after line 25 on P429 and after
line 8 on P431
p433: Why are half of the MIB attributes in italics?
= change all italics to normal case
p434: Change "he agent" to "The agent".
= line 13 accept
p434: Change "align with the wall clock" to "aligns with the wall clock".
= line 13 accept
= p434 line 18 Change "RFC XXXX" to RFC 3705. Remove text in parens, remove
editors note and add reference to RFC 3705 in clause 2
p435: Change "Clause 12" to "Clause 13" and correct the reference to use a
Frame cross-reference.
= line 37 accept
p436: Change "Clause 13" to use a Frame cross-reference.
= line 1 accept
p436: Add an asterisk after LME2.
= line 17 accept
p436: Change "Table13.2 and Table 13.3" to use Frame cross-references.
= line 32 chance to cross references
p436: Correct table border.
= line 36 change to thin
= p436 center subclause, status, support and item
p690: Remove Editor's Note in K.2.3.
= accept
Move: Leon Bruckman
Second: John Lemon
No objection
----------------------------------------------------------------------------
Global Motions and Decisions:
Editors are allowed to fix errors with spaces, hyphens
etc since IEEE editing staff would do so anyway without
WG permission or approval.
Fix all cross references and such like.
Change all italicized names to use name-italic
Motion 3:
Move to replace subclause 3.4.1 state table notation with 3.4.1
state table notation in contribs1.pdd (as of 2/17/2004 8am)
from DVJ.
Move: David James
Second: John Lemon
No objection
---------------------------------------------------------------------------
MAC Motions and Decisions:
Deferred
* Page 138, line 29: The "reserved shaping" alarm is not defined in the MIB.
Delete the sentence: "If rated...station management."
= Leon to resubmit and John to determine who made the request to
determine what the reason was.
- Page 152, Table 7.20 row 26: The ttlExpiredFrames counters counts here
frames with ttl=0 that indicate something wrong since there should be no
frames with ttl=0, and frames with ttl=1 in table 7.22 row 15 which are
valid frames. This is misleading, I recommend to count only frames with
ttl=0.
= resubmit as a comment on D3.1
---------------------------------------------------------------------------
Motion 4:
Move to accept all of the typographical errors detailed
below:
P112 l 23: Remove stqLowThreshold from 7.2.4.
= accept
p114: Change reference for stqHighThreshold definition from 7.2 to 10.3.2.2 in 7.3.1.
= accept
p121 l41 : Remove "nominally".
= accept
p123 l27 : Remove "nominally".
= accept
p164: Modify Table 7.24 to add optional rows below Row 11 to check for secondary addresses.
= accept
p184: Change 7.7.2 as suggested by Editor's Note.
= accept, John to change the CRD to this instruction.
p198: Remove "nominal".
= accept
p208: Approve change already made to Table 7.35, Row 10.
= accept
p215: Move asterisks to the right side of the predicates.
= accept
Page 83, line 1: Move table 6.3 to appear before 6.4.1.3
= accept
Page 84, line 23: Missing comma after "extended_frame"
= accept
- Page 120, line 31: Replace "are greater" with: "is greater"
= accept
- Page 140, line 48: In this sentence there are two different ways of
referring a row within a table: Table-Row 7.23-5 and Table7.29, Row 12. Make
it consistent.
= accept
- Page 237, line 38: Missing word. Replace "filed is" with: "field that is"
see also 9.2.2.12.
= accept
- Page 238, line 22: Replace "frames, and to reduce" with: "frames, to
reduce"
= accept
Move: John Lemon
Second: Peter Jones
No Objection
-----------------------------------------------------------------------------
Motion 5:
Move to accept the technical changes detailed below
p164: Modify Table 7.24 to add optional rows below Row 11 to check for secondary addresses.
= accept
p198: Change lowThreshold to stqLowThreshold in (two places in) 7.7.7.
= accept
p198: Change highThreshold to stqHighThreshold in 7.7.7.
= accept
p208: Approve change already made to Table 7.35, Row 10.
= accept
- Page 133, line 3: Replace "short of the congestion point" with: "beyond
the congestion point"
= accept
- Page 140, line 31: It is not clear what "in flight" means and what is the
difference between these frames and the frames in the transit queues.
= create a definition and add it to clause 3
- Page 142 Figure 7.8: It would be useful to show from which sub-entity the
frame is passed to the Filter (an arrow pointing up between Count and Strip)
=accept
- Page 192 Table 7.31: In the action of rows 17 and 18 replace "SC_CLASSB"
with: "SC_CLASSC"
= accept
- Page 235, line 10: According to Table 7.29 row 16 the da and daExtended my
be different and not as stated here.
= accept
- Page 238, line 25: The ttl definition is wrong. In 9.2.2.1 the ttl is said
to be set to: "the number of hops the frame is expected to cover before
reaching the destination", but for fairness frames the right definition is
written in 10.4.1. Copy the definition from there, and delete it from
10.4.1.
= accept
Move: John Lemon
Second: David James
No Objections
-----------------------------------------------------------------------------
David's run together name suggestions.
Motion 6:
Move to rename state machines using the following mapping of
current names to new run-together names in all text including
headings, body, references. The first time a state machine is referenced
in a paragraph it should be referred to as "The <name> state machine" and
can be referred to as simply <name> later..
Idle shaper ==> IdleShaper
MAC control shaper ==> MacControlShaper
ClassA shaper ==> ClassAShaper
ClassB shaper ==> ClassBShaper
Pre-congestion shaper ==> PreCongestionShaper
Post-congestion shaper ==> PostCongestionShaper
Source shaper ==> SourceShaper
Fairness eligible send indication ==> FairnessEligibleIndication
Downstream shaper ==> DownstreamShaper
Receive from edge ==> ReceiveFromEdge
Receive check ==> ReceiveCheck
Receive count ==> ReceiveCount
Receive strip ==> ReceiveStrip
Receive adjust ==> ReceiveAdjust
Receive filter select ==> ReceiveFilter
Receive filter data count ==> ReceiveFilterDataCount
Receive filter control count ==> ReceiveFilterControlCount
Wrong ringlet ==> WrongRinglet
Ringlet selection ==> RingletSelection
Stage queue selection ==> StageQueueSelection
Data add count ==> DataAddCount
Control add count ==> ControlAddCount
Single-queue transmit selection ==> SingleQueueTransmit
Dual-queue transmit selection ==> DualQueueTransmit
Transmit count state table ==> TransmitCount
Per-byte activities ==> PerByte
agingInterval updates ==> PerAgingInterval
Aggressive rate adjustment ==> AggressiveRateAdjust
Conservative rate adjustment ==> ConservativeRateAdjust
Per-advertisingInterval ==> PerAdvertisingInterval
Per-reportingInterval ==> PerReportingInterval
Per-activeWeightsInterval ==> ActiveWeightsComputation
Receive fairness frame ==> FairnessFrameReceive
Send FDD frames ==> FddFrameTransmit
FRTT computation ==> FrttComputation
EchoResponse+OamReceive ==> OamFrameReceive
OamRequest ==> OamFrameTransmit
Move: David James
Second: John Lemon
No Objection
--------------------------------------------------------------------------
Bridging:
Motion 7:
Move to accept all of the typographical errors detailed
below:
p641: Reference to dot1 IEC/ISO is out of date
= add an editors note requesting that it be updated when possible or
removed by the IEEE staff editors.
p633: Footnote to IEEE 802.1Q 2003
= leave for IEEE staff to fix
p635:Change "(a, c, and e of Figure F.3)" to "
(see a, c, and e of Figure F.3)". Add "see" to other such parenthetical
cross-references.
= accept modify
= line 4 (see Figure F.3a, Figure F.3c and Figure F.3e)
= line 6 (see Figure F.3b, Figure F.3d and Figure F.3f)
= lines 13,14,15,19,20,22,26,28,30,33,34,35 of page 635
add the word "see" after the left paren and a dash in the figure
label
= lines 7,23 of page 637
add the word "see" after the left paren and a dash in the figure
label
= lines 39 of page 638
add the word "see" after the left paren and a dash in the figure
label
= lines 12,13 of page 643
add the word "see" after the left paren and a dash in the figure
label
= lines 3 of page 646
change "defined in" to "see"
= p639 l 1 remove the word below
= p641 l21-23 replace existing sentence with
MAC clients, invoking optional RPR MA_DATA.request service
parameters,should maintain 802.1D/Q bridging conformance and 802
bridged network filtering integrity (see F.1.4)
by adhering to the following rules:
- p641 l28 : Change "source_address_extended; and wishes to be compliant
with 802.1D/1Q, the client should follow these rules" to
"source_address_extended should follow these rules".
= replace the semi-colon with a comma. Then strike all text following
the comma until and including the words "the client" on l29
- p647: Resolve and remove Editor's Note in F.3.1.
= follow the editors note and then remove it.
= p650 l49 remove linebreak
= p650 l50/51/52 fix font
= p650 l51 change "tx rules" to "transmit rules"
= p651 l1,3,5,8 remove linebreak
Move: Robert Castellano
Second: Marc Holness
No Objection
--------------------------------------------------------------------------
Fairness Editorials
Motion 8:
Move to accept all of the typographical errors detailed
below:
p248: Move 10.2 and subclauses to 10.1.1 and subclauses.
= accept
p248: Remove italics from parenthesis.
= accept
p248: Replace the period after "similar manner".
= accept
p248: Change "Figure 10.2 illustrates fairness instances on an open ring" to
"Figure 10.3 illustrates fairness instances on an open ring".
=accept
p248: Change "Figure 10.5 illustrates" to "Figure 10.4 illustrates".
=accept
p248: Change "As illustrated by the leftmost station of Figure 10.5" to
"As illustrated by the leftmost station of Figure 10.4".
=accept
p249: Change "As illustrated by the rightmost station of Figure 10.5" to
"As illustrated by the rightmost station of Figure 10.4".
=accept
p257: Force Figure 10.14 onto one page.
=accept
p259: Change "(see 32)" to "(see 7.5.7)".
=accept
p269: Move list to previous page.
= accept move the list and remove the italics on ) and () in item e)
p298 l29, l52: Remove italics from parenthesis.
=accept
p299 l1: Remove italics from parenthesis.
=accept
p300 l15: Remove trailing "(".
= accept
- Page 250, line 4: Wrong use of term. Replace "cleave point" with: "failed
span"
= accept
- Page 252, line 52: Inconsistent text "S1 to S4 and S5 to S6" compared with
page 257, line 32 "S1-S4 and S5-S6". Use same format.
= accept
- Page 254, line 52: Wrong reference. Replace "Station S4 illustrates" with:
"Station S3 illustrates"
= accept
- Page 257, line 3: Text formatting. Italize normCoef
= accept
- Page 262, line 39: Wrong reference. The LINK_RATE is specified in Table
10.16 and not in the PHY annex.
= accept
- Page 263, line 46: Redundant text. Replace "The agingInterval is...as
illustrated in Table 10.2" with: "The agingInterval allowed values are
specified in Table 10.2"
= accept
- Page 299, line 20: Remove note, it is not relevant anymore.
= accept
= p 256 l 39 change "as shown in" to "as specified by"
Move: Necdet Uzun
Second: John Lemon
No Objection
--------------------------------------------------------------------------
Fairness Technicals:
Motion 9:
Move to accept the technical changes detailed below
p264: Change "The value, in units of 2-32 second, of the fairness round
trip time" to "The value, in microseconds, of the fairness round trip time"
= accept - restore the text to the original due to procedural issues
surrounding whether the resolution was accepted by the WG
p295: Change "units of 2-32 second" to "microseconds". (Comment 287 does
not appear to have been resolved by the WG.)
= accept - restore the text to the original due to procedural issues
surrounding whether the resolution was accepted by the WG
p297: Change "units of 2-32 second" to "microseconds".
= accept - restore the text to the original due to procedural issues
surrounding whether the resolution was accepted by the WG
p299: Change "units of 1/(232) second" to "microseconds".
= accept - restore the text to the original due to procedural issues
surrounding whether the resolution was accepted by the WG
p336: Change both instances of "units of 1/232 seconds (a full scale
value corresponds to 1 second)" to "microseconds".
= accept - restore the text to the original due to procedural issues
surrounding whether the resolution was accepted by the WG
Comment #287: This comment is rejected as the WG cannot reach consensus
on the units to use for the timestamp values. Commenter and WG members
are requested to discuss and reach consensus, and bring in a new comment
with the appropriate remedy against the next draft.
p2651: Change "the protection protocol" to either "the topology protocol"
or "the topology and protection protocol".
=change "by the protection protocol" to "in the topology database"
- Page 261 Table 10.1: Redundant text. Replace the value of agingInterval
with: "See Table 10.2"
= accept
* Page 271, line 1: The definition of fddTimeStamp is not complete. See
11.3.4.2. Also define that a station is allowed to set fddTimeStamp to 0 if
it is not able to populate it.
=accept - use text from Topology for 11.3.4.2 as a starter
- Page 298 Figure 10.19: Change "LRTT response path" and "LRTT request path"
to "affected data path" and "fairness path" as shown in figures 10.18 and
10.20.
= accept
= Page 272 l 16 replace sentence by "The contents of an RPR frame"
= Page 291 l 47 replace sentence by "The contents of an RPR frame"
= page 288 l 21 replace sentence by "The contents of an RPR frame"
= page 284 l 48 replace sentence by "The contents of an RPR frame"
- page 294 10.13 Row 8/9 lines 16, 20
= change "as described in" to "(see ... )"
- page 297 l12 & L 19.5
= change "as described by" to "as specified by"
- p276 Add a note next to equation 10.7, LowPassFilterRates().
= "NOTE: The calculations in Equation 10.7 require signed arithmetic for
correct operation. These equations can also be rephrased to only use
unsigned arithmetic. For example:
lpfwRate = ((lpCoef - 1) * lpfwRate + fwRate) / lpCoef;"
Move: Necdet Uzun
Second: John Lemon
No Objection:
--------------------------------------------------------------------------
Fairness deferals:
- Page 269 Table 10.3: This table includes information already available in
Table 9.5, and also contradicts it in the setting of "we". Remove table from
here, resolve the "we" issue, and add the "ri" to Table 9.5.
= defer to a new comment in the next draft
--------------------------------------------------------------------------
Protection and Topology
Motion 10:
Move to accept the technical changes detailed below
P 375 L 37 Request to change defect name set in row 6 of Table 11.16.
UnexpectedInvalidity?
= change topoInstabilityDefect to topoEntryInvalidDefect
= add definition in 11.2.9 for topoEntryInvalidDefect: A critical
severity defect indicating that an invalid entry has been found
within the scope of the topology
p. 269 line 8 D3.0a: Remove italicization from e) and () following Other.
= accept
p329: Add Q_RX_ATD.
Q_RX_ATD_SMAC currently referred to in text
= accept
p333: Add a period after "as specified in Figure 11.9".
= accept
p335: Remove Editor's Note in 11.3.4. TTL is specified correctly in Table 7.30.
= accept
p336: Change both instances of "units of 1/232 seconds (a full scale value corresponds to 1 second)" to
"microseconds".
= accepted - backed out as in fairness clause.
p338: Change "8-1023" to "4-1023".
= accept
p343: Change all instances of "8" in Figures 11.23 and 11.24 to "4".
= accept
p352: Resolve Editor's Note in 11.6.1.
= accept
p368: Resolve Editor's Note in 11.6.5.5.
= accept
p372: Remove mark on page after end of text.
?
p401: SM12 status should be O (and the clause text should reflect this optionality).
= accept
p402: Add an asterisk after FF3.
= accept
p403: Widen Subclause column.
= accept
p403: Add an asterisk after ATT2, ATT7, and ATT10.
= accept
p404: Add an asterisk after PP4.
= accept
1. Row 27 of ParseTpFrame should NOT set topoEntry[rid][hops].reachable to FALSE
(that line should be deleted from the action). The other lines in the action are fine.
(This solves the valid=TRUE and reachable=FALSE issue which Michael showed me in files that he sent).
NOT DONE CORRECTLY. topoEntry[rid][hops].reachable is FALSE for row 26. It retains the same value as before for row 27.
= accept
2. In ProtectionUpdate (thanks to Jim and David also): (Yes, change FALSE values to TRUE on rows 10 and 11)
Row 9: add wtr[ri].enabled = TRUE;
Row 10: add wtr[ri].enabled = TRUE;
Row 11: add wtr[ri].enabled = TRUE;
Row 12: add wtr[ri].enabled = TRUE;
Row 25: remove wtr[ri].enabled = TRUE;
Row 26: remove wtr[ri].enabled = TRUE;
Row 27: remove wtr[ri].enabled = TRUE;
= accept
This will improve the WTR behavior. In addition, Michael notes that when he brings down both sides of a station
then brings both sides of a station up simultaneously, he doesn't always get a WTR. I don't think that a WTR
can be guaranteed in this case, depending on the timing.
= accept
3. In SecondaryMacMarkInUse() (thanks to David also):
Near the beginning of the routine, excess = Min(...) should be excess = Max(...)
= accept
4. In the function DistantStateMax in 11.6.5.5:
Replace the line:
if (!tp.valid || tp.macAddress == myTopoInfo.macAddress)
With
if (!tp.valid || tp.macAddress == myTopoInfo.macAddress ||
(ringlet != ri && topoEntry [Other(ringlet)][1].valid &&
tp.macAddress == topoEntry [Other(ringlet)][1].macAddress))
= accept
EXTRA PAREN ON LINE 29
= accept
5. Table 11.16 (11.6.6.5), row 9:
Move the routine call MarkReachableEntries(); prior to the line myTopoInfo.myChecksum.value = ComputeTc();
= accept
6. Leon's list:
Some comments:
- Page 320, line 6: The sentence "The topology database...each
myTopoInfo..." seems to be wrong since this clause defines topoEntry and not
myTopoInfo. Should be replaced with text similar to the text in page 318
line 38, replacing "myTopoInfo" with topoEntry.
[JF] Yes
NOT DONE
- Page 324, line 24: "..severity defecting indicating.." replace with:
"severity defect indicating.."
[JF] Yes
= accept
- Page 328, line 13: "..is specified in Figure 11.11". replace with: "..as
illustrated in Figure 11.11"
= accept
- Page 334, line 1 to 25: This seems out of context in this clause. Consider
moving to another clause (maybe 11.2)
[JF] Put in editorial note stating that WG needs to determine where to move this routine.
= accept
- Page 334, line 43: The sentence in parenthesis is practically repeated in
the following sentence, so delete it (also it uses the term "retransmits"
where "transits" is better)
[JF] Yes
= accept
- Page 335, line 26: "When received..." this sentence shall be used in
almost all the clauses that define ATTs, or not at all.
[JF] Add this sentence in each ATT subclause as a useful reference.
= accept
- Page 343, line 18: In bullet c) What "ATD types" are defined ? I believe
there is only one ATD, so what does this rule mean ?
[JF] This should be re-worded to state "When an ATD frame is received, the information
contained in the ATTs within this frame is used to update the topology database."
= accept
- Page 343, line 24: "...for the entries are derived..." something wrong
with this sentence, maybe the word "are" should be deleted.
[JF] This should be re-worded to state "The index associated with this database represents
the distance-to-source hops on ringlet0 or ringlet1, derived from TP frames received on
ringlet1 and ringlet0, respectively."
= accept
- Page 349, line 19: In row 3 of the table, I missed this in D3.0, but this
is wrong (maybe this version is easier to read..). The whole idea of having
a holdoff is that if RPR is running over a SONET ring we allow the SONET to
protect, before the RPR protects. Assume that there is a failure in the
SONET ring, the keep alive will stop and RPR will protect without waiting
for the holdoff to expire ! So can anyone explain to me in which case the
MAC will wait for the holoff to expire before protecting ?
[JF] Change maximum value of keepaliveTimeout range (in 11.6.2.1) to 200 milliseconds
= accept
- Page 359, line 35: Editorial error "onboth" instead of "on both"
[JF] Yes (on p. 359 line 54)
= accept
- Page 360, line 7: "..other than S2" replace with "..other than S2 and S5"
[JF] S5 will see the FS conditions becuase S4 and S6 report the FS condition to S5 (as the FS
is configured on S4 and S6). S2 is the station that sees idle edges, as S1 and S3 report idle
edge conditions. I'll clarify that S5 is different from S2 for this reason.
= accept
- Page 362, line 52: The sentence starting with: "Indicates to the LME..."
shall be deleted, since the previous sentence already describes this.
[JF] Yes
= accept
- Page 373, line 16: The sentence: "In the case..." shall be moved to
precede the state table.
[JF] Yes
= accept
- Page 382, line 6: The sentence: "So that other stations time to react.."
does not make sense, see D3.0 clause 10.3.
[JF] The words "So that other stations time" will be replaced by "In order to allow other
stations time", so that the wording is identical to the first sentence of 10.13 from D3.0.
= accept
- Page 382, line 39: "ATD frames are illegal..." this sentence is wrong, the
ATD is legal, the ATT is discarded (see resolution of comment 456). Replace
with: "ATT shall be discarded under the following conditions"
[JF] Yes
= accept
- Page 388, line 17: The sentence: "In the case..." shall be moved to
precede the state table.
[JF] Yes
= accept
- Page 392, line 9: Missing sentence before state table: "In the case..."
[JF] Yes
= accept
It looks good to me overall. I have a few minor comments:
- Section 11.2.6 on page 321 line 4, the "NOT_USED" state should be listed
as well.
= accept
- Section 11.2.7 on page 322 line 39, the "break" should be replaced by
"continue".
= accept
- Section 11.2.7 on page 322 line 42, the "topoInfo" should be replaced by
"myTopoInfo".
= accept
- Section 11.4.8 on page 337 line 33, the "8" should be replaced by "4",
and the
"n=length-8" should be replaced by "n=length-4", since oui+depedentID is
4 bytes.
= accept
- Section 11.4.8 on page 338 line 8, the "8" should be replaced by "4",
since there are 4 bytes.
= accept
- Section 11.6.1 on page 346 line 39, the second "TC frame in/out" should
be replaced by
"ATD frame in/out".
= accept
- Section 11.6.3.3 on page 351 line 32, the "topoInfo" should be replaced
by "myTopoInfo".
= accept
- Section 11.6.4.4 on page 357 line 27, for row 27 of table 11.14, the
reachability should
be set to TRUE.
Duplicate of earlier one
No. topoEntry[rid][hops].reachable is FALSE for row 26. It retains the same value as before for row 27.
- Section 11.6.5.5 on page 362 line 17, the return value of the
"DistantStateMax" function
should be the protection state, not a boolean.
= accept
- Section 11.6.5.6 on page 365 line 30, in row 37 of table 11.15, the
"topoInfo" should be
replaced by "myTopoInfo".
= accept
- Section 11.6.6.3 on page 368 line 35, the "instability" should be
replaced by "stability".
Replace "instability defect" with "topology stability".
= accept
- Section 11.6.13.3 on page 384 line 15, the "macAddress" parameter should
be replaced
by "test".
= accept
- Section 11.6.13.3 on page 384 line 28, the "address != test" should be
replaced by
"address == test".
= accept
- Section 11.6.14.4 on page 392 line 38, in row 10 of table 11.22, the action
"txFrame.tailLatencyIn = frame.tailLatencyIn" should be replaced by
"txFrame.latencyTimeStamp = frame.latencyTimeStamp".
Yes. Also, the references to “frame” in row 10 will be replaced with “rxFrame”.
= accept
Move Jason Fan
Second John Lemon
No Objection
--------------------------------------------------------------------------
Motion 11:
Move to add the following definitions in clause 3.
in transit: received by a MAC's PHY and not yet transmitted by the MAC's other PHY
in flight: transmitted by the source MAC and not yet received
by the (final) destination MAC
Moved John Lemon
Second David James
No Objection
--------------------------------------------------------------------------
Motion 12:
Move to accept the following typographical changes:
p219: Change "RPR protection mechanism" to "the RPR protection mechanism".
=accept
p225: Change "this frame length parameter" to "the frame length parameter".
=accept
p225: Remove Editor's Note from 8.4?
=accept
p227: Change "The Flag sequence used indicates the beginning or end of a frame, and Control escape used for transparency are
specified in Table 8.3" to "The Flag sequence (used to indicate the beginning or end of a frame) and Control escape (used for transparency) are
specified in Table 8.3".
=accept
p231: Change width of Value/Comment column in 8.5.4.1 to prevent awkward break in "PHY_LINK_STATUS.indication".
=accept
p453: Items C1 and C2 subclause references need to be changed (from plain text) to Frame cross-references.
=accept
p476: Resolve and remove Editor's Note in C.5.1.3.
= accept under the valid data column insert txdata[63:0]
p488: Items C1, C3, and C4 subclause references need to be changed (from plain text) to Frame cross-references.
=accept
p489: Comment for SA4 should be "RENB" instead of "RVAL".
=accept
p490: Comment for SB3 should be "RENB" instead of "RVAL".
Move Harry Peng
Second John Lemon
No Objection