Skip to content

Conversation

@rhc54
Copy link
Contributor

@rhc54 rhc54 commented Jul 10, 2018

Do not have child jobs inherit launch directives unless requested to do so. This affects the map-by, rank-by, bind-to, npernode, pernode, npersocket, persocket, and cpus-per-rank directives. Values provided in the spawn call always take precedence - if a particular value isn't specified, then the ORTE defaults will be used if inheritance is not requested, and the values specified by MCA param will be used if inheritance is set.

Signed-off-by: Ralph Castain rhc@open-mpi.org

@rhc54
Copy link
Contributor Author

rhc54 commented Jul 10, 2018

Fixes #5376

@bosilca Please see if this does what you wanted.

@rhc54
Copy link
Contributor Author

rhc54 commented Jul 10, 2018

@hoopoepg Looks like something is still wrong with the Mellanox Jenkins environment:

13:33:52 --------------------------------------------------------------------------
13:33:52 No PCI locality information could be found on at least one node. Please, upgrade BIOS to expose NUMA info.
13:33:52 
13:33:52   Node: jenkins03
13:33:52 
13:33:52 Open MPI will map the application by default (BYSLOT).
13:33:52 --------------------------------------------------------------------------

Do not have child jobs inherit launch directives unless requested to do so. This affects the map-by, rank-by, bind-to, npernode, pernode, npersocket, persocket, and cpus-per-rank directives. Values provided in the spawn call always take precedence - if a particular value isn't specified, then the ORTE defaults will be used if inheritance is not requested, and the values specified by MCA param will be used if inheritance is set.

Always inherit oversubscribe for now as otherwise MTT will break

Signed-off-by: Ralph Castain <rhc@open-mpi.org>
@rhc54 rhc54 merged commit 55357ce into open-mpi:master Jul 10, 2018
@rhc54 rhc54 deleted the topic/dyn branch July 10, 2018 23:12
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.

1 participant