Skip to content

mca/fcoll: read_all Receive data without temp buff by using MPI datatypes #8433

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 1 commit into from
Feb 4, 2021

Conversation

raafatfeki
Copy link
Contributor

Changes are done for vulcan and dynamic_gen2
1- Instead of using a temporary buffer for non contiguous data to receive data from aggregator, we define a derived datatype to describe the received data layout.
2- Remove send/recv operations with empty data.
3- Make common directives in header file

Signed-off-by: raafatfeki [email protected]

…ypes

Changes are done for vulcan and dynamic_gen2
1- Instead of using a temporary buffer for non contiguous data to receive data from aggregator, we define a derived datatype to describe the received data layout.
2- Remove send/recv operations with empty data.
3- Make common directives in header file

Signed-off-by: raafatfeki <[email protected]>
@edgargabriel
Copy link
Member

bot:retest

@jsquyres
Copy link
Member

jsquyres commented Feb 4, 2021

You can ignore the readthedocs CI error -- it's only there for a pending PR (#8329). This PR has nothing to do with readthedocs, so whatever error it caused over there is ignoreable (and it may have been a transient RTD error, anyway).

@raafatfeki
Copy link
Contributor Author

Thanks for the clarification.

@raafatfeki raafatfeki merged commit e894138 into open-mpi:master Feb 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants