KEMBAR78
ENV/super: set `JEMALLOC_SYS_WITH_LG_PAGE` by cho-m · Pull Request #20647 · Homebrew/brew · GitHub
Skip to content

Conversation

cho-m
Copy link
Member

@cho-m cho-m commented Sep 7, 2025

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes? Here's an example.
  • Have you successfully run brew style with your changes locally?
  • Have you successfully run brew typecheck with your changes locally?
  • Have you successfully run brew tests with your changes locally?

64K should enable support for <= 64K. Default was system page size which means binaries only run on 4K. See:


Can see other usage in:

Copy link
Member

@MikeMcQuaid MikeMcQuaid left a comment

Choose a reason for hiding this comment

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

Makes sense, thanks! Can you update the comment so we're using ARM64 consistently inside Homebrew? Thanks!

AArch64 Linux supports up to 64KB page size

Co-authored-by: Mike McQuaid <mike@mikemcquaid.com>
@cho-m cho-m force-pushed the jemalloc-sys-page-size branch from daef8dd to 14d7942 Compare September 8, 2025 19:59
@cho-m
Copy link
Member Author

cho-m commented Sep 8, 2025

Also noting that Meta/Facebook's fork did accept this as default

Whether fork becomes the main repo now that original is archived or someone else takes stewardship is still unknown.

@cho-m cho-m enabled auto-merge September 8, 2025 20:24
@cho-m cho-m added this pull request to the merge queue Sep 8, 2025
Merged via the queue into main with commit 4d36861 Sep 8, 2025
36 of 37 checks passed
@cho-m cho-m deleted the jemalloc-sys-page-size branch September 8, 2025 20:36
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.

4 participants