Skip to content

Commit

Permalink
prov/util: Revert util_cntr->ep_list_lock to ofi_mutex
Browse files Browse the repository at this point in the history
util_cntr->ep_list_lock needs to always be a mutex lock to allow msg
endpoints to have a control thread, and claim FI_THREAD_DOMAIN.

Signed-off-by: Seth Zegelstein <[email protected]>
  • Loading branch information
a-szegel authored and j-xiong committed Oct 26, 2023
1 parent b65701a commit 2901c26
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions prov/util/src/util_cntr.c
Original file line number Diff line number Diff line change
Expand Up @@ -390,10 +390,7 @@ int ofi_cntr_init(const struct fi_provider *prov, struct fid_domain *domain,
return ret;
}

ofi_genlock_init(&cntr->ep_list_lock,
cntr->domain->threading == FI_THREAD_DOMAIN ||
cntr->domain->threading == FI_THREAD_COMPLETION ?
OFI_LOCK_NOOP : OFI_LOCK_MUTEX);
ofi_genlock_init(&cntr->ep_list_lock, OFI_LOCK_MUTEX);
ofi_atomic_inc32(&cntr->domain->ref);

/* CNTR must be fully operational before adding to wait set */
Expand Down

0 comments on commit 2901c26

Please sign in to comment.