-
Notifications
You must be signed in to change notification settings - Fork 865
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SRT Protocol Technical Overview #479
Comments
The Drop Request packet structure is missing in the spec. |
FindingsPage 5: " * Periodic NAK reports (UDT4 disabled this draft feature in favor of timeout retransmission of unACKed packets, which consumes too much bandwidth for real-time streaming applications)." (... plus the continued description on page 6, also details referred to on Page 22). This is kinda more complicated, but not sure how this can be explained in short. The LOSSREPORT sent immediately after seeing a loss is only increasing a probability This situation is "ultimate retransmission failure" and it needs a fallback in form
As UDT was about the file transmission, it could not allow for conditional dropping, Blind retransmission means is sending everything since last ACK again - "blind" The method named as Late retransmission means to perform Blind retransmission SRT added the following new mechanisms:
I think in short it might be described as: "Periodic NAKREPORT" which increases probability of a successful retransmission
In the latest SRT with HSv5, this is exchanged in the handshake. In HSv4 also I'd say that it can be exchanged by either extended control messages, or in the handshake. Page 8: The picture for the control packet has the "Reserved" word for a part that is Page 9: The "TIMESTAMP" and "Destination Socket ID" fields could be at least mentioned The weird names of symbols are used in the table: This is also weird that some names are versalized, others capitalized, and
Page 15: I think it's worth mentioning that the timestamps are 32-bit numbers with Page 16: Interesting to mention may be that the sender relies on the predefined sending Page 19: No, this time of course isn't absolute, but it isn't local either - it's Page 20: I think the NAK is described in a wrong order. When the packet #4 arrives, but not packet #3, what is happening in this
There's also a mechanism of delaying the lossreport until a configured Page 34: I'd describe more about the multiplexer. The mechanism called "Multiplexer" in UDT is actually a facility that is
An SRT listener is simply bound to the UDP port as declared in the SRT listener
Normally you don't bind the socket you want to use to do This mechanism is then a must in case of a listener socket, whereas for Page 35: The Congestion Control Mechanism has been completely reinvented since the 1.3.0 Actually the only thing about the Smoother type that is important for the Page 71: The scheme for the packet breakdown of HSv5 handshake contains the pink There's an explanation at the end of page, but defining things this way Page 76: The table, like it was also earlier, contains "4a17h", which defines a magic Don't know if this is important, but it might be mentioned that this value |
The TLPKTDROP abbreviation is not explained.
|
@maxlovic , can you also add some information about the |
@stoyanovgeorge Your request has been addressed in PR-602. |
Field StreamID in Handshake v5 contains text field of a variable length , multiple to 4 bytes. Each 4 bytes are stored as BIG_ENDIAN. E.G. when `StreamID="HISTORY", then the field will have the value "TSIH YRO". Might be worth mentioning in the Tech. Spec. This reordering is done in fact for the whole control packet, so each textual field would be affected. |
Now that the big-endian fix has been added, we can now fix it in the protocol description. |
Added direct link to PDF instead of to Issue Haivision#479
Over the past several months, some of the original SRT developers have been working on documenting the protocol as we work towards bringing it to the point where an RFC can be submitted. You will find attached an initial draft release of the SRT Protocol Technical Overview, which we invite you to review. All comments will be addressed in subsequent releases.
SRT_Protocol_TechnicalOverview_DRAFT_2018-10-17.pdf
The text was updated successfully, but these errors were encountered: