Skip to content

osc/ucx: One Sided fixes #10126

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 17, 2022
Merged

osc/ucx: One Sided fixes #10126

merged 1 commit into from
Mar 17, 2022

Conversation

MamziB
Copy link
Contributor

@MamziB MamziB commented Mar 15, 2022

  • Failure in window Post and Complete routines
  • Failure in Put with 1 byte for Dynamic windows
  • Failure in Accumulate with noncontig dt
  • Hang in PSCW
  • Flush every attached win in fence for Dynamic win

Signed-off-by: Mamzi Bayatpour [email protected]
Co-authored-by: Tomislav Janjusic [email protected]

@janjust janjust changed the title One Sided fixes: Osc/ucx One Sided fixes Mar 15, 2022
@janjust janjust changed the title Osc/ucx One Sided fixes osc/ucx One Sided fixes Mar 15, 2022
@janjust janjust self-requested a review March 15, 2022 23:28
Copy link
Contributor

@wzamazon wzamazon left a comment

Choose a reason for hiding this comment

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

would you please add osc/ucx: to the title of the commit too?

@janjust
Copy link
Contributor

janjust commented Mar 16, 2022

@wzamazon will do

 - Failure in window Post and Complete routines
 - Failure in Put with 1 byte for Dynamic windows
 - Failure in Accumulate with noncontig dt
 - Hang in PSCW
 - Flush every attached win in fence for dynamic win

Signed-off-by:  Mamzi Bayatpour                 <[email protected]>
Co-authored-by: Tomislav Janjusic               <[email protected]>
@awlauria
Copy link
Contributor

ok to test

@jsquyres jsquyres changed the title osc/ucx One Sided fixes osc/ucx: One Sided fixes Mar 17, 2022
@janjust janjust merged commit b981e34 into open-mpi:master Mar 17, 2022
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