dcsctp: Bump rto_min to 220 ms

The minimum RTO time shouldn't be lower than the delayed ack timeout
of the peer to avoid sending retransmissions before the peer has
actually intended to reply.

In usrsctp, the default delayed ack timeout is 200ms and configurable
using the `sctp_delayed_sack_time_default` option. In dcsctp, it's
min(RTO/2, 200ms), to avoid this issue.

Bug: webrtc:12614
Change-Id: Ie84c331334af660d66b1a7d90d20f5cf7e2a5103
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/219100
Reviewed-by: Florent Castelli <orphis@webrtc.org>
Commit-Queue: Victor Boivie <boivie@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#34026}
This commit is contained in:
Victor Boivie
2021-05-17 17:25:23 +02:00
committed by WebRTC LUCI CQ
parent 7ddadbc108
commit cfa932f6fc

View File

@ -95,8 +95,9 @@ struct DcSctpOptions {
// Maximum RTO value.
DurationMs rto_max = DurationMs(800);
// Minimum RTO value.
DurationMs rto_min = DurationMs(120);
// Minimum RTO value. This must be larger than an expected peer delayed ack
// timeout.
DurationMs rto_min = DurationMs(220);
// T1-init timeout.
DurationMs t1_init_timeout = DurationMs(1000);