Page MenuHomeFreeBSD
Differential D23977

Add COUNTER_U64_SYSINIT().
ClosedPublic
Actions

Authored by markj on Mar 5 2020, 7:11 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 4, 2:39 PM
Unknown Object (File)
Sat, Oct 3, 7:15 AM
Unknown Object (File)
Tue, Sep 22, 10:58 PM
Unknown Object (File)
Thu, Sep 17, 10:18 AM
Unknown Object (File)
Wed, Sep 16, 4:03 PM
Unknown Object (File)
Tue, Sep 15, 1:51 AM
Unknown Object (File)
Tue, Sep 15, 1:32 AM
Unknown Object (File)
Mon, Sep 14, 6:19 PM
View All Files
Subscribers
imp

Details

Reviewers
kib
glebius
Group Reviewers
manpages
Commits
rS358715: Add COUNTER_U64_SYSINIT() and COUNTER_U64_DEFINE_EARLY().
Summary

This is the same idea as MTX_SYSINIT and friends: register a global
counter for initialization when the counter(9) subsystem is ready,
instead of having to write a custom sysinit function.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj created this revision.Mar 5 2020, 7:11 PM
Harbormaster completed remote builds in B29814: Diff 69234.Mar 5 2020, 7:11 PM
markj added reviewers: kib, glebius.Mar 5 2020, 7:12 PM
kib accepted this revision.Mar 5 2020, 9:25 PM
kib added inline comments.
sys/sys/counter.h
78 ↗(On Diff #69234)

I am somewhat curious about selection of the si_sub there.

This revision is now accepted and ready to land.Mar 5 2020, 9:25 PM
markj added inline comments.Mar 5 2020, 9:31 PM
sys/sys/counter.h
78 ↗(On Diff #69234)

It is just inherited from the existing sysinits which initialize counters. It could be moved earlier, I think the only prerequisite is that the pcpu zones are initialized.

markj updated this revision to Diff 69267.Mar 6 2020, 3:53 PM
  • Add COUNTER_U64_DEFINE_EARLY.
  • Add SI_SUB_COUNTER to make initialization ordering more explicit.
This revision now requires review to proceed.Mar 6 2020, 3:53 PM
Harbormaster completed remote builds in B29827: Diff 69267.Mar 6 2020, 3:53 PM
kib accepted this revision.Mar 6 2020, 4:22 PM
This revision is now accepted and ready to land.Mar 6 2020, 4:22 PM
Closed by commit rS358715: Add COUNTER_U64_SYSINIT() and COUNTER_U64_DEFINE_EARLY(). (authored by markj). · Explain WhyMar 6 2020, 7:09 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rS358715: Add COUNTER_U64_SYSINIT() and COUNTER_U64_DEFINE_EARLY()..
Herald added a reviewer: manpages. · View Herald TranscriptMar 6 2020, 7:09 PM
Herald added a subscriber: imp. · View Herald Transcript

Revision Contents
Changeset List

PathSize
head/
share/
man/
man9/
19 lines
sys/
kern/
18 lines
2 lines
sys/
13 lines
3 lines

Diff 69274

View Options

head/share/man/man9/counter.9

Loading...
View Options

head/sys/kern/subr_counter.c

Loading...
View Options

head/sys/kern/subr_pcpu.c

Loading...
View Options

head/sys/sys/counter.h

Loading...
View Options

head/sys/sys/kernel.h

Loading...