Page MenuHomeFreeBSD

Add counters to smr.
ClosedPublic

Authored by jeff on Feb 2 2020, 12:25 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 12, 6:41 PM
Unknown Object (File)
Tue, Dec 3, 10:30 PM
Unknown Object (File)
Oct 5 2024, 7:03 AM
Unknown Object (File)
Oct 5 2024, 5:03 AM
Unknown Object (File)
Oct 4 2024, 8:35 PM
Unknown Object (File)
Oct 2 2024, 10:06 AM
Unknown Object (File)
Oct 1 2024, 3:00 AM
Unknown Object (File)
Sep 26 2024, 9:51 PM
Subscribers

Details

Summary

These should eventually be per-smr I suspect but for now this is helpful.

Diff Detail

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

Event Timeline

jeff added reviewers: rlibby, markj.
jeff set the repository for this revision to rS FreeBSD src repository - subversion.

Logic looks fine. I trust you've checked any performance effects.

In advance you count on entry, and then also on the slow path, whereas in poll you count either fast or slow. Did you mean to approach those differently?

sys/kern/subr_smr.c
168 ↗(On Diff #67646)

Should they be initialized with EARLY_COUNTER? Since it seems that UMA SMR is available after SI_SUB_VM.

This revision is now accepted and ready to land.Feb 2 2020, 8:23 PM
sys/kern/subr_smr.c
39 ↗(On Diff #67646)

Sort includes while here?

316 ↗(On Diff #67646)

This and the call below can be counter_u64_add_protected() since you're in a critical section.

Logic looks fine. I trust you've checked any performance effects.

In advance you count on entry, and then also on the slow path, whereas in poll you count either fast or slow. Did you mean to approach those differently?

It might make more sense to count all polls and then number of polls that scan. I will look at that.