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
F153369922: D23978.id69275.diff
Mon, Apr 20, 6:35 PM
Unknown Object (File)
Wed, Apr 15, 2:06 AM
Unknown Object (File)
Thu, Apr 9, 7:22 PM
Unknown Object (File)
Tue, Apr 7, 1:49 PM
Unknown Object (File)
Tue, Apr 7, 4:48 AM
Unknown Object (File)
Sat, Apr 4, 3:56 PM
Unknown Object (File)
Tue, Mar 24, 4:34 PM
Unknown Object (File)
Mon, Mar 23, 10:33 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...