malloc_domain(): return the allocation size to the caller.
ClosedPublic
Actions

Authored by markj on Jan 8 2020, 9:28 PM.

Details

Reviewers

jeff
rlibby

Commits

rS356555: Change malloc_domain() to return the allocation size to the caller.

Summary

Otherwise malloc type accounting is wrong.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Jan 8 2020, 9:28 PM

Harbormaster completed remote builds in B28556: Diff 66520.Jan 8 2020, 9:28 PM

markj added reviewers: jeff, rlibby.Jan 8 2020, 9:29 PM

markj added a subscriber: kaktus.

lgtm, at least from the vmstat -m perspective

Looks good. Did this fix the bus-sc accounting?

sys/kern/kern_malloc.c
673 ↗	(On Diff #66520)	Too bad the compiler didn't warn about this.

This revision is now accepted and ready to land.Jan 8 2020, 10:23 PM

In D23095#506160, @rlibby wrote:

Looks good. Did this fix the bus-sc accounting?

Yes. I'm not sure about the M_TEMP issue though, I only see a couple of malloc_domainset(M_TEMP) callers and they don't look to be responsible.

Closed by commit rS356555: Change malloc_domain() to return the allocation size to the caller. (authored by markj). · Explain WhyJan 9 2020, 3:02 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rS356555: Change malloc_domain() to return the allocation size to the caller..

Herald added a subscriber: imp. · View Herald TranscriptJan 9 2020, 3:02 PM

Revision Contents
Changeset List

Path

Size

head/

sys/

kern/

kern_malloc.c

15 lines

Diff 66558

View Options

head/sys/kern/kern_malloc.c

malloc_domain(): return the allocation size to the caller.ClosedPublicActions