Page MenuHomeFreeBSD
Differential D40424

tcp: make the maximum number of retransmissions tunable per VNET
ClosedPublic
Actions

Authored by rscheff on Jun 5 2023, 5:51 PM.
Tags
None
Referenced Files
F108763834: D40424.diff
Mon, Jan 27, 7:18 PM
Unknown Object (File)
Sun, Jan 5, 7:13 AM
Unknown Object (File)
Thu, Jan 2, 4:08 PM
Unknown Object (File)
Nov 19 2024, 9:47 PM
Unknown Object (File)
Oct 23 2024, 10:32 AM
Unknown Object (File)
Oct 2 2024, 2:20 AM
Unknown Object (File)
Oct 1 2024, 5:30 PM
Unknown Object (File)
Sep 26 2024, 9:25 AM
View All 76 Files
Subscribers
imp
melifaro

Details

Reviewers
cc
rrs
jtl
tuexen
glebius
Group Reviewers
transport
Commits
rG43b117f88f30: tcp: make the maximum number of retransmissions tunable per VNET
Summary

With Windows (TcpMaxDataRetransmissions) and Linux (tcp_retries2)
there is precedent that sometimes it is useful to restrict the
maximum number of retransmissions sent - as each of these will
be sent after an exponentially increasing interval.

This adds that same capability on a per-VNet basis to FreeBSD.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 51907
Build 48798: arc lint + arc unit

Event Timeline

rscheff created this revision.Jun 5 2023, 5:51 PM
Herald added a reviewer: transport. · View Herald TranscriptJun 5 2023, 5:51 PM
Herald added subscribers: melifaro, imp. · View Herald Transcript
rscheff requested review of this revision.Jun 5 2023, 5:51 PM
Harbormaster completed remote builds in B51907: Diff 122856.Jun 5 2023, 5:51 PM
tuexen added inline comments.Jun 5 2023, 7:14 PM
sys/netinet/tcp_timer.c
225

Maybe something like "maximum number of consecutive timer based retransmissions"?

519

What happens if the sysctl variable is changed during the existence of a TCP connection?

rscheff updated this revision to Diff 122865.Jun 5 2023, 8:39 PM
  • deal with V_tcp_retries reduced while in a RTO sequence
Harbormaster completed remote builds in B51910: Diff 122865.Jun 5 2023, 8:39 PM
rscheff updated this revision to Diff 122866.Jun 5 2023, 8:43 PM
rscheff marked an inline comment as done.
  • better short description
Harbormaster completed remote builds in B51911: Diff 122866.Jun 5 2023, 8:43 PM
rscheff marked an inline comment as done.Jun 5 2023, 8:43 PM
cc added inline comments.Jun 5 2023, 9:24 PM
share/man/man4/tcp.4
37

Back to the future? Shall be Jun?

sys/netinet/tcp_timer.c
225

Maybe something like "maximum number of consecutive timer based retransmissions"?

+1

rscheff updated this revision to Diff 122876.Jun 6 2023, 8:20 AM
  • man date proper month
Harbormaster completed remote builds in B51919: Diff 122876.Jun 6 2023, 8:20 AM
rscheff marked 2 inline comments as done.Jun 6 2023, 8:21 AM
cc accepted this revision as: cc.Jun 6 2023, 12:24 PM
cc added inline comments.
sys/netinet/tcp_timer.c
227

Remove this extra empty line. And I am OK with the rest of the change.

rscheff updated this revision to Diff 122893.Jun 6 2023, 1:08 PM
  • fix whitespace
Harbormaster completed remote builds in B51929: Diff 122893.Jun 6 2023, 1:08 PM
tuexen accepted this revision.Jun 6 2023, 8:17 PM
This revision is now accepted and ready to land.Jun 6 2023, 8:17 PM
rscheff closed this revision.Jun 6 2023, 9:58 PM
rscheff added a commit: rG43b117f88f30: tcp: make the maximum number of retransmissions tunable per VNET.

Revision Contents
Changeset List

PathSize
share/
man/
man4/
5 lines
sys/
netinet/
2 lines
tcp_stacks/
8 lines
8 lines
32 lines
2 lines

Diff 122856

View Options

share/man/man4/tcp.4

Loading...
View Options

sys/netinet/tcp_output.c

Loading...
View Options

sys/netinet/tcp_stacks/bbr.c

Loading...
View Options

sys/netinet/tcp_stacks/rack.c

Loading...
View Options

sys/netinet/tcp_timer.c

Loading...
View Options

sys/netinet/tcp_var.h

Loading...