Skip to main content

Last Call Review of draft-ietf-netext-pmip-qos-wifi-06
review-ietf-netext-pmip-qos-wifi-06-opsdir-lc-korhonen-2015-02-27-00

Request Review of draft-ietf-netext-pmip-qos-wifi
Requested revision No specific revision (document currently at 08)
Type Last Call Review
Team Ops Directorate (opsdir)
Deadline 2015-02-26
Requested 2015-02-17
Authors John Kaippallimalil , Rajesh Pazhyannur , Parviz Yegani
I-D last updated 2015-02-27
Completed reviews Opsdir Last Call review of -06 by Jouni Korhonen (diff)
Opsdir Telechat review of -07 by Warren "Ace" Kumari (diff)
Assignment Reviewer Jouni Korhonen
State Completed
Request Last Call review on draft-ietf-netext-pmip-qos-wifi by Ops Directorate Assigned
Reviewed revision 06 (document currently at 08)
Result Has nits
Completed 2015-02-27
review-ietf-netext-pmip-qos-wifi-06-opsdir-lc-korhonen-2015-02-27-00
I was assigned as the OPS-DIR reviewer for draft-ietf-netext-pmip-qos-wifi-06.
You should treat these comments as any LC comments. In my comments I use
datatracker idnits' line numbering.

Short summary:
 The I-D is ready technically but needs another editorial pass.

Operational & management concerns:
  It is a bit unclear to me how the stuff is suppose to work & managed when
  there are STAs with mixed WLAN QoS support, e.g. with VMM and VMM-AC.

Editorial:
  * Throughout the document in many places "section" and "chapter" appear in
    lowercase where they should start with uppercase letter.
  * Line 144: what is QSTA? Explain and give a reference
  * Section 1.1. Except for "RECOMMENDED" no other RFC2119 terminology is used.
    Could we change that one in line 711 and then get rid of the Section 1.1
    all together?
  * Section 1.2:
    Line 176: s/Enhanced/Evolved
    Line 182: s/Indicator/identifier
    Line 187: s/Conditioning Spec/Specification
    Then you are missing many many abbreviations from the list. Do a scan for
    missing one and add them.
  * Line 194 & 195: MAC and PHY never expanded nor defined.
  * Line 219: what is PDN?
  * Line 242: s/AIFS.)./AIFS).
  * Line 275: s/PMIP/PMIPv6   (check for other occurrences as well.. there are
  few). * Line 289: PBU/PBA never expanded. * Line 330: SIP never expanded or
  given a reference. * Line 428 and onwards: What is QoSx? It is never
  explained or given a ref. * Line 483: UPN is never expanded. Also a reference
  missing here. * Line 520: UPA is never expanded. Also a reference missing
  here. * Line 655: Expand QCI and also give a short explanation what it is. *
  Line 655: s/[[GSMA-/[GSMA- * Section 8.2: no reference to SIP RFC. * Section
  8.2: For WLCP reference you could consider 24.244 instead of 23.402??

- Jouni