Skip to content

Conversation

@ggouaillardet
Copy link
Contributor

…number of hosts

This is a port of the v1.10 serires one-off open-mpi/ompi-release@8c5ce45

@ggouaillardet
Copy link
Contributor Author

@rhc54 can you please review this ?

that fixes the issue described in http://www.open-mpi.org/community/lists/devel/2015/12/18455.php

with the current machinefile

localhost slots=2

mpirun --machinefile machinefile -np 3 ... works without any warning without this commit.
with this commit, it aborts because of oversubscription (same behaviour as v1.10)

@rhc54
Copy link
Contributor

rhc54 commented Dec 25, 2015

This looks correct, but don't we also need the changes to the other mappers as well?

@ggouaillardet
Copy link
Contributor Author

I guess so, will do from Monday

@ggouaillardet ggouaillardet force-pushed the topic/rmaps_rr_oversubsription branch from 51b9c09 to 352b05a Compare December 28, 2015 01:39
@ggouaillardet
Copy link
Contributor Author

@rhc54 i back ported the other changes

@rhc54
Copy link
Contributor

rhc54 commented Dec 28, 2015

@ggouaillardet Thanks!

rhc54 pushed a commit that referenced this pull request Dec 28, 2015
…ption

rmaps/round_robin: warn if oversubscribing when manually setting the …
@rhc54 rhc54 merged commit b7b4231 into open-mpi:master Dec 28, 2015
jsquyres added a commit to jsquyres/ompi that referenced this pull request Aug 23, 2016
patcher/overwrite: remove warning message
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