Page MenuHomeFreeBSD
Differential D21512

Fix two TCP Rack issues
ClosedPublic
Actions

Authored by tuexen on Sep 3 2019, 7:02 PM.
Tags
None
Referenced Files
F142457607: D21512.id61619.diff
Tue, Jan 20, 4:21 AM
F142411065: D21512.id61618.diff
Mon, Jan 19, 6:10 PM
Unknown Object (File)
Sun, Jan 18, 7:49 AM
Unknown Object (File)
Dec 21 2025, 2:41 AM
Unknown Object (File)
Nov 20 2025, 3:42 PM
Unknown Object (File)
Nov 20 2025, 3:40 PM
Unknown Object (File)
Nov 20 2025, 3:37 PM
Unknown Object (File)
Nov 20 2025, 3:14 PM
View All 76 Files
Subscribers
imp

Details

Reviewers
rrs
rscheff
Group Reviewers
transport
Commits
rS351782: Fix two TCP RACK issues:
Summary

This patch fixes two issues:

  • Convert the TCP delayed ACK timer from ms to ticks as required. This fixes the timer on platforms with hz !=1000.
  • Don't delay acknowledgements which report duplicate data using DSACKs.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 26258

Event Timeline

tuexen created this revision.Sep 3 2019, 7:02 PM
Herald added a reviewer: transport. · View Herald TranscriptSep 3 2019, 7:02 PM
Herald added a subscriber: imp. · View Herald Transcript
Harbormaster completed remote builds in B26258: Diff 61618.Sep 3 2019, 7:02 PM
rrs accepted this revision.Sep 3 2019, 7:07 PM
This revision is now accepted and ready to land.Sep 3 2019, 7:07 PM
Closed by commit rS351782: Fix two TCP RACK issues: (authored by tuexen). · Explain WhySep 3 2019, 7:48 PM
This revision was automatically updated to reflect the committed changes.
tuexen added a commit: rS351782: Fix two TCP RACK issues:.

Revision Contents
Changeset List

PathSize
sys/
netinet/
tcp_stacks/
7 lines

Diff 61618

View Options

sys/netinet/tcp_stacks/rack.c

Loading...