Page MenuHomeFreeBSD

Refactor uma_zfree_arg()
ClosedPublic

Authored by jeff on Nov 21 2019, 10:40 PM.

Details

Summary

This is a more significant refactoring than uma_zalloc_arg() because it was in worse shape. We only need the zone lock if we're going to cache a bucket. The crossdomain logic really needed to be simplified and merged with the other case. Fundamentally we're just freeing a bucket and allocating a bucket, no matter which bucket that is.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

jeff added reviewers: markj, glebius, kib, cem.

The bucket_push and item_dtor refactors look reasonable. I'm not super familiar with this area of code.

sys/vm/uma_core.c
3214 ↗(On Diff #64701)

Missing space after while.

3311 ↗(On Diff #64701)

Isn't the use of lockfail redundant? We can write:

if (!ZONE_TRYLOCK(zone)) {
    ZONE_LOCK(zone);
    if (zone->uz_count < zone->uz_count_max)
        ....
}
jeff edited the summary of this revision. (Show Details)

Review feedback. Pull another function out, zone_free_bucket().

markj added inline comments.
sys/vm/uma_core.c
3317 ↗(On Diff #64910)

Missing parens.

This revision is now accepted and ready to land.Nov 27 2019, 5:01 PM