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 <[email protected]>
  • Loading branch information
sydidelot committed Dec 1, 2023
1 parent 9a7b0b8 commit bf1ed58
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 Down Expand Up @@ -969,6 +974,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 bf1ed58

Please sign in to comment.