Skip to content

Commit d728380

Browse files
author
Ralph Castain
committed
If job is fully described, there will be no ppn string to unpack
Signed-off-by: Ralph Castain <rhc@open-mpi.org>
1 parent c79eb1d commit d728380

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

orte/mca/odls/base/odls_base_default_fns.c

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -619,17 +619,17 @@ int orte_odls_base_default_construct_child_list(opal_buffer_t *buffer,
619619
}
620620
}
621621

622-
/* extract the ppn regex */
623-
cnt = 1;
624-
if (OPAL_SUCCESS != (rc = opal_dss.unpack(buffer, &ppn, &cnt, OPAL_STRING))) {
625-
ORTE_ERROR_LOG(rc);
626-
goto REPORT_ERROR;
627-
}
628-
629622
/* if the job is fully described, then mpirun will have computed
630623
* and sent us the complete array of procs in the orte_job_t, so we
631624
* don't need to do anything more here */
632625
if (!orte_get_attribute(&jdata->attributes, ORTE_JOB_FULLY_DESCRIBED, NULL, OPAL_BOOL)) {
626+
/* extract the ppn regex */
627+
cnt = 1;
628+
if (OPAL_SUCCESS != (rc = opal_dss.unpack(buffer, &ppn, &cnt, OPAL_STRING))) {
629+
ORTE_ERROR_LOG(rc);
630+
goto REPORT_ERROR;
631+
}
632+
633633
if (!ORTE_PROC_IS_HNP) {
634634
/* populate the node array of the job map and the proc array of
635635
* the job object so we know how many procs are on each node */

0 commit comments

Comments
 (0)