uma_zalloc_domain: call uma_zalloc_debug in multi-domain path
ClosedPublic
Actions

Authored by vangyzen on Mar 7 2022, 6:59 PM.

Details

Reviewers

markj
rstone

Commits

rG490b09f24006: uma_zalloc_domain: call uma_zalloc_debug in multi-domain path

Summary

It was only called in the non-NUMA and single-domain paths.
Some of its assertions were duplicated in uma_zalloc_domain,
but some things were missed, especially memguard.

MFC after: 1 week
Sponsored by: Dell EMC Isilon

Test Plan

I ran the kyua suite and some internal tests on a multi-domain OneFS node. There were no surprises.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

vangyzen created this revision.Mar 7 2022, 6:59 PM

Herald added subscribers: dab, badger, imp. · View Herald TranscriptMar 7 2022, 6:59 PM

vangyzen requested review of this revision.Mar 7 2022, 6:59 PM

Harbormaster completed remote builds in B44686: Diff 103599.Mar 7 2022, 6:59 PM

vangyzen mentioned this in D34452: uma_zalloc: assert M_NOWAIT ^ M_WAITOK.Mar 7 2022, 6:59 PM

vangyzen edited the test plan for this revision. (Show Details)Mar 10 2022, 7:03 PM

rstone accepted this revision.Mar 10 2022, 8:37 PM

This revision is now accepted and ready to land.Mar 10 2022, 8:37 PM

markj accepted this revision.Mar 14 2022, 3:37 PM

Closed by commit rG490b09f24006: uma_zalloc_domain: call uma_zalloc_debug in multi-domain path (authored by vangyzen). · Explain WhyMar 26 2022, 1:14 AM

This revision was automatically updated to reflect the committed changes.

vangyzen added a commit: rG490b09f24006: uma_zalloc_domain: call uma_zalloc_debug in multi-domain path.

Revision Contents
Changeset List

Path

Size

sys/

vm/

uma_core.c

11 lines

Diff 104221

View Options

uma_zalloc_domain: call uma_zalloc_debug in multi-domain pathClosedPublicActions