Skip to content

Commit

Permalink
Need to replace the entire proc in fence with group member
Browse files Browse the repository at this point in the history
Replacing only the nspace leaves the group rank instead of
the proc's nspace rank, so we need to replace the entire
proc entry with the grp member's ID.

Signed-off-by: Ralph Castain <rhc@pmix.org>
  • Loading branch information
rhc54 committed Aug 17, 2022
1 parent d9aa3a0 commit 5ea3328
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/server/pmix_server_ops.c
Original file line number Diff line number Diff line change
Expand Up @@ -931,9 +931,9 @@ pmix_status_t pmix_server_fence(pmix_server_caddy_t *cd, pmix_buffer_t *buf,
rc = PMIX_ERR_BAD_PARAM;
goto cleanup;
}
/* we own the procs array, so just replace the procs nspace
* (which is the group ID) with that of the member */
PMIX_LOAD_NSPACE(procs[n].nspace, grp->members[n].nspace);
/* we own the procs array, so just replace the procs entry
* with that of the member with that group rank */
memcpy(&procs[n], &grp->members[procs[n].rank], sizeof(pmix_proc_t));
}
}
}
Expand Down

0 comments on commit 5ea3328

Please sign in to comment.