Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add llmalloc #219

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Add llmalloc #219

wants to merge 3 commits into from

Conversation

mjp41
Copy link
Collaborator

@mjp41 mjp41 commented Feb 19, 2025

No description provided.

@mjp41
Copy link
Collaborator Author

mjp41 commented Feb 20, 2025

@akhin I was attempting to add llmalloc to the mimalloc-bench suite of allocators and benchmarks, but it fails to run xmalloc-test benchmark. This is a heavy producer/consumer benchmark with half the threads allocating and sending messages to the second half, which receive the messages and perform deallocations. Is this expected for llmalloc?

@akhin
Copy link

akhin commented Mar 12, 2025

Hi @mjp41 , sorry for late reply. The way xmalloc test works is against llmalloc design. It uses distinct threads for allocations and frees , but never interleaved ops. In llmalloc, freed pointers are never being returned to the original heaps. Therefore it is unable to serve alloc requests by re-using freed pointers and it is constantly forced to grow. ( OTH, grows currently are slow in llmalloc which is in my todo list. ) And eventually gets killed by OOM.

@KuSh KuSh mentioned this pull request Mar 13, 2025
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