[rohc] Another SigComp document review

["Lars-Erik Jonsson (EPL)" <[e-mail]>]

Dear all,

I have also reviewed the SigComp documents, and have some minor
editorial comments.


"Signaling Compression Requirements & Assumptions"
<draft-ietf-rohc-signaling-req-assump-05.txt>

- Abstract 
  Should be expanded. "ROHC" should be removed to avoid confusion
  (might be interpreted as if the desired solution is at the
  link-layer, based on the ROHC framework). Further must the 
  mnemonics be expanded (e.g. SIP/SDP). For more details
  about abstracts, see draft-rfc-editor-rfc2223bis-02.txt.

- Section 1., second paragraph. Should be either one or two
  paragraphs, now it is one "broken" paragraph.

- Section 1.1. It could be clearer that these protocols are
  examples.

- Section 2.1. Should we move Figure 2.1 to appear earlier in the text?

- Section 2.1., second paragraph. First two sentences do not say
  what they were intended to say, I think. Proposed modified text:
  "It should be noted that the used figures represent a very narrow
  band link. E.g. a WCDMA system can provide maximum bit rates up
  to 2 Mbits/s in ideal conditions, but that means one single user
  would consume all radio resources in the cell."

- Section 4., "Unreliable transport". TCP is an example, so we 
  should say "e.g. [TCP]."

- Appendix A. I think the appendix should now be removed, since
  this was intended for testing, and is now covered by a separate
  draft defining a dictionary to use.


"Signaling Compression"
<draft-ietf-rohc-sigcomp-06.txt>

- Section 5. Usage of keywords should be consistent and clear. For
  requirement 1. I suggest:
  "1. For robustness, it is RECOMMENDED that the compressor..."

- Section 5.1. Usage of keywords should be correct and clear.
  For the second paragraph, I suggest:
    "...ratio the compressor should request..."
  For the third paragraph, I suggest:
    "...however, it MUST ensure that...", and
    "...however, the compressor MUST provide a..."
  For the fourth paragraph, I suggest:
    "The compressor MUST also ensure..."

- Section 7., 5th paragraph. The examples "...(e.g. one or more 
  previously received messages, a dictionary of common SIP
  phrases etc.)" are unclear and confusing. What does it mean
  to explicitly indicate these? Is it to send some amount of
  data, or is it to use references? Further, if one says 
  "previously received messages", what does it mean? Is it
  about received messages at the compressor or the decompressor?
  I would suggest to remove the examples, and keep the rest of
  the text, which, apart from this, is rather clear.
  
- Section 8., second paragraph. The text "...provide unlimited
  flexibility..." is in my opinion not correct, and if it was I
  think it had been really bad. We could just remove "unlimited".

- Section 8.4, last paragraph. We should separate the instructions
  listed with at least a comma, as done in other parts of the draft.


"SigComp - Extended Operations"
<draft-ietf-rohc-sigcomp-extended-03.txt"

- Abstract could be improved. It could clearer state that this is
  about how to implement certain mechanisms within SigComp. Further,
  overall it could be clearer and more complete in itself.

- Section 2., "Explicit acknowledgement" definition. Rewrite
  text. Now it is not clear whether it is the acknowledgement or
  the state that is explicitly sent.

- Section 5.6, second paragraph. End of page says "As input into
  to...", remove "into".


Cheers,
/L-E


-----------------------------------
Lars-Erik Jonsson, M.Sc
Wireless IP Optimizations
AWARE - Advanced Wireless Algorithm Research and Experiments
Ericsson Research, Corporate Unit
Ericsson AB
Box 920, S-971 28 Luleå, Sweden
E-mail: [e-mail]
Phone: [phone]
Fax: [phone]
Home: [phone]

My opinions are my personal opinions and should not be considered
as the opinions of my employer, if not explicitly stated. 

_______________________________________________
Rohc mailing list
[e-mail]
https://www1.ietf.org/mailman/listinfo/rohc