Page MenuHomeFreeBSD

sdp: garbage collect sdp_ctlinput
ClosedPublic

Authored by glebius on Aug 16 2022, 12:01 AM.
Tags
None
Referenced Files
F82043296: D36208.diff
Wed, Apr 24, 10:36 PM
Unknown Object (File)
Fri, Apr 19, 12:59 PM
Unknown Object (File)
Feb 13 2024, 3:03 AM
Unknown Object (File)
Jan 10 2024, 2:15 AM
Unknown Object (File)
Dec 12 2023, 7:39 AM
Unknown Object (File)
Dec 8 2023, 10:05 PM
Unknown Object (File)
Dec 8 2023, 9:53 PM
Unknown Object (File)
Dec 3 2023, 11:25 PM

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.