Skip to content

Preferentially use /dev/shm on Linux #5230

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 2 commits into from
Jun 6, 2018

Conversation

hjelmn
Copy link
Member

@hjelmn hjelmn commented Jun 5, 2018

These commits add MCA variables to control where the osc/rdma and osc/sm components store shared memory segments. This variable is set to /dev/shm if it exists. This works around possible performance issues if the session directory is not a tmpfs.

Master will move this support into opal/shmem.

hjelmn added 2 commits June 5, 2018 09:53
This commit adds a new MCA variable to set the location of the backing
store: osc_rdma_backing_directory. The default on Linux has been
changed to use /dev/shm to improve performance in cases where /tmp is
not a tmpfs.

Signed-off-by: Nathan Hjelm <[email protected]>
(cherry picked from commit d0d59b1)
Signed-off-by: Nathan Hjelm <[email protected]>
This commit adds a new MCA variable to set the location of the backing
store: osc_sm_backing_directory. The default on Linux has been
changed to use /dev/shm to improve performance in cases where /tmp is
not a tmpfs.

Signed-off-by: Nathan Hjelm <[email protected]>
(cherry picked from commit e9de425)
Signed-off-by: Nathan Hjelm <[email protected]>
@hjelmn hjelmn added this to the v3.1.1 milestone Jun 5, 2018
@hjelmn hjelmn requested a review from bosilca June 5, 2018 15:55
@jjhursey
Copy link
Member

jjhursey commented Jun 5, 2018

bot:ibm:xl:retest

@open-mpi open-mpi deleted a comment from ibm-ompi Jun 5, 2018
@bwbarrett bwbarrett merged commit 7064237 into open-mpi:v3.1.x Jun 6, 2018
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.

4 participants