Skip to content

coll/base: reduce memory consumption in Scatter #8626

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
Mar 16, 2021

Conversation

mkurnosov
Copy link
Contributor

This PR reduces memory consumption in non-root and non-leaf processes of binomial tree algorithm for Scatter operation.

Signed-off-by: Mikhail Kurnosov [email protected]

This PR reduces memory consumption in non-root and non-leaf processes of binomial tree algorithm for Scatter operation.

Signed-off-by: Mikhail Kurnosov <[email protected]>
@mkurnosov
Copy link
Contributor Author

@bosilca @jsquyres this PR is a small optimization for #8383

@jsquyres jsquyres merged commit 1da3f93 into open-mpi:master Mar 16, 2021
@jsquyres
Copy link
Member

@mkurnosov Could you cherry-pick this to relevant release branches?

Thanks!

@mkurnosov
Copy link
Contributor Author

@jsquyres I have cherry-picked a commit to v4.0.x and v4.1.x. What is a preferred way to add this changes to v5.0.x?

@awlauria
Copy link
Contributor

@mkurnosov please cherry-pick to the v5.0.x branch just like you have for v4.x.x branches.

Thanks!

@mkurnosov
Copy link
Contributor Author

@awlauria This PR adds small changes to not yet merged PR #8619.

@mkurnosov
Copy link
Contributor Author

@jsquyres @awlauria Do I have to wait for merging PR #8619 to v5.0.x before cherry-picking or is it possible to add this commit to PR #8619? Thanks.

@jsquyres
Copy link
Member

Looks like #8619 is merged, so I guess you get to create a new 5.0 PR. 😄

@awlauria
Copy link
Contributor

Ah, I didn't see your response @mkurnosov . Had I seen that I would have been fine with you adding it to #8619 if possible. Sorry about that.

@mkurnosov
Copy link
Contributor Author

@jsquyres @awlauria Thanks!

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.

4 participants