Page MenuHomeFreeBSD
Differential D32511

timecounter: Lock the timecounter list
ClosedPublic
Actions

Authored by markj on Oct 15 2021, 8:07 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 31, 11:05 AM
Unknown Object (File)
Sat, Jan 31, 11:05 AM
Unknown Object (File)
Fri, Jan 30, 9:25 PM
Unknown Object (File)
Thu, Jan 29, 11:55 PM
Unknown Object (File)
Dec 27 2025, 2:12 PM
Unknown Object (File)
Dec 27 2025, 9:40 AM
Unknown Object (File)
Dec 12 2025, 6:15 AM
Unknown Object (File)
Dec 7 2025, 10:27 PM
View All Files
Subscribers
None

Details

Reviewers
kib
imp
jhb
Commits
rG621fd9dcb2d8: timecounter: Lock the timecounter list
Summary

Timecounter registration is dynamic, i.e., there is no requirement that
timecounters must be registered during single-threaded boot. Loadable
drivers may in principle register timecounters (which can be switched to
automatically). Timecounters cannot be unregistered, though this could
be implemented.

Registered timecounters belong to a global linked list. Add a mutex to
synchronize insertions and the traversals done by (mpsafe) sysctl
handlers. No functional change intended.

MFC after: 2 weeks
Sponsored by: The FreeBSD Foundation

Diff Detail

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

Event Timeline

markj requested review of this revision.Oct 15 2021, 8:07 PM
markj created this revision.
markj added a parent revision: D32510: x86: Remove some leftover APM support.
Harbormaster completed remote builds in B42155: Diff 96926.Oct 15 2021, 8:07 PM
markj added a child revision: D32512: x86: Implement deferred TSC calibration.Oct 15 2021, 8:08 PM
imp accepted this revision.Oct 15 2021, 8:09 PM
This revision is now accepted and ready to land.Oct 15 2021, 8:09 PM
kib added inline comments.Oct 15 2021, 8:28 PM
sys/kern/kern_tc.c
1550

Doesn't drain function for sysctl sbufs do copyout? It might do it over the wired userspace memory, but I do not think it would help against pagefaults while owning the mutex.

markj added inline comments.Oct 15 2021, 8:31 PM
sys/kern/kern_tc.c
1550

The pages should indeed be wired. Certainly spurious faults are possible, but isn't it legal to hold a mutex there nonetheless? I thought that was the whole point of sbuf_new_for_sysctl(): to permit non-sleepable locks to be held while writing to the output buffer.

kib accepted this revision.Oct 15 2021, 8:35 PM
kib added inline comments.
sys/kern/kern_tc.c
1550

Yes, it uses copyout_nofault() in the end. So tc_lock suddently appears before pmap locks on some arches, ok.

markj marked 2 inline comments as done.Oct 15 2021, 8:51 PM
markj added inline comments.
sys/kern/kern_tc.c
1550

You are right though that an explicit sysctl_wire_old_buffer() call is needed. I thought sbuf_new_for_sysctl() handled it.

markj updated this revision to Diff 96933.Oct 15 2021, 8:52 PM
markj marked an inline comment as done.

Need sysctl_wire_old_buffer()

This revision now requires review to proceed.Oct 15 2021, 8:52 PM
Harbormaster completed remote builds in B42157: Diff 96933.Oct 15 2021, 8:52 PM
kib accepted this revision.Oct 15 2021, 9:17 PM
This revision is now accepted and ready to land.Oct 15 2021, 9:17 PM
Closed by commit rG621fd9dcb2d8: timecounter: Lock the timecounter list (authored by markj). · Explain WhyOct 18 2021, 2:09 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rG621fd9dcb2d8: timecounter: Lock the timecounter list.

Revision Contents
Changeset List

PathSize
sys/
kern/
40 lines

Diff 96933

View Options

sys/kern/kern_tc.c

Loading...