From c24b1ff14f2110b19664969532d7da39a2fa34be Mon Sep 17 00:00:00 2001 From: Nusrat Islam Date: Wed, 26 Jun 2024 16:56:10 -0500 Subject: [PATCH] graph: fix minNchannels for multi-node overwrite (#1230) [ROCm/rccl commit: b09ea29d66ab5620543ee06e03cc6b9aa4b997f1] --- projects/rccl/src/graph/connect.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/projects/rccl/src/graph/connect.cc b/projects/rccl/src/graph/connect.cc index de3e992de3..c704205f4c 100644 --- a/projects/rccl/src/graph/connect.cc +++ b/projects/rccl/src/graph/connect.cc @@ -674,7 +674,7 @@ ncclResult_t ncclTopoPostset(struct ncclComm* comm, int* firstRanks, int* treePa int minNchannels = ncclMinNchannels(); if (comm->nNodes > 1) { - minNchannels = std::min(64, maxChannels); + minNchannels = std::min(64, minNchannels); } if (comm->nRanks < 8 && 64 < minNchannels) { minNchannels = 2;