Page MenuHomeFreeBSD

net: remove legacy in_addmulti()
ClosedPublic

Authored by kp on Feb 24 2021, 5:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 5, 5:31 PM
Unknown Object (File)
Oct 10 2024, 7:40 AM
Unknown Object (File)
Oct 9 2024, 9:56 AM
Unknown Object (File)
Sep 9 2024, 1:13 AM
Unknown Object (File)
Sep 5 2024, 3:11 PM
Unknown Object (File)
Aug 31 2024, 6:06 AM
Unknown Object (File)
Aug 27 2024, 3:55 PM
Unknown Object (File)
Aug 19 2024, 10:27 PM
Subscribers

Details

Summary

Despite the comment to the contrary neither pf nor carp use
in_addmulti(). Nothing does, so get rid of it.

Carp stopped using it in 08b68b0e4c6b132127919cfbaf7275c727ca7843
(2011). It's unclear when pf stopped using it, but before
d6d3f01e0a3395c1fae34a3c4be7b051cb2d7581 (2012).

Diff Detail

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

Event Timeline

kp requested review of this revision.Feb 24 2021, 5:16 PM
bz added a subscriber: bz.

Looks ok to me; Do we assume that no kmods from port would still use this?

This revision is now accepted and ready to land.Feb 24 2021, 5:30 PM

That would be my assumption.

It's hard to know for sure no one uses this. The best way to figure that out is to remove it and wait for the shouting to start.
I'd say we should remove it in main (i.e. 14), but not MFC the removal. That should give us plenty of time to notice if this does break something.

+1 for removing and not MFCin.

This revision was automatically updated to reflect the committed changes.