Page MenuHomeFreeBSD
Differential D42527

kern linker: Do not try to unload module if it has dependants
ClosedPublic
Actions

Authored by zlei on Nov 9 2023, 10:06 AM.
Tags
None
Referenced Files
F170484397: D42527.diff
Sat, Sep 5, 1:15 AM
Unknown Object (File)
Tue, Sep 1, 8:23 PM
Unknown Object (File)
Tue, Sep 1, 7:48 PM
Unknown Object (File)
Sun, Aug 30, 3:02 PM
Unknown Object (File)
Sun, Aug 30, 12:29 AM
Unknown Object (File)
Fri, Aug 28, 1:16 PM
Unknown Object (File)
Fri, Aug 28, 1:16 PM
Unknown Object (File)
Fri, Aug 28, 12:39 PM
View All Files
Subscribers
imp
jhb
olce
patmaddox

Details

Reviewers
dfr
kib
dab
jhb
Commits
rGa2f57656620d: kern linker: Do not unload a module if it has dependants
rGf1994d1eb215: kern linker: Do not unload a module if it has dependants
rGf43ff3e15c8b: kern linker: Do not unload a module if it has dependants
Summary

linker_file_unload() will just drop a reference in case the module have more than one
references. For user requested unload, we are going to unload but not just to drop a
reference.

PR: 274986
MFC after: 2 weeks

Test Plan
# kldload bridgestp
# kldload if_bridge
# kldunload bridgestp
kldunload: can't unload file: Device busy

Diff Detail

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

Event Timeline

zlei created this revision.Nov 9 2023, 10:06 AM
Herald added a subscriber: imp. · View Herald TranscriptNov 9 2023, 10:06 AM
zlei requested review of this revision.Nov 9 2023, 10:06 AM
dfr accepted this revision.Nov 9 2023, 3:46 PM
This revision is now accepted and ready to land.Nov 9 2023, 3:46 PM
dab accepted this revision.Nov 10 2023, 2:04 AM
jhb accepted this revision.Nov 15 2023, 4:08 PM
jhb added a subscriber: jhb.
jhb added inline comments.
sys/kern/kern_linker.c
1285–1286

I think this is more readable.

patmaddox added a subscriber: patmaddox.Dec 13 2023, 9:58 PM
Closed by commit rGf43ff3e15c8b: kern linker: Do not unload a module if it has dependants (authored by zlei). · Explain WhyMar 26 2024, 3:57 AM
This revision was automatically updated to reflect the committed changes.
zlei added a commit: rGf43ff3e15c8b: kern linker: Do not unload a module if it has dependants.
Herald added a subscriber: olce. · View Herald TranscriptMar 26 2024, 3:57 AM
zlei marked an inline comment as done.Apr 9 2024, 4:11 AM
zlei added a commit: rGf1994d1eb215: kern linker: Do not unload a module if it has dependants.
zlei added a commit: rGa2f57656620d: kern linker: Do not unload a module if it has dependants.Apr 9 2024, 4:14 AM

Revision Contents
Changeset List

PathSize
sys/
kern/
2 lines

Diff 136214

View Options

sys/kern/kern_linker.c

Loading...