Skip to content
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

Multiple commits #1491

Merged
merged 3 commits into from
Sep 6, 2022
Merged

Multiple commits #1491

merged 3 commits into from
Sep 6, 2022

Conversation

rhc54
Copy link
Contributor

@rhc54 rhc54 commented Sep 3, 2022

stream-buffering is an OMPI option, so make it an MCA

  • It looks like --stream-buffering CLI was copied from the OMPI schizo
    to others, but is not processed by others.
  • In OMPI make this an MCA parameter, and translate the --stream-buffering
    to that MCA parameter.

Signed-off-by: Joshua Hursey jhursey@us.ibm.com
(cherry picked from commit 1207c64)

Deprecated the --stream-buffering CLI option

Signed-off-by: Joshua Hursey jhursey@us.ibm.com
(cherry picked from commit 1a90bcd)

Fix hetero topology operations

Resolve some race conditions that resulted in hangs
and potentially incorrect topology assignments.

Signed-off-by: Ralph Castain rhc@pmix.org
(cherry picked from commit adc8fd7)

jjhursey and others added 3 commits September 2, 2022 17:30
 * It looks like `--stream-buffering` CLI was copied from the OMPI schizo
   to others, but is not processed by others.
 * In OMPI make this an MCA parameter, and translate the `--stream-buffering`
   to that MCA parameter.

Signed-off-by: Joshua Hursey <jhursey@us.ibm.com>
(cherry picked from commit 1207c64)
Signed-off-by: Joshua Hursey <jhursey@us.ibm.com>
(cherry picked from commit 1a90bcd)
Resolve some race conditions that resulted in hangs
and potentially incorrect topology assignments.

Signed-off-by: Ralph Castain <rhc@pmix.org>
(cherry picked from commit adc8fd7)
@jjhursey
Copy link
Member

jjhursey commented Sep 6, 2022

bot:ibm:retest

@rhc54 rhc54 merged commit 917c4c2 into openpmix:v3.0 Sep 6, 2022
@rhc54 rhc54 deleted the cmr30/up2 branch September 6, 2022 13:42
@jjhursey jjhursey mentioned this pull request Sep 6, 2022
14 tasks
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.

2 participants