Skip to content

Commit

Permalink
prov/verbs: Report EQ error when connection establishment fails
Browse files Browse the repository at this point in the history
Fix two code paths where some connection establishment may fail
and we don't report the error to the EQ.

Signed-off-by: Sylvain Didelot <sdidelot@ddn.com>
  • Loading branch information
sydidelot committed Nov 20, 2023
1 parent f7f150b commit 5e0a062
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions prov/verbs/src/verbs_eq.c
Original file line number Diff line number Diff line change
Expand Up @@ -940,6 +940,11 @@ vrb_eq_cm_process_event(struct vrb_eq *eq,
ofi_genlock_lock(&vrb_ep2_progress(ep)->ep_lock);
ret = vrb_eq_addr_resolved_event(ep);
ofi_genlock_unlock(&vrb_ep2_progress(ep)->ep_lock);
if (ret != -FI_EAGAIN) {
eq->err.err = -ret;
eq->err.prov_errno = ret;
goto err;
}
goto ack;

case RDMA_CM_EVENT_ROUTE_RESOLVED:
Expand All @@ -966,6 +971,11 @@ vrb_eq_cm_process_event(struct vrb_eq *eq,
ret = -FI_EAGAIN;
}
ofi_genlock_unlock(&vrb_ep2_progress(ep)->ep_lock);
if (ret != -FI_EAGAIN) {
eq->err.err = -ret;
eq->err.prov_errno = ret;
goto err;
}
goto ack;
case RDMA_CM_EVENT_CONNECT_REQUEST:
*event = FI_CONNREQ;
Expand Down

0 comments on commit 5e0a062

Please sign in to comment.