Skip to content

Add support for mimalloc#27

Open
MarkusPettersson98 wants to merge 4 commits intomainfrom
mimalloc-feature
Open

Add support for mimalloc#27
MarkusPettersson98 wants to merge 4 commits intomainfrom
mimalloc-feature

Conversation

@MarkusPettersson98
Copy link
Contributor

@MarkusPettersson98 MarkusPettersson98 commented Nov 16, 2025

This PR adds mi-malloc as an alternative memory allocator to libc. This is exposed via the new mimalloc feature flag in the gotatun-cli crate.

I've not played around too much with it yet, but adding the initial infrastructure for experimenting with different memory allocators proved to be rather straightforward.


This change is Reviewable

Copy link
Contributor

@Serock3 Serock3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Serock3 reviewed 6 of 6 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @hulthe)

Copy link
Contributor

@hulthe hulthe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should test whether this allocator also fragments before adding it imo. We wouldn't want to encourage something that doesn't perform well.

@hulthe reviewed 6 of 6 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @MarkusPettersson98)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants