Page MenuHomeFreeBSD

ng_iface(4): Remove unsupported protocols.
ClosedPublic

Authored by markj on Jul 28 2020, 3:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Oct 6 2024, 5:14 AM
Unknown Object (File)
Oct 3 2024, 8:44 PM
Unknown Object (File)
Oct 1 2024, 8:52 PM
Unknown Object (File)
Sep 28 2024, 10:39 AM
Unknown Object (File)
Sep 12 2024, 1:41 AM
Unknown Object (File)
Sep 12 2024, 1:40 AM
Unknown Object (File)
Sep 12 2024, 1:40 AM
Unknown Object (File)
Sep 12 2024, 1:40 AM

Details

Summary

Support for non-IPv4 and v6 hooks was removed long ago.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 32617
Build 30073: arc lint + arc unit

Event Timeline

markj requested review of this revision.Jul 28 2020, 3:29 PM
markj added a reviewer: donner.

What's the reason behind this proposal?

You remove a possible functionality (which is provided by other players) for the gain of couple of bytes in a static table and a slightly shorter man page.
Of course there private data field will become a bit shorter, too.

Why do you keep the definitions of these constants in ng_iface.h? Are there any other consumers?

Îf you want to stay in touch with the kernel capabilities, please consider a dynamic hook management which obtains the valid address family names from the real kernel table. This way, any change in the kernel supported protocols would be automatically available on the ng_iface.

In D25862#572900, @lutz_donnerhacke.de wrote:

What's the reason behind this proposal?

The documentation is incorrect. I wish to correct it. And if I do that I see no reason to keep around dead hooks either, so I removed them.

You remove a possible functionality (which is provided by other players) for the gain of couple of bytes in a static table and a slightly shorter man page.
Of course there private data field will become a bit shorter, too.

The actual functionality was removed a long time ago and has not come back. Sure, this is not a significant change, but I noticed the disparity when reading ng_iface and see no reason to leave it that way.

Why do you keep the definitions of these constants in ng_iface.h?

I missed them, thanks.

Are there any other consumers?

Nothing in the base system it seems.

Îf you want to stay in touch with the kernel capabilities, please consider a dynamic hook management which obtains the valid address family names from the real kernel table. This way, any change in the kernel supported protocols would be automatically available on the ng_iface.

Update ng_iface.h as well.

This revision is now accepted and ready to land.Jul 28 2020, 11:00 PM
This revision was automatically updated to reflect the committed changes.