From 3989a525d727e0b60316646fdd426bde916f11fc Mon Sep 17 00:00:00 2001 From: mengdilin Date: Tue, 3 Sep 2024 11:30:48 -0700 Subject: [PATCH] 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 --- .github/actions/build_cmake/action.yml | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/.github/actions/build_cmake/action.yml b/.github/actions/build_cmake/action.yml index 1492e6d691..2606ce9fec 100644 --- a/.github/actions/build_cmake/action.yml +++ b/.github/actions/build_cmake/action.yml @@ -36,7 +36,12 @@ runs: # install base packages for ARM64 if [ "${{ runner.arch }}" = "ARM64" ]; then - conda install -y -q -c conda-forge openblas gxx_linux-aarch64 sysroot_linux-aarch64 + # installing libmamba-solver is necessary for openblas=*=*openmp* + conda install --solver=classic conda-forge::conda-libmamba-solver conda-forge::libmamba conda-forge::libmambapy conda-forge::libarchive + + # installing openblas that was compiled with USE_OPENMP=1. + # Context: https://github.com/facebookresearch/faiss/wiki/Troubleshooting#surprising-faiss-openmp-and-openblas-interaction + conda install -y -q -c conda-forge openblas=*=*openmp* gxx_linux-aarch64 sysroot_linux-aarch64 fi # install base packages for X86_64