Skip to content
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

Main/backup: Repurpose SRTO_GROUPSTABTIMEO to set the minimum timeout value #1792

Closed
1 task
maxsharabayko opened this issue Feb 5, 2021 · 0 comments · Fixed by #2081
Closed
1 task

Main/backup: Repurpose SRTO_GROUPSTABTIMEO to set the minimum timeout value #1792

maxsharabayko opened this issue Feb 5, 2021 · 0 comments · Fixed by #2081
Assignees
Labels
[core] Area: Changes in SRT library core Type: Enhancement Indicates new feature requests
Milestone

Comments

@maxsharabayko
Copy link
Collaborator

Consider repurposing SRTO_GROUPSTABTIMEO to provide the minimum timeout value.

PR #1775 introduces runtime link stability timeout (RLST), making SRTO_GROUPSTABTIMEO unused.
However, both runtime stability timeout (RLST) and initial link stability timeout (ILST) have a constant minimum allowed value of 60ms. Based on experimental data this value is considered an optimal one, but in some cases, it might be useful to allow specifying it via a socket option.
SRTO_GROUPSTABTIMEO is a good candidate for it.

Minimum Link Stability Timeout = 60 ms.
ILST = max(Minimum Link Stability Timeout, SRT Latency);
RLST = min(max(Minimum Link Stability Timeout, 2RTT + 4RTTVar), SRT Latency)

TODO

  • Rename SRTO_GROUPSTABTIMEO to SRTO_MINLINKSTABTIMEO and use it as the minimum link stability timeout.
@maxsharabayko maxsharabayko added Type: Enhancement Indicates new feature requests [core] Area: Changes in SRT library core labels Feb 5, 2021
@maxsharabayko maxsharabayko added this to the v1.4.3 milestone Feb 5, 2021
@maxsharabayko maxsharabayko self-assigned this Feb 5, 2021
@maxsharabayko maxsharabayko modified the milestones: v1.4.3, v1.4.4 Apr 14, 2021
@maxsharabayko maxsharabayko modified the milestones: v1.4.4, v1.4.5 Aug 12, 2021
@mbakholdina mbakholdina changed the title Repurpose SRTO_GROUPSTABTIMEO to set the minimum timeout value Main/backup: Repurpose SRTO_GROUPSTABTIMEO to set the minimum timeout value Sep 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[core] Area: Changes in SRT library core Type: Enhancement Indicates new feature requests
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant