Skip to content

Commit 1c938de

Browse files
[integ-tests] Simplify NCCL test parameters
This is consistent with internal testing recommendations
1 parent 3a47a55 commit 1c938de

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

tests/integration-tests/tests/common/data/nccl/nccl_tests_submit_openmpi.sh

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,12 +13,11 @@ else
1313
OFI_PATH=$(cat /etc/ld.so.conf.d/100_ofinccl.conf)
1414
fi
1515

16+
# -x NCCL_ALGO=ring is not needed after NCCL 2.12. NCCL autodetects which ALGO (ring or tree) to use
17+
# -x FI_EFA_USE_DEVICE_RDMA=1 is not needed from aws nccl pfi plugin version v1.6.0
1618
mpirun \
17-
-x FI_PROVIDER="efa" \
18-
-x FI_EFA_USE_DEVICE_RDMA=1 \
1919
-x LD_LIBRARY_PATH=/shared/openmpi/nccl-${NCCL_VERSION}/build/lib/:${OFI_PATH}:$LD_LIBRARY_PATH \
20-
-x RDMAV_FORK_SAFE=1 \
2120
-x NCCL_DEBUG=WARNING \
2221
-x NCCL_TESTS_SPLIT_MASK=0x0 \
23-
--mca pml ^cm --mca btl tcp,self --mca btl_tcp_if_exclude lo,docker0 --bind-to none \
22+
--bind-to none \
2423
/shared/openmpi/nccl-tests-${NCCL_BENCHMARKS_VERSION}/build/all_reduce_perf -b 1024 -e 8G -f 2 -g 1 -c 1 > /shared/nccl_tests.out

0 commit comments

Comments
 (0)