Skip to content

RFC 4960 Errata fixes

Marcelo Ricardo Leitner linux-sctp netdev 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest4 patches 1 replyIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4sctp: update order of adjustments of partial_bytes_acked and cwndnet/sctp/transport.c+8 -70 replies
2/4sctp: allow increasing cwnd regardless of ctsn moving or notnet/sctp/transport.c+8 -90 replies
3/4sctp: adjust cwnd increase in Congestion Avoidance phasenet/sctp/transport.c+18 -80 replies
4/4sctp: adjust ssthresh when transport is idlenet/sctp/transport.c+2 -00 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH net-next 0/4] RFC 4960 Errata fixes

This patchset contains fixes for 4 Errata topics from
https://tools.ietf.org/html/draft-ietf-tsvwg-rfc4960-errata-01
Namely, sections:
3.12. Order of Adjustments of partial_bytes_acked and cwnd 3.22. Increase of partial_bytes_acked in Congestion Avoidance 3.26. CWND Increase in Congestion Avoidance Phase 3.27. Refresh of cwnd and ssthresh after Idle Period

Tests performed with netperf using net namespaces, with drop rates at 0%, 0.5% and 1% by netem, IPv4 and IPv6, 10 runs for each combination. I couldn't spot differences on the stats. With and without these patches the results vary in a similar way in terms of throughput and retransmissions.

Tests with 20ms delay and 20ms delay + drops at 0.5% and 1% also had results in a similar way, no noticeable difference.

Looking at cwnd, it was possible to notice slightly lower values being used while still sustaining same throughput profile.

Marcelo Ricardo Leitner (4):
sctp: update order of adjustments of partial_bytes_acked and cwnd
sctp: allow increasing cwnd regardless of ctsn moving or not
sctp: adjust cwnd increase in Congestion Avoidance phase
sctp: adjust ssthresh when transport is idle

 net/sctp/transport.c | 54 ++++++++++++++++++++++++++++++++--------------------
 1 file changed, 33 insertions(+), 21 deletions(-)
Signature · 1 line
2.9.4