Page MenuHomeFreeBSD

sdp: garbage collect sdp_ctlinput
ClosedPublic

Authored by glebius on Aug 16 2022, 12:01 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 8, 9:16 AM
Unknown Object (File)
Fri, Nov 29, 7:19 AM
Unknown Object (File)
Mon, Nov 25, 5:05 AM
Unknown Object (File)
Sat, Nov 23, 8:00 AM
Unknown Object (File)
Fri, Nov 15, 4:27 PM
Unknown Object (File)
Nov 7 2024, 3:17 AM
Unknown Object (File)
Oct 17 2024, 6:15 PM
Unknown Object (File)
Oct 15 2024, 9:39 AM

Details

Summary

A pr_ctlinput method was a feature of IPv4/IPv6 with exeception of
pfctlinput(), which broadcasted a call to pr_ctlinput on all protocols
ever registered statically or with pf_proto_register(). Now that
this broadcast call is gone, the only protocols that get their
pr_ctlinput ever called are those that have registered itselves with
ipproto_register() or ip6proto_register().

It is entirely possible that code deleted now was dead code from very
beginning. Just a copy-paste from TCP.

Diff Detail

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

Event Timeline

Can we just get rid of SDP now? I think OneFS was the last major user, but it has switched away from IB. The SDP implementation was pretty fragile when I worked on it and it hasn't been changed much since. OneFS had a number of local patches for it.

Maybe @vangyzen @rstone or @bdrewery can comment on whether it's ok to remove?

Can we just get rid of SDP now? I think OneFS was the last major user, but it has switched away from IB. The SDP implementation was pretty fragile when I worked on it and it hasn't been changed much since. OneFS had a number of local patches for it.

Maybe @vangyzen @rstone or @bdrewery can comment on whether it's ok to remove?

Meanwhile I would consider it as "Reviewed" for removing its pr_ctlinput :)

Can we just get rid of SDP now? I think OneFS was the last major user, but it has switched away from IB.

IB and SDP remain alive and well in OneFS, I'm afraid.

This revision is now accepted and ready to land.Aug 16 2022, 6:56 PM
This revision was automatically updated to reflect the committed changes.