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

🐛 <jemalloc>: Unsupported system page size #1692

Closed
1 task done
vhscom opened this issue Jan 27, 2024 · 9 comments
Closed
1 task done

🐛 <jemalloc>: Unsupported system page size #1692

vhscom opened this issue Jan 27, 2024 · 9 comments

Comments

@vhscom
Copy link

vhscom commented Jan 27, 2024

Environment information

bunx @biomejs/biome range
<jemalloc>: Unsupported system page size
<jemalloc>: Unsupported system page size
memory allocation of 5 bytes failed

What happened?

  1. Clone reproduction which adds Biome to a fresh Bun app.
  2. Run bun install && bun lint to run the Node binary installed as a dev dependency.
  3. Observe <jemalloc> error on aarch64 systems.

Expected result

Formatted code in src directory in accordance with specified Biome configuration. Instead, I see the following error:

Screenshot from 2024-01-27 17-00-15

Biome version: 1.5.3
Bun version: 1.0.21
Node version: 20.10.0
Linux kernel: 6.6.3-411.asahi.fc39.aarch64+16k

Code of Conduct

  • I agree to follow Biome's Code of Conduct
@ematipico
Copy link
Member

You should report the bug in the bun repository, as it aims to have Node.js compatibility. npx works fine.

@ematipico ematipico closed this as not planned Won't fix, can't repro, duplicate, stale Jan 27, 2024
@vhscom
Copy link
Author

vhscom commented Jan 27, 2024

I don't use npx in my workflow. I'll revisit Biome.js compatibility with Bun later. The repro I will leave for now.

@ematipico
Copy link
Member

It was an example. If npx works (or pnpm dlx, yarn dlx), which is based on Node.js, so should bunx.

Jarred explicitly said that incompatibility bugs with Node.js should be reported in the bun repository.

@jnlmmln
Copy link
Contributor

jnlmmln commented Mar 5, 2024

Hi,

I have the same issue using npx:

❯ npx @biomejs/biome format .

<jemalloc>: Unsupported system page size
<jemalloc>: Unsupported system page size
memory allocation of 5 bytes failed

Manually editing node_modules/@biomejs/biome/bin/biome and letting isMusl() always return true, so the @biomejs/cli-linux-arm64-musl/biome gets used is working.

node: v18.19.0
biome: 1.5.3
kernel: 6.6.3-413.asahi.fc39.aarch64+16k

@ematipico
Copy link
Member

Probably your OS doesn't fit the heuristic of the function. Not sure what we should update to match your OS

@jnlmmln
Copy link
Contributor

jnlmmln commented Mar 5, 2024

Thanks for the quick reply.
Reading through some issues the problem should be with jemallocator. Those should be related:

gnzlbg/jemallocator#68
gnzlbg/jemallocator#170


And setting the page size for aarch64 builds to 16 should fix it.

JEMALLOC_SYS_WITH_LG_PAGE=16

https://github.com/astral-sh/ruff/blob/af6ea2f5e4b1702d5062831624492d94f57a0f81/.github/workflows/release.yaml#L247


I would be happy to support on this.

@ematipico
Copy link
Member

Would you like to send a PR?

@jnlmmln
Copy link
Contributor

jnlmmln commented Mar 5, 2024

Sure: #1975

@arendjr arendjr closed this as completed Mar 6, 2024
@jnlmmln
Copy link
Contributor

jnlmmln commented Mar 6, 2024

Just installed 1.5.3-nightly.4fa841c and it works! 🎉

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

No branches or pull requests

4 participants