Skip to content

v4.0.x: schizo/ompi: correctly handle the yield_when_idle option #7168

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
Jan 3, 2020

Conversation

wbailey2
Copy link
Contributor

in schizo/ompi, sets the new OMPI_MCA_mpi_oversubscribe environment
variable according to the node oversubscription state.

This MCA parameter is used to set the default value of the
mpi_yield_when_idle parameter.

This two steps tango is needed so the mpi_yield_when_idle setting
is always honored when set in a config file.

Refs. #6433

Signed-off-by: Gilles Gouaillardet [email protected]

@ompiteam-bot
Copy link

Can one of the admins verify this patch?

@jsquyres jsquyres changed the title schizo/ompi: correctly handle the yield_when_idle option v4.0.x: schizo/ompi: correctly handle the yield_when_idle option Nov 14, 2019
@jsquyres jsquyres added this to the v4.0.3 milestone Nov 14, 2019
@jsquyres
Copy link
Member

ok to test

@gpaulsen
Copy link
Member

@wbailey2, Thanks for the PR. Normally when cherry-picking to a release branch, we require the developer to specify -x to the git cherry-pick command. This provides a reference to the master commit, so that we can more easily audit code flow.
This is fine, this time to leave it since you left the Reference in the commit message, but I wanted to point it out for future Open MPI dev work you may do. :)

Thanks.

@gpaulsen gpaulsen added the NEWS label Nov 22, 2019
in schizo/ompi, sets the new OMPI_MCA_mpi_oversubscribe environment
variable according to the node oversubscription state.

This MCA parameter is used to set the default value of the
mpi_yield_when_idle parameter.

This two steps tango is needed so the mpi_yield_when_idle setting
is always honored when set in a config file.

Refs. open-mpi#6433

Signed-off-by: Gilles Gouaillardet <[email protected]>
(cherry-picked from cc97c0f)
@wbailey2 wbailey2 force-pushed the pr/fix-yield_when_idle branch from ed66b65 to b004f4c Compare December 2, 2019 22:22
@wbailey2
Copy link
Contributor Author

wbailey2 commented Dec 2, 2019

I updated the commit to include where the cherry-pick was from.

@jsquyres
Copy link
Member

jsquyres commented Dec 3, 2019

bot:ompi:retest

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.

5 participants