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
F110428766: D23978.diff
Tue, Feb 18, 8:26 AM
Unknown Object (File)
Jan 11 2025, 12:08 AM
Unknown Object (File)
Jan 9 2025, 8:29 AM
Unknown Object (File)
Dec 27 2024, 12:45 AM
Unknown Object (File)
Dec 27 2024, 12:44 AM
Unknown Object (File)
Dec 25 2024, 8:56 AM
Unknown Object (File)
Nov 24 2024, 10:23 PM
Unknown Object (File)
Oct 6 2024, 4:10 PM
View All 70 Files
Subscribers
imp

Details

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

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 29828
Build 27654: arc lint + arc unit

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
167

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
167

That makes sense. Maybe COUNTER_U64_DEFINE_EARLY?

kib added inline comments.Mar 5 2020, 9:41 PM
sys/vm/vm_object.c
167

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
sys/
kern/
22 lines
29 lines
vm/
13 lines
16 lines
16 lines
21 lines

Diff 69268

View Options

sys/kern/subr_smr.c

Loading...
View Options

sys/kern/vfs_cache.c

Loading...
View Options

sys/vm/swap_pager.c

Loading...
View Options

sys/vm/vm_object.c

Loading...
View Options

sys/vm/vm_page.c

Loading...
View Options

sys/vm/vm_reserv.c

Loading...