Page MenuHomeFreeBSD

Add counters to smr.
ClosedPublic

Authored by jeff on Feb 2 2020, 12:25 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 8, 11:16 PM
Unknown Object (File)
Mar 19 2024, 2:06 PM
Unknown Object (File)
Mar 19 2024, 9:04 AM
Unknown Object (File)
Feb 27 2024, 5:44 PM
Unknown Object (File)
Dec 23 2023, 3:44 AM
Unknown Object (File)
Oct 20 2023, 9:11 PM
Unknown Object (File)
Sep 7 2023, 1:16 AM
Unknown Object (File)
Sep 7 2023, 1:14 AM
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.