-
Notifications
You must be signed in to change notification settings - Fork 421
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
UCT/TCP: Add assert to check that don't send CM data when connected #4559
Conversation
9bd7698
to
5d9c560
Compare
Mellanox CI: UNKNOWN on 25 workers (click for details)Note: the logs will be deleted after 17-Dec-2019
|
Mellanox CI: PASSED on 25 workers (click for details)Note: the logs will be deleted after 17-Dec-2019
|
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
5d9c560
to
0ce9394
Compare
Mellanox CI: FAILED on 25 of 25 workers (click for details)Note: the logs will be deleted after 18-Dec-2019
|
bot:retest |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Mellanox CI: PASSED on 25 workers (click for details)Note: the logs will be deleted after 18-Dec-2019
|
0ce9394
to
80a2251
Compare
@@ -156,6 +156,9 @@ ucs_status_t uct_tcp_cm_send_event(uct_tcp_ep_t *ep, uct_tcp_cm_conn_event_t eve | |||
UCT_TCP_CM_CONN_ACK | | |||
UCT_TCP_CM_CONN_WAIT_REQ)), | |||
"ep=%p", ep); | |||
ucs_assertv(!(ep->ctx_caps & UCS_BIT(UCT_TCP_EP_CTX_TYPE_TX)) || |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why ||, not &&?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if it does not have TX capability (i.e. only RX capability) and it is in CONNECTED state, it has to be able to send CM REQ event to a peer in order to set RX capability for the peer
Mellanox CI: PASSED on 25 workers (click for details)Note: the logs will be deleted after 19-Dec-2019
|
What
Add assert to check that don't send CM data when connected
Why ?
In order to catch potential bug found by #4525 and #4406
How ?
Add assert and fix comment (
them
->they
)