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 experimental jemalloc support #367

Merged
merged 1 commit into from
Jan 5, 2024
Merged

Add experimental jemalloc support #367

merged 1 commit into from
Jan 5, 2024

Conversation

johha
Copy link
Contributor

@johha johha commented Jan 5, 2024

Jemalloc is an alternative malloc implementation which improves memory fragmentation.
It also reduces the overall memory consumption and thus might be beneficial when running on puma.
Further information can be found in this article about malloc and jemalloc: https://www.speedshop.co/2017/12/04/malloc-doubles-ruby-memory.html

This change adds the jemalloc packages and introduces the config property cc.experimental.use_jemalloc_memory_allocator to enable jemalloc (default is false).

  • I have viewed signed and have submitted the Contributor License Agreement

  • I have made this pull request to the develop branch

  • I have run CF Acceptance Tests on bosh lite

Jemalloc is an alternative malloc implementation which improves memory fragmentation.
It also reduces the overall memory consumption and thus might be beneficial when running on puma.
Further information can be found in this article about malloc and jemalloc: https://www.speedshop.co/2017/12/04/malloc-doubles-ruby-memory.html

This change adds the jemalloc packages and introduces the config
property `cc.experimental.use_jemalloc_memory_allocator` to enable jemalloc (default is false).
@johha johha requested review from a team January 5, 2024 13:23
@johha johha merged commit 62c5d07 into develop Jan 5, 2024
2 checks passed
@johha johha deleted the add-jemalloc branch January 5, 2024 13:32
johha added a commit that referenced this pull request Feb 7, 2024
Some jobs consume the linked property `use_jemalloc_memory_allocator` to
enable jemalloc but don't specify the required package in their spec
file. Resulting in jemalloc not being used.
Thus the missing packages need be added.

Related to commit 62c5d07/ PR #367
where we added experimental jemalloc support for all ccng based jobs.
johha added a commit that referenced this pull request Feb 7, 2024
Some jobs consume the linked property `use_jemalloc_memory_allocator` to
enable jemalloc but don't specify the required package in their spec
file. Resulting in jemalloc not being used.
Thus the missing packages need be added.

Related to commit 62c5d07/ PR #367
where we added experimental jemalloc support for all ccng based jobs.
johha added a commit that referenced this pull request Feb 7, 2024
Some jobs consume the linked property `use_jemalloc_memory_allocator` to
enable jemalloc but don't specify the required package in their spec
file. Resulting in jemalloc not being used.
Thus the missing packages need be added.

Related to commit 62c5d07/ PR #367
where we added experimental jemalloc support for all ccng based jobs.
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.

2 participants