Page MenuHomeFreeBSD

tcp: further cleanup old options
ClosedPublic

Authored by glebius on Sep 24 2024, 9:46 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 5, 11:24 PM
Unknown Object (File)
Sat, Nov 2, 3:26 PM
Unknown Object (File)
Tue, Oct 29, 1:29 AM
Unknown Object (File)
Tue, Oct 29, 1:29 AM
Unknown Object (File)
Tue, Oct 29, 1:29 AM
Unknown Object (File)
Tue, Oct 29, 1:29 AM
Unknown Object (File)
Tue, Oct 29, 1:29 AM
Unknown Object (File)
Tue, Oct 29, 1:29 AM
Subscribers

Details

Summary

They all were experimental and some comments refer to internal Netflix
versions. There is not reason to leak that into the header. Style unused
options so that they have the available value aligned with really used
values.

Diff Detail

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

Event Timeline

Internally at Netflix I will make all private options relative to TCP_VENDOR. This will make my merging much easier. Ideally we even want this to become a enum, and that would protect again any collisions and other failures.

I was using the style used in netinet/in.h. But I do not care how we mark them as unused as long as it is consistent.

Internally at Netflix I will make all private options relative to TCP_VENDOR. This will make my merging much easier. Ideally we even want this to become a enum, and that would protect again any collisions and other failures.

Using an enum will break the logic, if someone uses the pattern

#ifdef TCP_OPTION_NAME
....
#endif

I do this, for example, in symbols_freebsd.c for packetdrill.
One solution would be to use an enum and also #define. Not elegant...

sys/netinet/tcp.h
279

This is used by BBR.

This revision is now accepted and ready to land.Sep 29 2024, 2:25 PM
This revision was automatically updated to reflect the committed changes.