Skip to content

Commit 8bb489d

Browse files
committed
v3.1.x/pmix: fixed the direct modex request
This commit fixes the case when local client asks for the key from the process on the remote node. The local server don't have commit count for remote ranks, it is maintained by another PMIx server, so commit count should be ignored for remote requests. Signed-off-by: Boris Karasev <[email protected]>
1 parent 7848d55 commit 8bb489d

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

opal/mca/pmix/pmix2x/pmix/src/server/pmix_server_get.c

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -610,7 +610,14 @@ static pmix_status_t _satisfy_request(pmix_nspace_t *nptr, pmix_rank_t rank,
610610

611611
/* retrieve the data for the specific rank they are asking about */
612612
if (PMIX_RANK_WILDCARD != rank) {
613-
if (!peer->commit_cnt) {
613+
if (!PMIX_PROC_IS_SERVER(peer) && !peer->commit_cnt) {
614+
/* this condition works only for local requests, server does
615+
* count commits for local ranks, and check this count when
616+
* local request.
617+
* if that request performs for remote rank on the remote
618+
* node (by direct modex) so `peer->commit_cnt` should be ignored,
619+
* it is can not be counted for the remote side and this condition
620+
* does not matter for remote case */
614621
return PMIX_ERR_NOT_FOUND;
615622
}
616623
proc.rank = rank;

0 commit comments

Comments
 (0)