Page MenuHomeFreeBSD

Remove pr_destroy in favour of VNET_SYSUNINITs.
ClosedPublic

Authored by bz on May 30 2016, 8:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 21, 4:30 PM
Unknown Object (File)
Sat, Dec 21, 4:29 PM
Unknown Object (File)
Tue, Dec 17, 6:32 PM
Unknown Object (File)
Tue, Dec 10, 11:08 AM
Unknown Object (File)
Mon, Dec 9, 10:19 PM
Unknown Object (File)
Tue, Nov 26, 5:07 AM
Unknown Object (File)
Nov 20 2024, 10:33 AM
Unknown Object (File)
Nov 14 2024, 2:23 PM
Subscribers

Details

Summary

The pr_destroy field does not allow us to run the teardown code
in a specific order. VNET_SYSUNINITs however are doing exactly that.
Thus remove the VIMAGE conditional field from the domain(9) protosw
structure and replace it with VNET_SYSUNINITs.
This also allows us to change some order and to make the teardown functions
file local static.
Also convert divert(4) as it uses the same mechanism ip(4) and ip6(4) use
internally.

Slightly reshuffle the SI_SUB_* fields in kernel.h and add a new ones, e.g.,
for pfil consumers (firewalls), partially for this commit and for others
to come.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4040
Build 4083: arc lint + arc unit

Event Timeline

bz retitled this revision from to Remove pr_destroy in favour of VNET_SYSUNINITs..
bz updated this object.
bz edited the test plan for this revision. (Show Details)
bz added reviewers: gnn, rwatson, emaste, jhb.
bz added a subscriber: network.

Add Michael for the SCTP change.

tuexen edited edge metadata.

The SCTP changes are fine, therefore the 'Accept Revision' for them.
Once this hits the svn tree, I'll integrate the changes upstream.

gnn edited edge metadata.
This revision is now accepted and ready to land.May 31 2016, 3:07 PM
sys/sys/kernel.h
142

s/Intialize/Initialize/.

147

Maybe SI_SUB_FIREWALL? FW is perhaps ambiguous (e.g. "firmware") OTOH, given it's in the middle of SI_SUB_PROTO_* perhaps SI_SUB_PROTO_FW? (I think it would be clearer if we did 's/SI_SUB_PROTO_/SI_SUB_NET_/' but that's a separate topic.)

bz marked 2 inline comments as done.Jun 1 2016, 9:37 AM

Addressed the changes suggested by @jhb . Thanks.

This revision was automatically updated to reflect the committed changes.