Page MenuHomeFreeBSD

tcp: account sent/received IP ECN markings independently
ClosedPublic

Authored by rscheff on Nov 9 2022, 11:22 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 10, 2:45 PM
Unknown Object (File)
Tue, May 7, 6:57 PM
Unknown Object (File)
Fri, May 3, 11:21 PM
Unknown Object (File)
Tue, Apr 16, 8:23 PM
Unknown Object (File)
Tue, Apr 16, 6:24 AM
Unknown Object (File)
Tue, Apr 16, 2:24 AM
Unknown Object (File)
Tue, Apr 16, 12:17 AM
Unknown Object (File)
Tue, Apr 16, 12:11 AM
Subscribers

Details

Summary

TCPS (netstat -s) would not differentiate between received and sent
ECN-marked packets, but would only account received packets if the
tcp session had negotiated ECN support.

It is known that some devices incorrectly set the historic TOS byte
and may infringe on the IP ECN bits - but without accounting IP ECN
packets where the session is not ECN-enabled would mask such misconfigurations.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

sys/netinet/tcp_var.h
828

not sure if we will need a sent ce counter - this may only be relevant for future CC where a receiver driving marking is also possible. Or for path probing...

sys/netinet/tcp_ecn.c
124

Why are you removing this line?

sys/netinet/tcp_var.h
831

I think you want to go from 6 to 4 now, not to 3.

However, wasn't the plan to not change this at all in the main branch?

usr.bin/netstat/inet.c
831

Don't you need to change the "\t{:*/%ju} " stuff?

This revision is now accepted and ready to land.Nov 10 2022, 9:20 AM