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
F167523146: D21512.id61619.diff
Sat, Aug 22, 11:29 AM
Unknown Object (File)
Fri, Aug 21, 3:25 PM
Unknown Object (File)
Fri, Aug 21, 12:38 PM
Unknown Object (File)
Thu, Aug 20, 4:41 AM
Unknown Object (File)
Sun, Aug 9, 10:10 PM
Unknown Object (File)
Sun, Aug 9, 3:45 PM
Unknown Object (File)
Sun, Aug 9, 3:44 PM
Unknown Object (File)
Fri, Jul 24, 7:06 PM
View All 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...