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 15, 2023
1 parent 59ed720 commit e0cb324
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 @@ -939,6 +939,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 @@ -965,6 +970,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 e0cb324

Please sign in to comment.