Adding the script to build and run the rccl-tests for PTS #26
+2,188
−765
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Build with CUDARTLIB=cudart_static to remove dynamic linkage Also removed unused curand and nvToolsExt dependencies BUG 95
Add option to statically link cudart
* Added "verifiable", a suite of kernels for generating and verifying reduction input and output arrays in a bit-precise way. * Data corruption errors now reported in number of wrong elements instead of max deviation. * Use ncclGetLastError. * Don't run hypercube on non-powers of 2 ranks. * Fix to hypercube data verification. * Use "thread local" as the defaut CUDA capture mode. * Replaced pthread_yield -> sched_yield() * Bugfix to the cpu-side barrier/allreduce implementations.
as relative to top-level directory. This done is by abspath'ing it before
passing it to subdirectory Makefile's.
The old behavior had two cases: with and without BUILDDIR being set by
the user. With BUILDDIR not set, the build dir would be named "build"
in the top-level directory. If BUILDDIR was set, then the build dir
would be placed at "src/${BUILDDIR}".
The new behavior is simpler, if BUILDDIR is not set then it defaults
to "build", and the directory holding the final build is always at just
"${BUILDDIR}" in the top level.
AlltoAll does not support in-place buffers
ncclGetLastError() was added in NCCL 2.13.0
all files compile now. mpi tests also pass
Topic/v2.13.4 sync
error introduced with the web merger-resolution tool :-(
fix a messing endif statement
add the rccl/lib directory to the link path
the subdir entry is not actually required for the compilation.
make cmake stage also pass in CI
PedramAlizadeh
requested review from
eidenyoshida,
lawruble13 and
RobsonRLemos
avoid a division by zero which seems to only occur for op=prod and datatype=half, since the maximum exponent is small (15) and can exceed the number of ranks.
fix algorithm assigning values in testsuite
auto-detect and enable MPI
* Adding -pthread flag for linking issues into src/Makefile * Adding -pthread flag for linking issues into CMakeLists.txt
we honor user requested MPI installations using MPI_PATH first, and check afterwards for MPICH and Open MPI in the default Ubuntu and RHEL installation directories.
revamp cmake MPI detection
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.