heistp / irtt

Isochronous Round-Trip Tester
GNU General Public License v2.0
185 stars 23 forks source link

Added --ecn server option to ship ECN bits (currently IPv6 only) #28

Closed eric-s-raymond closed 3 years ago

eric-s-raymond commented 3 years ago

The smaller, cleaner pull request I promised.

This meets Dave's stated request to be able to see what if anything has been done to the ECN bits between the irtt server and the client.

Note that the new --ecn server option and the existing --set-src-ip have the same effect, forcing the server not to use UDP for replies. I was trying to disturb the existing code as little as possible, your call whether to merge or rename these options.

I'm working on enabling --ecn for IPv4 as well.

heistp commented 3 years ago

I'll merge this in, but haven't had time to test it so I trust it works. I'll add something to the roadmap somewhere to refactor lconn and this code so as to not require setting the src IP to use this. The addition of this feature makes it apparent that that setSrcIP stuff was never very clean.