[Sis-dtn] Some thoughts to the current HPRP draft (Section 1 - 4)

[구철회]

Hi, colleagues.

I just read the current HPRP draft and let me share some thoughts from me on the contents of Section 1 to 4.

1) 4.1 HPRP Segment
It could be informative if we add a ‘Data Segment Section (Optional, variable)’ after ‘Header Extensions’ field.
Refer the figure on page. 10 of RFC 5326 “LTP - Specification”.

2) 4.1 HPRP Segment
Segment Type ID (bit field of 4 - 5) could be extended to ‘bit field of 4 - 7’ to hold an indication of the ‘end of reliable data transmission in current session’ like EORP or EOB in LTP (RFC-5326). Why are the EORP and EOB field deprecated in HPRP? Unlikely CP (Checkpoint), these are useful for space saving and good match with FPGA, e.g., a fastest way of generating an interrupt for this event.
IMO, removing the descriptor of the ‘EORP’ or ‘EOB’ does not bring much benefit. Sending the DAR (Data Acknowledgement Request) at the edge of the data transmission may not be efficient although DAR is quite good message scheme for asynchronous state check of data reception. I think DAR is better way than ‘CP’ in LTP spec for this purpose.

3) 4.1.8.3
The value session number length --> The value of the session number
(Q) The session number can be 0? I think it should be non-zero because 0 means, generally, the first index of an array or indicator that means no actions required.

4) 4.2.8.2.2.1
  Data Acknowledgement extensions must be acknowledged by the receiving (--> sending) engine via a metadata acknowledgment message.

5) 4.3.2.2 Client Service ID
It could be informative if we add a description about how this field relate RFC-7116. Also, we may need to update this RFC-7116 document for adapting HPRP.

6) 5.2.3
 Upon the end of reception --> Upon the reception of un-successful data acknowledgement

7) 5.2.4 Transmit Acknowledgement Request
 IMO, ‘Timeout’ could be more semantically appropriate word than ‘Interval’ in this context.

Best,
Cheol


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.ccsds.org/pipermail/sis-dtn/attachments/20240723/34e4901a/attachment.htm>