Removed payload size dependency and option#1130
Conversation
maxsharabayko
left a comment
There was a problem hiding this comment.
Please provide a description on the changes this PR introduces, and estimate the impact of those changes on the behavior of SRT in live mode.
E.g. is the average payload size calculated correctly?
Which payload size is used to calculate byte statistics for missing packets: default or average?
Is is still possible to limit the payload size to 1316?
|
In general: The There's still an internally calculated payload size that is needed for various calculations in congestion control module. This is then calculated as starting from the maximum possible value. Maximum possible value is simply MSS minus UDP and SRT header size, although it also undergoes correction with the extra FEC header in case when filter is on. Also, when payloadsize is set to a nonzero value, which is required for correct functioning of the live CC, it's still checked if a single sending function doesn't try to send more at once. New functions:
|
|
@maxsharabayko Please review the PR, consider the issue #1322 while reviewing and let @ethouris know if any additional tests should be done on his side with regards to #1322. |
@ethouris It feels like it will not be possible to limit to e.g. 1316 if needed then? |
|
Yes. But I don't think anyone uses this limit for anything. At best you can limit MSS. |
Fixes #876