Page MenuHomeFreeBSD
Differential D4366

convert tcp timestamps to scaled sbintime
AbandonedPublic
Actions

Authored by kmacy on Dec 4 2015, 2:04 AM.
Tags
Referenced Files
Unknown Object (File)
Tue, Dec 9, 7:05 AM
Unknown Object (File)
Fri, Nov 28, 1:11 PM
Unknown Object (File)
Tue, Nov 25, 5:51 PM
Unknown Object (File)
Nov 21 2025, 4:46 PM
Unknown Object (File)
Nov 11 2025, 7:52 PM
Unknown Object (File)
Nov 9 2025, 10:25 AM
Unknown Object (File)
Nov 5 2025, 12:48 AM
Unknown Object (File)
Nov 4 2025, 10:50 PM
View All Files
Subscribers
benno
gnn
imp
jch
rstone
rwatson
transport

Details

Reviewers
rrs
lstewart
Group Reviewers
transport
Summary
  • convert tcp timestamps to using a scaled sbintime based off of the TSC

See the following paper for background:
Safe and Effective Fine-grained TCP Retransmissions for Datacenter Communication

NB: this a first iteration and still needs to be extended to support unsynchronized TSCs and refactor the MD bits more appropriately

Test Plan

Compare the incast mitigation effects of sub-millisecond RTO versus legacy tick based timestamps

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

kmacy updated this revision to Diff 10717.Dec 4 2015, 2:04 AM
kmacy retitled this revision from to convert tcp timestamps to scaled sbintime.
kmacy updated this object.
kmacy edited the test plan for this revision. (Show Details)
kmacy added a reviewer: lstewart.
kmacy set the repository for this revision to rS FreeBSD src repository - subversion.
kmacy added a project: transport.
kmacy added subscribers: benno, gnn, rwatson.
Herald added a subscriber: imp. · View Herald TranscriptDec 4 2015, 2:04 AM
rstone added a subscriber: rstone.Dec 4 2015, 3:24 AM
jch added a subscriber: jch.Dec 4 2015, 11:57 AM
kmacy updated this revision to Diff 10756.Dec 5 2015, 6:33 AM

convert tcp_timer_activate to sbintime

kmacy updated this revision to Diff 10757.Dec 5 2015, 6:34 AM

restore 40 line context

hiren added a reviewer: rrs.Oct 4 2016, 12:45 AM
hiren added a subscriber: transport.
kmacy added a comment.May 3 2018, 3:55 AM

@lstewart is this worth updating or should I just abandon?

Herald added a reviewer: transport. · View Herald TranscriptMay 3 2018, 3:55 AM
kmacy abandoned this revision.May 7 2018, 8:00 AM

Revision Contents
Changeset List

PathSize
sys/
amd64/
include/
2 lines
netinet/
93 lines
11 lines
68 lines
28 lines
10 lines
5 lines
41 lines
2 lines
30 lines
22 lines
2 lines
sys/
2 lines
x86/
x86/
31 lines

Diff 10757

View Options

sys/amd64/include/clock.h

Loading...
View Options

sys/netinet/tcp_input.c

Loading...
View Options

sys/netinet/tcp_output.c

Loading...
View Options

sys/netinet/tcp_seq.h

Loading...
View Options

sys/netinet/tcp_subr.c

Loading...
View Options

sys/netinet/tcp_syncache.c

Loading...
View Options

sys/netinet/tcp_timer.h

Loading...
View Options

sys/netinet/tcp_timer.c

Loading...
View Options

sys/netinet/tcp_timewait.c

Loading...
View Options

sys/netinet/tcp_usrreq.c

Loading...
View Options

sys/netinet/tcp_var.h

Loading...
View Options

sys/netinet/toecore.c

Loading...
View Options

sys/sys/callout.h

Loading...
View Options

sys/x86/x86/tsc.c

Loading...