[manet] WGLC comments on draft-ietf-manet-dlep-credit-flow-control

["Velt, R. (Ronald) in 't" <[e-mail]>]

Hi Authors, MANET WG participants,

At long last, here is my review of draft-ietf-manet-dlep-credit-flow-control (with chair mask off). Reviews of the other flow control related I-Ds in WGLC will follow shortly.

Thanks,
Ronald

--

Major Issues
============

None

Minor Issues
============

Section 1, Introduction, contains the somewhat puzzling sentence "Traffic sent by a router will use traffic flow classification information provided by the modem as defined in [I-D.ietf-manet-dlep-traffic-classification] to identify which traffic is associated with each credit window." I would think it is the router, as opposed to the traffic, that uses the classification information. The traffic, which may originate from an upstream source, is what it is. But then the question is HOW the router should make use of the classification information. Should it e.g. re-mark certain traffic flows (i.e., alter their DSCPs) to ensure that they receive preferential treatment (i.e., are throttled last in case of congestion)? Can the router assume that the modem understands Per-Hop Behaviors? In my opinion, at the very least the sentence quoted above should be rephrased; in addition some further clarification on the router-side mechanism foreseen could be helpful.

Section 2.1, Data Plane Considerations, states: "Note that routers will typically view a DLEP destination as the next hop MAC address." Does this still hold true, now that we have the Link Identifier extension as specified in RFC8703?
Both the Credit Window Initialization Data Item (section 2.3.1) and the Credit Window Status Data Item (section 2.3.4) have a field named Credit Window Size, but these have different meanings: in the former case, it represents a maximum window size, whereas in the latter case, it holds the actual (current) window size. I would suggest renaming either of these, e.g. to Credit Window Limit in the Credit Window Initialization Data Item or to Current Credit Window Size in the Credit Window Status Data Item.
A special Flow Identifier value of 0xFFFF, indicating all FIDs, is defined in the specification of the Credit Window Request Data Item. Should it be stated, that if this special value is used, it MUST be the only FID present in the Data Item?

Nits
====

Section 1

1st para: "... does not include any flow identification beyond DLEP endpoints or flow control capability" => "... does not include any flow identification beyond DLEP endpoints nor flow control capability" (?)

1st para: replace [I-D.ietf-manet-dlep-pause-extension] by [RFC8651]

2nd para: Redundant full stop after [I-D.ietf-manet-dlep-traffic-classification] (but see comment under Minor Issues on the sentence as a whole).

Section 2

2nd para: "negotiate the use of extension" => "negotiate the use of this extension"

3rd pare: "per logical "Credit Windows" basis" => "per logical "Credit Window" basis"

3rd para: DSCP, expand on first use

3rd para: FID, expand on first use

3rd para, sentence before last: "In addition to the traffic classification information associated with an FID, routers provide an initial credit window size, as well as ..." should presumably be: "In addition to the traffic classification information associated with an FID, modems provide an initial credit window size, as well as ..."

7th para: "a modem "Associates" ..." => "a modem "associates" ..."

7th para: TID, expand on first use

Section 2.1

1st para: "... including any MAC headers ..." => "... including any MAC headers and trailers ..." (i.e., take Frame Check Sequence into account?)

Section 2.2.1

2nd para: "... the credit window value previous provided to the router." => "... the credit window value previously provided to the router."

3rd para: "... and processing frequent credit window requests against a having data traffic available to send, ..." => "... and processing frequent credit window requests against a router having data traffic available to send, ..."

Section 2.2.2

2nd para: "A Data Item for every Credit Window Request Data Item contained in the corresponding Credit Control Response Message received by the modem MUST be included." => "A Data Item for every Credit Window Request Data Item contained in the corresponding Credit Control Message received by the modem MUST be included."

Section 2.3

1st para: "The Credit Request is used by a router to request additional credits." => "The Credit Window Request is used by a router to request additional credits."

Section 2.3.1

Scale field definition: "... used in the Credit Window Size fields." => "... used in the Credit Window Size field."

last para: "... that the FID field value has been provided by the modem in a Traffic Classification Data Item carried in either the current or previous message." => "... that the FID field value has been provided by the modem in a Traffic Classification Data Item carried in either the current or a previous message." (not completely sure about this one)

Section 2.3.2

Throughout this section (including its title), replace "Credit Window Associate" by "Credit Window Association"

Section 2.3.3

1st para: "... or are listed in a Credit Window Initialization Data Item carried in the same messages as the Data Item." => "... or are listed in a Credit Window Initialization Data Item carried in the same message as the Data Item."

Add definition of 'Reserved' field, as was done for the Credit Window Initialization Data Item in section 2.3.1

Section 2.3.4

Add definition of 'Reserved' field, as was done for the Credit Window Initialization Data Item in section 2.3.1

last para: "... at the time the most Credit Window Initialization or Grant Data Item associated with the indicated FID was sent." => "... at the time the most recent Credit Window Initialization or Grant Data Item associated with the indicated FID was sent."

Section 2.3.5

2nd para: "Credit windows identified using a FID as defined above in Section 2.3.1." => "Credit windows are identified using a FID as defined above in Section 2.3.1."

Section 3

"The data items defined in this document will only be used when extensions require their use." =>  "The messages and data items defined in this document will only be used when extensions require their use."

Section 6.2

Replace informative reference [I-D.ietf-manet-dlep-pause-extension] by [RFC8651]

Appendix A

1st para: "We morn the loss ..." => "We mourn the loss ..."
This message may contain information that is not intended for you. If you are not the addressee or if this message was sent to you by mistake, you are requested to inform the sender and delete the message. TNO accepts no liability for the content of this e-mail, for the manner in which you use it and for damage of any kind resulting from the risks inherent to the electronic transmission of messages.