Closed CaRoLZhangxy closed 2 days ago
In the latest update, checks for CUDA support within MPI were integrated into the build system and source code. Conditional logic was added to manage CUDA awareness based on whether MPIX_Query_cuda_support
is available. If unavailable, NO_CUDA_AWARE
is defined, altering configuration and functionality accordingly. This adjustment ensures compatibility and efficient handling of MPI versions with and without CUDA support.
File | Summary |
---|---|
source/op/pt/CMakeLists.txt |
Added a check for MPIX_Query_cuda_support using CheckCXXSymbolExists and defined NO_CUDA_AWARE if absent. Updated linking and compilation definitions for deepmd_op_pt . |
source/op/pt/comm.cc |
Added conditional logic related to CUDA support based on MPI version. Introduced preprocessor directives (NO_CUDA_AWARE , I_MPI_VERSION , MPIX_Query_cuda_support ) and adjusted the Border class accordingly. |
sequenceDiagram
participant CMakeLists.txt
participant Compilation
participant comm.cc
participant MPI
participant Execution
CMakeLists.txt->>+Compilation: Check for MPIX_Query_cuda_support
Compilation-->>CMakeLists.txt: Define NO_CUDA_AWARE if absent
comm.cc-->>comm.cc: Conditional compilation (NO_CUDA_AWARE)
MPI-->>comm.cc: Provide MPI version and CUDA support
Execution->>comm.cc: Instantiate Border class
comm.cc-->>Execution: Adjust behavior based on CUDA awareness
[!TIP]
Early access features: enabled
We are currently testing the following features in early access: - **OpenAI `gpt-4o` model for code reviews and chat**: OpenAI claims that this model is better at understanding and generating code than the previous models. We seek your feedback over the next few weeks before making it generally available. Note: - You can enable or disable early access features from the CodeRabbit UI or by updating the CodeRabbit configuration file. - Please join our [Discord Community](https://discord.com/invite/GsXnASn26c) to provide feedback and report issues. - OSS projects are currently opted into early access features by default.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 82.87%. Comparing base (
e809e64
) to head (98c9937
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I can find MPIX_Query_cuda_support
in mpi.h
, so I don't know what you mean.
grep MPIX_Query_cuda_support ~/intel/oneapi/mpi/2021.13/include/mpi.h
int MPIX_Query_cuda_support(void) MPICH_API_PUBLIC;
int PMPIX_Query_cuda_support(void) MPICH_API_PUBLIC;
grep MPIX_Query_cuda_support ~/intel/oneapi/mpi/2021.13/include/mpi.h
My intel MPI version is 2021.5.1 and there is no definition for MPIX_Query_cuda_support
. What's more, it seems that there is no official api document for intel MPI.
My intel MPI version is 2021.5.1 and there is no definition for
MPIX_Query_cuda_support
.
Perhaps use CMake's CheckSymbolExists
. It seems very hard to check whether this function exists according to other flags.
MPIX_Query_cuda_support()
is not defined in intelMPISummary by CodeRabbit