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.

Sign up for GitHub

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

Jump to bottom

Disable Shared Windows when using OpenMPI #282

Merged
merged 7 commits into from
Feb 16, 2017
Merged

Disable Shared Windows when using OpenMPI #282

merged 7 commits into from
Feb 16, 2017

Conversation

devreal
Copy link
Member

@devreal devreal commented Feb 14, 2017 •
edited
Loading

  1. Temporarily disable shared windows if OpenMPI is detected because MPI_Win_allocate_shared returns memory with broken alignment.
  2. Fix the buddy allocator to handle minimum 8 byte allocations. Note that this has no effect on memory alignment (the memory is still allocated through MPI_Win_allocate_shared) but reduces meta data overhead from 2x to 1/4x.

Hotfix for #280

@fuchsto fuchsto merged commit 10e3dca into development Feb 16, 2017
@devreal devreal mentioned this pull request Feb 17, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fuchsto fuchsto fuchsto approved these changes

Assignees

@fuchsto fuchsto

@devreal devreal

Labels
backend:openmpi backend:openmpi2 bug:external-issue enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@devreal @fuchsto