Skip to content

Commit

Permalink
Build SVE CI with openblas that was compiled with USE_OPENMP=1 (faceb…
Browse files Browse the repository at this point in the history
…ookresearch#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`

Pull Request resolved: facebookresearch#3776

Reviewed By: ramilbakhshyiev

Differential Revision: D61856775

Pulled By: mengdilin

fbshipit-source-id: 950bd68ba438d221b39d25b2d6e185bc61512243
  • Loading branch information
mengdilin authored and facebook-github-bot committed Sep 3, 2024
1 parent ca1ab78 commit 5e61450
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion .github/actions/build_cmake/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 5e61450

Please sign in to comment.