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)
Sun, May 17, 12:30 PM
Unknown Object (File)
Sun, May 17, 10:34 AM
Unknown Object (File)
Sat, May 16, 9:44 PM
Unknown Object (File)
Sat, May 16, 5:21 PM
Unknown Object (File)
Thu, May 14, 6:11 AM
Unknown Object (File)
Thu, May 7, 2:46 PM
Unknown Object (File)
Thu, May 7, 12:41 PM
Unknown Object (File)
Wed, May 6, 1:42 AM
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/
78 lines
6 lines
68 lines
19 lines
8 lines
18 lines
2 lines
16 lines
20 lines
x86/
x86/
31 lines

Diff 10717

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.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/x86/x86/tsc.c

Loading...