Skip to content

Commit 69a7a00

Browse files
mengdilinfacebook-github-bot
authored andcommitted
Build SVE CI with openblas that was compiled with USE_OPENMP=1 (#3776)
Summary: I noticed by default, conda install openblas installs `libopenblas-pthreads` on our SVE CI. This can be problematic as described in https://github.com/facebookresearch/faiss/wiki/Troubleshooting#surprising-faiss-openmp-and-openblas-interaction Updating installation of openblas to be more specific and use the version that works well with openmp. Sees version `0.3.27-openmp_h1b0c31a_0` for openblas instead of `pthread` Reviewed By: ramilbakhshyiev Differential Revision: D61856775 Pulled By: mengdilin
1 parent 95e0a66 commit 69a7a00

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

.github/actions/build_cmake/action.yml

+6-1
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,12 @@ runs:
3636
3737
# install base packages for ARM64
3838
if [ "${{ runner.arch }}" = "ARM64" ]; then
39-
conda install -y -q -c conda-forge openblas gxx_linux-aarch64 sysroot_linux-aarch64
39+
# installing libmamba-solver is necessary for openblas=*=*openmp*
40+
conda install --solver=classic conda-forge::conda-libmamba-solver conda-forge::libmamba conda-forge::libmambapy conda-forge::libarchive
41+
42+
# installing openblas that was compiled with USE_OPENMP=1.
43+
# Context: https://github.com/facebookresearch/faiss/wiki/Troubleshooting#surprising-faiss-openmp-and-openblas-interaction
44+
conda install -y -q -c conda-forge openblas=*=*openmp* gxx_linux-aarch64 sysroot_linux-aarch64
4045
fi
4146
4247
# install base packages for X86_64

0 commit comments

Comments
 (0)