Page MenuHomeFreeBSD

Add new port: net/tcprtt: reliably measure the TCP handshake round trip time using stats(9)
ClosedPublic

Authored by trasz on Jun 15 2019, 3:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 3, 7:30 AM
Unknown Object (File)
Mon, Dec 2, 9:45 PM
Unknown Object (File)
Wed, Nov 13, 12:43 PM
Unknown Object (File)
Oct 17 2024, 6:37 PM
Unknown Object (File)
Oct 4 2024, 8:39 AM
Unknown Object (File)
Oct 3 2024, 10:48 PM
Unknown Object (File)
Oct 3 2024, 4:36 PM
Unknown Object (File)
Oct 3 2024, 6:37 AM
Subscribers

Details

Summary

Add new port: net/tcprtt

The tcprtt utility reliably measures the TCP handshake round trip time
using the new stats(3) statistics framework.

Note that this depends on https://reviews.freebsd.org/D20655.

Sponsored By: Klara Inc, Netflix
Obtained from: Netflix

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 25269
Build 23936: arc lint + arc unit

Event Timeline

trasz retitled this revision from Add new port: net/tcprttt to Add new port: net/tcprtt.
matthew requested changes to this revision.Jun 19 2019, 5:57 AM

I'm getting compilation failures due to missing sys/qmath.h on 12.0 and 11.2

tcprtt.c:32:10: fatal error: 'sys/qmath.h' file not found
#include <sys/qmath.h>
         ^~~~~~~~~~~~~
1 error generated.
*** [tcprtt.o] Error code 1

make[1]: stopped in /wrkdirs/usr/ports/net/tcprtt/work/tcprtt-1.0
1 error

make[1]: stopped in /wrkdirs/usr/ports/net/tcprtt/work/tcprtt-1.0
net/tcprtt/files/Makefile
1 ↗(On Diff #58672)

I'm wondering about the split of actions between this Makefile and the ports Makefile. It seems odd to have the
build stage controlled here, but the install stage from elsewhere.

This revision now requires changes to proceed.Jun 19 2019, 5:57 AM

Given this:

Note that this depends on https://reviews.freebsd.org/D20655.

Presumably this port will be limited to CURRENT once D20655 gets committed, and then waiting on MFC to a release branch
before it can be packaged for wider distribution? I'm not sure about adding a port that only works on CURRENT. That's
a question for portmgr@

However it should have tests against ${OSVERSION} to ensure we don't waste time trying to build it on OS versions where
it is never going to work...

linimon retitled this revision from Add new port: net/tcprtt to Add new port: net/tcprtt: reliably measure the TCP handshake round trip time using stats(9) .Jun 20 2019, 9:46 AM

The source files should not be put in the "files" directory, if one day you need to add support files, like *.in files, it will clutter everything. It would be preferable to put them in a src directory.

trasz marked an inline comment as done.
trasz retitled this revision from Add new port: net/tcprtt: reliably measure the TCP handshake round trip time using stats(9) to Add new port: net/tcprtt: reliably measure the TCP handshake round trip time using stats(9).

Move sources to GitHub, add __FreeBSD_version check.

Since I don't know what the version will be, the number
will probably change before committing.

trasz edited the summary of this revision. (Show Details)

Fix build with upstreamed changes.

Update after fix from thj@.

allanjude added inline comments.
net/tcprtt/Makefile
21

stats.h is committed now, so we should update this to the correct number

net/tcprtt/Makefile
21

This also needs the TCP part of the stats(3) framework, which is not committed yet.

This revision was not accepted when it landed; it landed in state Needs Review.Dec 2 2019, 9:33 PM
Closed by commit rP518879: Add new port: net/tcprtt (authored by trasz). · Explain Why
This revision was automatically updated to reflect the committed changes.