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
openblas: apply patch to prevent SIGILL on aarch64 #117004
openblas: apply patch to prevent SIGILL on aarch64 #117004
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would refer to patch by the commit hash
/rebase staging |
The applied patch contains the changes from this PR: OpenMathLib/OpenBLAS#3060 This patch prevents gcc from reordering instructions in a way that can cause crashes with SIGILL on aarch64. This issue crashes python when importing numpy on aarch64: numpy/numpy#18131
Co-authored-by: Dmitry Kalinkin <dmitry.kalinkin@gmail.com>
7d472d8
to
0226220
Compare
Rebased, please reopen the pull request to restart CI |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just curious: why does the output contain an empty bin directory?
This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 117004
run on x86_64-linux 1
1 package built:
- openblas
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I don't know. It's there on both |
The updated version of nixpkgs includes NixOS/nixpkgs#117004 so that TensorFlow should now work with the openblas version from nixpkgs on aarch64 as well.
The applied patch contains the changes from this PR:
OpenMathLib/OpenBLAS#3060
This patch prevents gcc from reordering instructions in a way
that can cause crashes with SIGILL on aarch64.
This issue crashes python when importing numpy on aarch64:
numpy/numpy#18131
Motivation for this change
Without this
python -c "import numpy"
crashes withSIGILL
onaarch64
.Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)