A pair of UMA optimizations to improve cache behavior.
ClosedPublic
Actions

Authored by jeff on Feb 24 2020, 11:26 PM.

Details

Reviewers

rlibby
markj

Commits

rS358377: A pair of performance improvements.

Summary

This adds a missing swap to uma_zfree to protect lifo behavior. The alloc bucket is now always the most recent data. This was +8% on my test.

It changes the load balancing algorithm for ROUNDROBIN to more strongly prefer the current domain unless there is a severe imbalance. I don't recall the actual % gain but it was not insignificant.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jeff created this revision.Feb 24 2020, 11:26 PM

Harbormaster completed remote builds in B29584: Diff 68779.Feb 24 2020, 11:26 PM

jeff edited the summary of this revision. (Show Details)Feb 24 2020, 11:39 PM

jeff added reviewers: rlibby, markj.

jeff set the repository for this revision to rS FreeBSD src repository - subversion.

Herald added a subscriber: imp. · View Herald TranscriptFeb 24 2020, 11:39 PM

markj added inline comments.Feb 25 2020, 1:52 AM

sys/vm/uma_core.c
586 ↗	(On Diff #68779)	Style: missing spaces around '*'.
587 ↗	(On Diff #68779)	Suppose we have domains 0, 1 with 100 and 99 items, respectively, and pref == 0. Then prefitems is left equal to LONG_MAX and we will return domain 1, but it looks like the intent is to return 0 in this case.