Skip to content

coll/libnbc: add recursive doubling algorithm for MPI_Iexscan #5718

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
Oct 6, 2018

Conversation

mkurnosov
Copy link
Contributor

Implements recursive doubling algorithm for MPI_Iexscan. The algorithm preserves order of operations so it can be used both by commutative and non-commutative operations.

The MCA parameter coll_libnbc_iexscan_algorithm was added for dynamic algorithm selection.

Signed-off-by: Mikhail Kurnosov mkurnosov@gmail.com

@mkurnosov
Copy link
Contributor Author

I think that more discussion on dynamic algorithm selection in the libnbc is needed. For example: use approach from the coll/tuned (dynamic decisions) in the libnbc or add non-blocking collectives into the coll/tuned.

Implements recursive doubling algorithm for MPI_Iexscan.
The algorithm preserves order of operations so it can be used both
by commutative and non-commutative operations.

The MCA parameter 'coll_libnbc_iexscan_algorithm' was added for dynamic
algorithm selection.

Signed-off-by: Mikhail Kurnosov <mkurnosov@gmail.com>
@mkurnosov mkurnosov force-pushed the coll-iexscan-recursivedoubling branch from d880e17 to dfe203e Compare September 23, 2018 12:54
@mkurnosov mkurnosov force-pushed the coll-iexscan-recursivedoubling branch 2 times, most recently from 1b15cd6 to dfe203e Compare October 5, 2018 14:33
Signed-off-by: Mikhail Kurnosov <mkurnosov@gmail.com>
@mkurnosov
Copy link
Contributor Author

@hjelmn I resolved merge conflicts. This PR is ready for merge.

@hjelmn hjelmn merged commit f11fea0 into open-mpi:master Oct 6, 2018
@awlauria awlauria added this to the v5.0.0 milestone Oct 12, 2020
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.

4 participants