Page MenuHomeFreeBSD
Differential D31832

bus: mark sysctls MPSAFE
ClosedPublic
Actions

Authored by imp on Sep 4 2021, 5:08 AM.
Tags
None
Referenced Files
F171223157: D31832.id99803.diff
Wed, Sep 9, 3:29 PM
F171135924: D31832.diff
Tue, Sep 8, 10:35 PM
Unknown Object (File)
Tue, Sep 8, 5:56 AM
Unknown Object (File)
Sat, Sep 5, 10:27 AM
Unknown Object (File)
Sat, Sep 5, 6:02 AM
Unknown Object (File)
Sat, Sep 5, 3:21 AM
Unknown Object (File)
Sat, Sep 5, 1:45 AM
Unknown Object (File)
Fri, Sep 4, 1:47 PM
View All Files
Subscribers
None

Details

Reviewers
mav
jhb
• hselasky
Commits
rG815a0973b23c: bus: mark sysctls MPSAFE
Summary

Mark the sysctls MPSAFE and pickup the bus topo lock while processing
them.

Sponsored by: Netflix

Diff Detail

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

Event Timeline

imp requested review of this revision.Sep 4 2021, 5:08 AM
imp created this revision.
Harbormaster completed remote builds in B41366: Diff 94651.Sep 4 2021, 5:08 AM
imp added reviewers: mav, jhb, • hselasky.Sep 4 2021, 5:09 AM
• hselasky added a comment.Sep 4 2021, 8:52 AM

Looks good execept for the naming of "bus_topo_lock". See other review.

mav accepted this revision.Sep 4 2021, 2:59 PM
This revision is now accepted and ready to land.Sep 4 2021, 2:59 PM
jhb accepted this revision.Nov 29 2021, 6:17 PM
jhb added a parent revision: D31831: Create wrapper for Giant taken for newbus.
jhb added a child revision: D31833: newbus: add bus_topo_assert.
Closed by commit rG815a0973b23c: bus: mark sysctls MPSAFE (authored by imp). · Explain WhyDec 10 2021, 12:06 AM
This revision was automatically updated to reflect the committed changes.
imp added a commit: rG815a0973b23c: bus: mark sysctls MPSAFE.

Revision Contents
Changeset List

PathSize
sys/
kern/
19 lines

Diff 99803

View Options

sys/kern/subr_bus.c

Loading...