Skip to content

Conversation

@antiguru
Copy link
Member

@antiguru antiguru commented Sep 2, 2025

Exposes a feature flag clusterd_malloc_conf that conditionally sets the MALLOC_CONF environment variable for clusterd. An empty value maps to None, so no env symbol will be set.

This allows us to experiment with interesting settings, specifically passing thp:always to enable hughe pages.

@antiguru antiguru force-pushed the enable_thp branch 4 times, most recently from 08fc1f2 to 947520b Compare September 2, 2025 16:03
This enables transparent huge pages in jemalloc for clusterd, if
enable_transparent_hugepages is enabled at the time we create the cluster
replica.

Signed-off-by: Moritz Hoffmann <[email protected]>
@antiguru antiguru changed the title Enable transparent huge pages MALLOC_CONF for clusterd Sep 3, 2025
@antiguru antiguru requested a review from def- September 3, 2025 07:52
@antiguru antiguru marked this pull request as ready for review September 3, 2025 07:52
@antiguru antiguru requested review from a team as code owners September 3, 2025 07:53
Copy link
Contributor

@def- def- left a comment

Choose a reason for hiding this comment

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

Since you've set thp:always in CI, I have triggered a nightly run to see what effect it has: https://buildkite.com/materialize/nightly/builds/13056

@antiguru antiguru marked this pull request as draft September 3, 2025 09:26
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