Skip to content

[release/2.12] rocm-smi deprecation #3434

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.

Sign up for GitHub

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

Open
zjliu-amd wants to merge 1 commit into
base: release/2.12
Choose a base branch
from
+0 −8
Open

[release/2.12] rocm-smi deprecation #3434

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 0 additions & 7 deletions cmake/Dependencies.cmake
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1100,13 +1100,6 @@ if(USE_ROCM)
endif()
endif()

# ROCM-SMI needed to support symmetric memory
if(USE_DISTRIBUTED AND UNIX)
list(APPEND Caffe2_PUBLIC_HIP_DEPENDENCY_LIBS
rocm_smi64
)
endif()

# ---[ Kernel asserts
# Kernel asserts is disabled for ROCm by default.
# It can be turned on by turning on the env USE_ROCM_KERNEL_ASSERT to the build system.
Expand Down
1 change: 0 additions & 1 deletion cmake/public/LoadHIP.cmake
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -223,7 +223,6 @@ if(HIP_FOUND)
if(UNIX)
find_package_and_print_version(rccl)
find_package_and_print_version(hsa-runtime64 REQUIRED)
find_package_and_print_version(rocm_smi REQUIRED)
endif()

# Optional components.
Expand Down