Skip to content

Commit d619de4

Browse files
author
Ralph Castain
committed
Fix a threadlock when notifying clients of failures
Signed-off-by: Ralph Castain <[email protected]>
1 parent cefcf7d commit d619de4

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

opal/mca/pmix/pmix2x/pmix2x.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -240,9 +240,9 @@ void pmix2x_event_hdlr(size_t evhdlr_registration_id,
240240
size_t n;
241241
opal_pmix2x_event_t *event;
242242

243-
opal_output_verbose(2, opal_pmix_base_framework.framework_output,
244-
"%s RECEIVED NOTIFICATION OF STATUS %d",
245-
OPAL_NAME_PRINT(OPAL_PROC_MY_NAME), status);
243+
opal_output_verbose(2, opal_pmix_base_framework.framework_output,
244+
"%s RECEIVED NOTIFICATION OF STATUS %d",
245+
OPAL_NAME_PRINT(OPAL_PROC_MY_NAME), status);
246246

247247
OPAL_PMIX_ACQUIRE_THREAD(&opal_pmix_base.lock);
248248

opal/mca/pmix/pmix2x/pmix2x_server_south.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -413,9 +413,11 @@ void pmix2x_server_deregister_client(const opal_process_name_t *proc,
413413
(void)strncpy(p.nspace, jptr->nspace, PMIX_MAX_NSLEN);
414414
p.rank = pmix2x_convert_opalrank(proc->vpid);
415415
OPAL_PMIX_CONSTRUCT_LOCK(&lock);
416+
OPAL_PMIX_RELEASE_THREAD(&opal_pmix_base.lock);
416417
PMIx_server_deregister_client(&p, lkcbfunc, (void*)&lock);
417418
OPAL_PMIX_WAIT_THREAD(&lock);
418419
OPAL_PMIX_DESTRUCT_LOCK(&lock);
420+
OPAL_PMIX_ACQUIRE_THREAD(&opal_pmix_base.lock);
419421
break;
420422
}
421423
}

0 commit comments

Comments
 (0)