Page MenuHomeFreeBSD
Differential D23978

Use COUNTER_U64_SYSINIT() in a few places where it simplifies things.
ClosedPublic
Actions

Authored by markj on Mar 5 2020, 7:11 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jul 14, 1:50 AM
Unknown Object (File)
Tue, Jul 7, 1:30 PM
Unknown Object (File)
Mon, Jul 6, 8:11 AM
Unknown Object (File)
Mon, Jul 6, 4:03 AM
Unknown Object (File)
Sun, Jul 5, 9:37 PM
Unknown Object (File)
Thu, Jun 18, 9:23 PM
Unknown Object (File)
May 6 2026, 11:00 PM
Unknown Object (File)
May 3 2026, 11:42 PM
View All Files
Subscribers
imp

Details

Reviewers
kib
Commits
rS358716: Use COUNTER_U64_DEFINE_EARLY() in places where it simplifies things.

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 B29815: Diff 69235.Mar 5 2020, 7:11 PM
markj added a reviewer: kib.Mar 5 2020, 7:12 PM
kib added inline comments.Mar 5 2020, 9:28 PM
sys/vm/vm_object.c
169 ↗(On Diff #69235)

This sequence of declaration/EARLY_COUNTER initialization and SYSINIT repeats very often in your patch. Would it make sense to add e.g. COUNTER_U64_SYSINIT_EARLY that would do everything?

markj added inline comments.Mar 5 2020, 9:34 PM
sys/vm/vm_object.c
169 ↗(On Diff #69235)

That makes sense. Maybe COUNTER_U64_DEFINE_EARLY?

kib added inline comments.Mar 5 2020, 9:41 PM
sys/vm/vm_object.c
169 ↗(On Diff #69235)

Sure.

markj updated this revision to Diff 69268.Mar 6 2020, 3:54 PM

Use COUNTER_U64_DEFINE_EARLY.

Harbormaster completed remote builds in B29828: Diff 69268.Mar 6 2020, 3:54 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 rS358716: Use COUNTER_U64_DEFINE_EARLY() in places where it simplifies things. (authored by markj). · Explain WhyMar 6 2020, 7:10 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rS358716: Use COUNTER_U64_DEFINE_EARLY() in places where it simplifies things..
Herald added a subscriber: imp. · View Herald TranscriptMar 6 2020, 7:10 PM

Revision Contents
Changeset List

PathSize
head/
sys/
kern/
22 lines
29 lines
vm/
13 lines
16 lines
16 lines
21 lines

Diff 69275

View Options

head/sys/kern/subr_smr.c

Loading...
View Options

head/sys/kern/vfs_cache.c

Loading...
View Options

head/sys/vm/swap_pager.c

Loading...
View Options

head/sys/vm/vm_object.c

Loading...
View Options

head/sys/vm/vm_page.c

Loading...
View Options

head/sys/vm/vm_reserv.c

Loading...