Page MenuHomeFreeBSD
Differential D38877

NgATM: deprecate
ClosedPublic
Actions

Authored by brooks on Mar 2 2023, 11:19 PM.
Tags
None
Referenced Files
F97548434: D38877.diff
Mon, Sep 30, 12:38 AM
Unknown Object (File)
Fri, Sep 27, 11:19 PM
Unknown Object (File)
Fri, Sep 27, 7:00 AM
Unknown Object (File)
Thu, Sep 26, 5:24 AM
Unknown Object (File)
Tue, Sep 24, 1:23 PM
Unknown Object (File)
Tue, Sep 24, 5:53 AM
Unknown Object (File)
Sat, Sep 21, 10:48 AM
Unknown Object (File)
Tue, Sep 10, 3:02 PM
View All 63 Files
Subscribers
donner
glebius
manu
melifaro

Details

Reviewers
emaste
imp
manu
Commits
rGa338a96efa45: NgATM: deprecate
rG0ad9e0e8241f: NgATM: deprecate
rG08a5f8c5cbc7: NgATM: deprecate
rG211c302f10d7: NgATM: deprecate
Summary

Add DREPRECATION NOTICEs to the manpages and gone_in(14, ...) warnings
in the constructors (or module init for ngatmbase).

MFC after: 3 days
Relnotes: yes

Diff Detail

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

Event Timeline

brooks created this revision.Mar 2 2023, 11:19 PM
Herald added subscribers: glebius, melifaro, donner. · View Herald TranscriptMar 2 2023, 11:19 PM
brooks requested review of this revision.Mar 2 2023, 11:19 PM
Harbormaster completed remote builds in B50119: Diff 118224.Mar 2 2023, 11:19 PM
brooks added a parent revision: D38876: netgraph.4: remove obsolete ng_atm.4 reference.Mar 2 2023, 11:19 PM
brooks added a child revision: D38878: ng_atmllc: deprecate.
emaste accepted this revision.Mar 2 2023, 11:23 PM
emaste added inline comments.
share/man/man4/ng_ccatm.4
53

Are we going to do this in 14 or 15?

This revision is now accepted and ready to land.Mar 2 2023, 11:23 PM
manu accepted this revision.Mar 3 2023, 7:32 AM
manu added a subscriber: manu.
manu added inline comments.
share/man/man4/ng_ccatm.4
53

No need to wait for 15 IMHO, if people want to use ATM they should just use what was available back in 2000

Closed by commit rG211c302f10d7: NgATM: deprecate (authored by brooks). · Explain WhyMar 3 2023, 4:48 PM
This revision was automatically updated to reflect the committed changes.
brooks added a commit: rG211c302f10d7: NgATM: deprecate.
brooks added inline comments.Mar 3 2023, 5:40 PM
share/man/man4/ng_ccatm.4
53

I'd like to do this in 14, but am happy to push it to 15 and update the deprecation messages if we find there are users who need it. It's all pretty self-contained (unlike NATM which included bits in ifconfig, etc).

brooks added a commit: rG08a5f8c5cbc7: NgATM: deprecate.Mar 6 2023, 6:28 PM
brooks added a commit: rG0ad9e0e8241f: NgATM: deprecate.Mar 6 2023, 6:57 PM
brooks added a commit: rGa338a96efa45: NgATM: deprecate.Mar 6 2023, 9:23 PM

Revision Contents
Changeset List

PathSize
share/
man/
man4/
7 lines
7 lines
7 lines
7 lines
7 lines
sys/
netgraph/
atm/
ccatm/
2 lines
2 lines
sscfu/
2 lines
sscop/
2 lines
uni/
2 lines

Diff 118270

View Options

share/man/man4/ng_ccatm.4

Loading...
View Options

share/man/man4/ng_sscfu.4

Loading...
View Options

share/man/man4/ng_sscop.4

Loading...
View Options

share/man/man4/ng_uni.4

Loading...
View Options

share/man/man4/ngatmbase.4

Loading...
View Options

sys/netgraph/atm/ccatm/ng_ccatm.c

Loading...
View Options

sys/netgraph/atm/ngatmbase.c

Loading...
View Options

sys/netgraph/atm/sscfu/ng_sscfu.c

Loading...
View Options

sys/netgraph/atm/sscop/ng_sscop.c

Loading...
View Options

sys/netgraph/atm/uni/ng_uni.c

Loading...