Skip to content

Commit bc080d4

Browse files
committed
Turn on compile with NVSHMEM
ghstack-source-id: c745e1a Pull Request resolved: #154538
1 parent 241f8dc commit bc080d4

File tree

3 files changed

+9
-10
lines changed

3 files changed

+9
-10
lines changed

CMakeLists.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -265,6 +265,8 @@ cmake_dependent_option(USE_RCCL "Use RCCL" ON USE_NCCL OFF)
265265
cmake_dependent_option(USE_STATIC_NCCL "Use static NCCL" OFF "USE_NCCL" OFF)
266266
cmake_dependent_option(USE_SYSTEM_NCCL "Use system-wide NCCL" OFF "USE_NCCL"
267267
OFF)
268+
cmake_dependent_option(USE_NVSHMEM "Use NVSHMEM" ON
269+
"USE_CUDA OR USE_ROCM;UNIX;NOT APPLE" OFF)
268270
option(USE_NNAPI "Use NNAPI" OFF)
269271
option(USE_NNPACK "Use NNPACK" ON)
270272
cmake_dependent_option(USE_NUMA "Use NUMA. Only available on Linux." ON "LINUX"

caffe2/CMakeLists.txt

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -985,22 +985,18 @@ elseif(USE_CUDA)
985985
target_compile_definitions(torch_cuda PRIVATE USE_NCCL)
986986
endif()
987987

988-
# Use env var for these for now for prototyping purposes
989-
set(USE_NVSHMEM $ENV{USE_NVSHMEM} CACHE BOOL "Whether to build with NVSHMEM support")
988+
# Compile with NVSHMEM
989+
# Default value of `USE_NVSHMEM` is set in CMakeLists.txt under root, to ON.
990990
# If user has specified NVSHMEM_HOME, we use it;
991991
# Otherwise, NVSHMEM_HOME is auto detected in tools/setup_helpers/cmake.py
992992
if($ENV{NVSHMEM_HOME})
993993
set(NVSHMEM_HOME $ENV{NVSHMEM_HOME} CACHE PATH "Path to NVSHMEM build dir")
994+
else()
995+
message(WARNING "NVSHMEM_HOME not found. Building without NVSHMEM.")
994996
endif()
995997

996-
if(USE_NVSHMEM AND NOT DEFINED NVSHMEM_HOME)
997-
message(WARNING "USE_NVSHMEM set to 1 but NVSHMEM_HOME not found. Please run `pip install nvidia-nvshmem-<version>`, or set NVSHMEM_HOME to the NVSHMEM build dir")
998-
# Disable nvshmem if NVSHMEM_HOME is not found
999-
set(USE_NVSHMEM FALSE CACHE BOOL "Whether to build with NVSHMEM support")
1000-
endif()
1001-
1002-
if(USE_NVSHMEM)
1003-
message("Building with NVSHMEM support: '${NVSHMEM_HOME}'")
998+
if(USE_NVSHMEM AND DEFINED NVSHMEM_HOME)
999+
message(STATUS "Building with NVSHMEM support: '${NVSHMEM_HOME}'")
10041000
set(NVSHMEM_INCLUDE_DIR "${NVSHMEM_HOME}/include")
10051001
set(NVSHMEM_LIB_DIR "${NVSHMEM_HOME}/lib")
10061002

cmake/Summary.cmake

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -172,6 +172,7 @@ function(caffe2_print_configuration_summary)
172172
if(${USE_NCCL})
173173
message(STATUS " USE_SYSTEM_NCCL : ${USE_SYSTEM_NCCL}")
174174
endif()
175+
message(STATUS " USE_NVSHMEM : ${USE_NVSHMEM}")
175176
message(STATUS " USE_NNPACK : ${USE_NNPACK}")
176177
message(STATUS " USE_NUMPY : ${USE_NUMPY}")
177178
message(STATUS " USE_OBSERVERS : ${USE_OBSERVERS}")

0 commit comments

Comments
 (0)