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
msg/async: support of non-block connect in async messenger #5848
Conversation
if (r < 0) { | ||
if (r == -EINPROGRESS || r == -EALREADY) | ||
break; | ||
goto fail; |
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.
plz add level 1 error log above "goto"
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.
OK
a913b67
to
2994073
Compare
LGTM |
Tested in http://pulpito.ceph.com/haomai-2015-11-10_11:24:27-rados-wip-haomai-testing-distro-basic-multi/. Failed jobs 1145091 and 1145178 his http://tracker.ceph.com/issues/13767 |
ready to merge |
@liewegas give a look? It has passed tests. |
* > 0 just break, and wait for event | ||
* < 0 need to goto fail | ||
*/ | ||
int NetHandler::reconnect(const entity_addr_t &addr, int sd) |
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.
doxygen comments are normally put into the header file. and should be started with /**
, see src/mon/Monitor.h for more examples.
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.
ok
Fixes: ceph#12802 Signed-off-by: Jianhui Yuan <zuiwanyuan@gmail.com>
2994073
to
0fd8de3
Compare
msg/async: support of non-block connect in async messenger Reviewed-by: Haomai Wang <haomai@xsky.com>
msg/async: support of non-block connect in async messenger
Fixes: #12802
Signed-off-by: Jianhui Yuan zuiwanyuan@gmail.com