Skip to content

Conversation

@TheRealMDoerr
Copy link
Member

@TheRealMDoerr TheRealMDoerr commented Nov 24, 2025

Make glibc malloc arenas configurable, but use only 1 by default. See Issue below.

fixes #2109

@SapMachine
Copy link
Member

Hello @TheRealMDoerr, I'm sorry, this pull request doesn't meet the expectations. The pull request description has an invalid format. Please have a look at https://github.com/SAP/SapMachine/wiki/Formal-Requirements-of-Pull-Requests .

Copy link
Member

Choose a reason for hiding this comment

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

From my current investigation the malloc arena is 64MB on Linux x86_64.

Copy link
Member

Choose a reason for hiding this comment

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

Seems this can vary, based on distro, glibc version, CPUs etc. etc. .

\
/* SapMachine 2025-11-24: Configurable limit of malloc arenas */ \
product(int, GlibcMallocArenas, 1, \
"Limit glibc malloc arenas, 0 means use OS default") \
Copy link
Member

Choose a reason for hiding this comment

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

maybe add - 'our default is only one arena for minimum memory use.'

@TheRealMDoerr
Copy link
Member Author

Thanks for reviewing! I've improved the description and the comment.

@SAP SAP deleted a comment from SapMachine Nov 24, 2025
@RealCLanger
Copy link
Member

retest this please

@SapMachine
Copy link
Member

Hello @TheRealMDoerr, this pull request fulfills all formal requirements.

Copy link
Member

@RealCLanger RealCLanger left a comment

Choose a reason for hiding this comment

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

LGTM. Please squash when merging.

@TheRealMDoerr TheRealMDoerr merged commit f481a75 into sapmachine Nov 25, 2025
101 of 102 checks passed
@TheRealMDoerr TheRealMDoerr deleted the 2109_GlibcMallocArenas branch November 25, 2025 13:35
TheRealMDoerr added a commit that referenced this pull request Nov 25, 2025
…be possible (#2110)

Introduction of switch "GlibcMallocArenas" with default value 1 for minimized memory utilization.
RealCLanger pushed a commit that referenced this pull request Nov 27, 2025
…be possible (#2110)

Introduction of switch "GlibcMallocArenas" with default value 1 for minimized memory utilization.
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.

Linux: Configuration of glibc malloc arenas should be possible

6 participants