Page MenuHomeFreeBSD
Differential D58316

malloc: Fix domainset usage in malloc_domainset() for large allocs
AcceptedPublic
Actions

Authored by markj on Fri, Jul 17, 9:13 PM.

Details

Reviewers
alc
rlibby
Summary

We should of course pass the provided domainset rather than copying what
plain malloc() does.

Fixes: 89deca0a3361 ("malloc: make malloc_large closer to standalone")

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 74903
Build 71786: arc lint + arc unit

Event Timeline

markj created this revision.Fri, Jul 17, 9:13 PM
Herald added subscribers: olce, imp. · View Herald TranscriptFri, Jul 17, 9:13 PM
markj requested review of this revision.Fri, Jul 17, 9:13 PM
markj added a parent revision: D58272: malloc: Refactor redzone and KASAN handling.
Harbormaster completed remote builds in B74903: Diff 182160.Fri, Jul 17, 9:13 PM
rlibby accepted this revision.Fri, Jul 17, 9:22 PM

Maybe one day we should plumb it for malloc_dbg / memguard_alloc too.

This revision is now accepted and ready to land.Fri, Jul 17, 9:22 PM
markj added a comment.Fri, Jul 17, 9:28 PM
In D58316#1337411, @rlibby wrote:

Maybe one day we should plumb it for malloc_dbg / memguard_alloc too.

Do you use memguard at all? I haven't needed to reach for it since KASAN was added.

rlibby added a comment.Fri, Jul 17, 9:39 PM
In D58316#1337420, @markj wrote:
In D58316#1337411, @rlibby wrote:

Maybe one day we should plumb it for malloc_dbg / memguard_alloc too.

Do you use memguard at all? I haven't needed to reach for it since KASAN was added.

Yes, though maybe there's a better way to do what we're doing. memguard can be used with only small performance impact for types/zones that don't match vm.memguard.desc. Actually I was recently thinking about a way to use memguard more broadly for infrequent allocations.

Revision Contents
Changeset List

PathSize
sys/
kern/
2 lines

Diff 182160

View Options

sys/kern/kern_malloc.c

Loading...