Skip to content

v5.0: btl/sm: avoid using xpmem for fast boxes #9954

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

Conversation

hppritcha
Copy link
Member

xpmem has different behvior than other shared memory support mechanisms.
in particular, any xpmem-attached regions in a process will become invalid
once the exporting process exits.

Under certain circumstances, this behavior can result in SIGBUS errors
during mpi finalize.

Related to #9868

Signed-off-by: Howard Pritchard [email protected]

@hppritcha hppritcha requested a review from gpaulsen February 1, 2022 17:00
@jsquyres jsquyres changed the title btl/sm: avoid using xpmem for fast boxes v5.0: btl/sm: avoid using xpmem for fast boxes Feb 1, 2022
@jsquyres jsquyres added this to the v5.0.0 milestone Feb 1, 2022
@awlauria
Copy link
Contributor

awlauria commented Feb 2, 2022

@hppritcha I think this failed because there is no -x in the commit.

xpmem has different behvior than other shared memory support mechanisms.
in particular, any xpmem-attached regions in a process will become invalid
once the exporting process exits.

Under certain circumstances, this behavior can result in SIGBUS errors
during mpi finalize.

Related to open-mpi#9868

Signed-off-by: Howard Pritchard <[email protected]>
(cherry picked from commit 8bac539)
@hppritcha hppritcha force-pushed the topic/no_xpmem_for_fboxes_v5.0.x branch from 0d5dabc to 7d3f868 Compare February 2, 2022 19:07
@awlauria awlauria merged commit 17b620a into open-mpi:v5.0.x Feb 3, 2022
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.

3 participants