Skip to content

Commit 59eafeb

Browse files
author
Ralph Castain
authored
Merge pull request #2776 from rhc54/topic/fix
Add missing flag set to ensure nodes do not get double-added to job map.
2 parents 164fc64 + 639cdd4 commit 59eafeb

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

orte/mca/grpcomm/base/grpcomm_base_stubs.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -401,10 +401,10 @@ static int create_dmns(orte_grpcomm_signature_t *sig,
401401
*dmns = NULL;
402402
return ORTE_ERR_NOT_FOUND;
403403
}
404-
opal_output_verbose(5, orte_grpcomm_base_framework.framework_output,
404+
OPAL_OUTPUT_VERBOSE((5, orte_grpcomm_base_framework.framework_output,
405405
"%s sign: GETTING PROC OBJECT FOR %s",
406406
ORTE_NAME_PRINT(ORTE_PROC_MY_NAME),
407-
ORTE_NAME_PRINT(&sig->signature[n]));
407+
ORTE_NAME_PRINT(&sig->signature[n])));
408408
if (NULL == (proc = (orte_proc_t*)opal_pointer_array_get_item(jdata->procs, sig->signature[n].vpid))) {
409409
ORTE_ERROR_LOG(ORTE_ERR_NOT_FOUND);
410410
OPAL_LIST_DESTRUCT(&ds);

orte/mca/odls/base/odls_base_default_fns.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -462,6 +462,7 @@ int orte_odls_base_default_construct_child_list(opal_buffer_t *buffer,
462462
/* add the node to the map, if not already there */
463463
if (!ORTE_FLAG_TEST(dmn->node, ORTE_NODE_FLAG_MAPPED)) {
464464
OBJ_RETAIN(dmn->node);
465+
ORTE_FLAG_SET(dmn->node, ORTE_NODE_FLAG_MAPPED);
465466
opal_pointer_array_add(jdata->map->nodes, dmn->node);
466467
if (newmap) {
467468
jdata->map->num_nodes++;

0 commit comments

Comments
 (0)