Skip to content

Commit

Permalink
Add "cuda" to MPI backend capabilities
Browse files Browse the repository at this point in the history
Summary: Fixes #109543

Test Plan: We need to run CUDA aware MPI in PyTorch to actually test this change, we currently have no MPI tests.

Differential Revision: D49420438
  • Loading branch information
H-Huang authored and facebook-github-bot committed Sep 19, 2023
1 parent 70f2ada commit e463875
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion test/distributed/test_c10d_common.py
Original file line number Diff line number Diff line change
Expand Up @@ -1625,7 +1625,7 @@ def test_backend_config(self):
backend_config_strings_and_expected_values = [
(dist.Backend.GLOO, "cpu:gloo,cuda:gloo"),
(dist.Backend.NCCL, "cuda:nccl"),
(dist.Backend.MPI, "cpu:mpi"),
(dist.Backend.MPI, "cpu:mpi,cuda:mpi"),
(dist.Backend.UCC, "cpu:ucc,cuda:ucc"),
(dist.Backend.DUMMY, "cpu:dummy,cuda:dummy"),
("DUMMY", "cpu:dummy,cuda:dummy"),
Expand Down
2 changes: 1 addition & 1 deletion torch/distributed/distributed_c10d.py
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ class Backend:
GLOO : ["cpu", "cuda"],
NCCL : ["cuda"],
UCC : ["cpu", "cuda"],
MPI : ["cpu"],
MPI : ["cpu", "cuda"],
}

backend_type_map: Dict[str, ProcessGroup.BackendType] = {
Expand Down

0 comments on commit e463875

Please sign in to comment.