Page MenuHomeFreeBSD

kern_malloc: fold free and zfree together into one __always_inline func
ClosedPublic

Authored by bz on Jul 24 2024, 3:59 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 13, 12:24 PM
Unknown Object (File)
Wed, Nov 12, 4:56 AM
Unknown Object (File)
Sat, Nov 8, 10:03 AM
Unknown Object (File)
Wed, Nov 5, 5:31 AM
Unknown Object (File)
Mon, Nov 3, 3:00 AM
Unknown Object (File)
Wed, Oct 29, 2:11 PM
Unknown Object (File)
Sun, Oct 26, 1:25 PM
Unknown Object (File)
Tue, Oct 21, 1:12 AM
Subscribers

Details

Summary

free() and zfree() are essentially the same copy and pasted code with
the extra explicit_bzero() (and kasan) calls. Add a bool to add the
extra functionality and make both functions a wrapper around the common
code and let the compiler do the optimization based on the bool input
when inlining.

No functional changes intended.

Suggested by: kib (in D45812)
Sponsored by: The FreeBSD Foundation
MFC after: 10 days

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bz requested review of this revision.Jul 24 2024, 3:59 PM
kib added inline comments.
sys/kern/kern_malloc.c
935
This revision is now accepted and ready to land.Jul 24 2024, 4:19 PM
bz marked an inline comment as done.Jul 24 2024, 4:20 PM

Done locally.

This revision now requires review to proceed.Jul 24 2024, 4:23 PM
sys/kern/kern_malloc.c
944

Hmm, I'm not sure why I put these kasan_mark() calls in zfree(), but they are unnecessary and can be removed.

Remove kasan_mark calls as per @markj

bz marked an inline comment as done.Jul 25 2024, 12:56 PM
This revision is now accepted and ready to land.Jul 25 2024, 12:56 PM