Skip to content

coll/libnbc: add recursive doubling algorithm for MPI_Iscan #5744

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

Conversation

mkurnosov
Copy link
Contributor

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

The MCA parameter coll_libnbc_iscan_algorithm was added for dynamic algorithm selection.

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

@mkurnosov
Copy link
Contributor Author

bot:mellanox:retest

@@ -158,6 +158,15 @@ libnbc_register(void)
MCA_BASE_VAR_SCOPE_READONLY,
&libnbc_ibcast_skip_dt_decision);

libnbc_iscan_algorithm = 0;
(void) mca_base_component_var_register(&mca_coll_libnbc_component.super.collm_version,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should probably at some point add an enumerator for the algorithms so they can be selected by name.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hjelmn Ok, I will implement it.

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

The MCA parameter coll_libnbc_iscan_algorithm was added for dynamic algorithm selection.

Signed-off-by: Mikhail Kurnosov <mkurnosov@gmail.com>
@mkurnosov mkurnosov force-pushed the coll-iscan-recursivedoubling branch from 4dbe6ee to 3d43ff0 Compare September 22, 2018 14:11
@hjelmn hjelmn merged commit 88a560f into open-mpi:master Oct 3, 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