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

Conda installation of Ruff not running on aarch64 #13506

Closed
bcaddy opened this issue Sep 24, 2024 · 8 comments · Fixed by conda-forge/ruff-feedstock#224
Closed

Conda installation of Ruff not running on aarch64 #13506

bcaddy opened this issue Sep 24, 2024 · 8 comments · Fixed by conda-forge/ruff-feedstock#224
Labels
bug Something isn't working release Related to the release process

Comments

@bcaddy
Copy link

bcaddy commented Sep 24, 2024

I'm trying to run Ruff on an NVIDIA Grace (i.e. ARM Neoverse v2) CPU and getting the following error:

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

This appears to be a reappearance of the page size issue noted in #3791 and I'm not sure how to fix it. I've installed Ruff with Conda from the Conda-forge channel and I'm on the latest version. Any suggestions?

@MichaReiser
Copy link
Member

Do you know if Ruff installs a pre-built wheel or is it installed from source?

Do you know what page size your system uses?

@bcaddy bcaddy changed the title Ruff not running on aarch64 Conda installation of Ruff not running on aarch64 Sep 24, 2024
@bcaddy
Copy link
Author

bcaddy commented Sep 24, 2024

The page size is 64k and I'm just installing it with conda install -c conda-forge ruff.

I tried with the exact same version of Ruff installed via pip and it works just fine, so the issue is not actually with Ruff but with the version built/installed with Conda; I've updated the title accordingly.

@zanieb
Copy link
Member

zanieb commented Sep 24, 2024

Presumably we need to set this variable over in the conda-forge builds?

@MichaReiser
Copy link
Member

Yes, I can take a look. @zanieb do you know who has access to our conda forge recipe and how it can be changed?

For reference:

# see https://github.com/astral-sh/ruff/issues/3791
# and https://github.com/gnzlbg/jemallocator/issues/170#issuecomment-1503228963
maturin_docker_options: -e JEMALLOC_SYS_WITH_LG_PAGE=16
- target: armv7-unknown-linux-gnueabihf
arch: armv7
- target: s390x-unknown-linux-gnu
arch: s390x
- target: powerpc64le-unknown-linux-gnu
arch: ppc64le
# see https://github.com/astral-sh/ruff/issues/10073
maturin_docker_options: -e JEMALLOC_SYS_WITH_LG_PAGE=16
- target: powerpc64-unknown-linux-gnu
arch: ppc64
# see https://github.com/astral-sh/ruff/issues/10073
maturin_docker_options: -e JEMALLOC_SYS_WITH_LG_PAGE=16
- target: arm-unknown-linux-musleabihf
arch: arm

- target: aarch64-unknown-linux-musl
arch: aarch64
maturin_docker_options: -e JEMALLOC_SYS_WITH_LG_PAGE=16
- target: armv7-unknown-linux-musleabihf
arch: armv7

@MichaReiser MichaReiser added the release Related to the release process label Sep 25, 2024
@zanieb
Copy link
Member

zanieb commented Sep 25, 2024

Anyone can open a pull request https://github.com/conda-forge/ruff-feedstock

@MichaReiser
Copy link
Member

@bcaddy would you be interested in PRing a fix? It's on my todo list but it probably takes me a while before I get to and I have no prior conda forge knowledge

@bcaddy
Copy link
Author

bcaddy commented Sep 26, 2024

I also have no experience with conda forge and don't see an obvious place to change the jemalloc setting so I opened an issue on the ruff feedstock, hopefully they can help out.

conda-forge/ruff-feedstock#222

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working release Related to the release process
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants