Skip to content

coll/ucc: refactor UCC collective operations to handle MPI_IN_PLACE correctly #13224

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 2, 2025

Conversation

Sergei-Lebedev
Copy link
Contributor

This PR fixes two bugs in coll/ucc component:

  1. src or recv datatypes should be completely ignored when collective is inplace
  2. bigcount was not handled properly

Copy link

github-actions bot commented May 2, 2025

Hello! The Git Commit Checker CI bot found a few problems with this PR:

a650419: coll/ucc: fix bigcount support

  • check_signed_off: does not contain a valid Signed-off-by line

42c7353: coll/ucc: refactor UCC collective operations to ha...

  • check_signed_off: does not contain a valid Signed-off-by line

Please fix these problems and, if necessary, force-push new commits back up to the PR branch. Thanks!

…orrectly

Updated the initialization functions for allgather, allgatherv, alltoall, alltoallv, gather, gatherv, scatter, and scatterv to improve handling of the MPI_IN_PLACE argument. In case of MPI_IN_PLACE for these collectives corresponding datatype and count should be ignored.

Signed-off-by: Sergey Lebedev <[email protected]>
fixing support of bigcount in UCC coll component, coll flags were not set correctly

Signed-off-by: Sergey Lebedev <[email protected]>
@Sergei-Lebedev Sergei-Lebedev force-pushed the topic/fix_ucc_inplace branch from a650419 to 887e7af Compare May 2, 2025 16:50
@janjust
Copy link
Contributor

janjust commented May 2, 2025

@Sergei-Lebedev please cherry pick to v5.0.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants