Page MenuHomeFreeBSD

net: remove legacy in_addmulti()
ClosedPublic

Authored by kp on Feb 24 2021, 5:16 PM.
Tags
None
Referenced Files
F81447494: D28918.diff
Tue, Apr 16, 12:30 PM
Unknown Object (File)
Mar 7 2024, 6:02 PM
Unknown Object (File)
Feb 3 2024, 8:16 AM
Unknown Object (File)
Jan 18 2024, 1:16 PM
Unknown Object (File)
Dec 20 2023, 4:54 AM
Unknown Object (File)
Dec 18 2023, 11:44 PM
Unknown Object (File)
Dec 14 2023, 8:21 PM
Unknown Object (File)
Nov 27 2023, 6:43 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
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 37350
Build 34239: arc lint + arc unit

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.