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/simple: use my addr when setting sock priority #14878

Merged
merged 2 commits into from Apr 30, 2017

Conversation

Projects
None yet
3 participants
@tchaikov
Contributor

tchaikov commented Apr 29, 2017

@tchaikov tchaikov added the bug fix label Apr 29, 2017

@tchaikov tchaikov requested review from liewegas and yuyuyu101 Apr 29, 2017

tchaikov added some commits Apr 29, 2017

msg/simple: use my addr when setting sock priority
actually we always fails to set ToS of listening
socket. 6f1037e reveals this issue.

Fixes: http://tracker.ceph.com/issues/19801
Signed-off-by: Kefu Chai <kchai@redhat.com>
msg/async: refactor NetHandler::set_priority() to sync with msg/simple
so it's sync'ed with Pipe::set_socket_options()

Signed-off-by: Kefu Chai <kchai@redhat.com>
@yuyuyu101

This comment has been minimized.

Member

yuyuyu101 commented Apr 29, 2017

async commit isn't a fix, right?

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Apr 29, 2017

@yuyuyu101 no, it's not. i am not tagging it as a fix either.

@liewegas liewegas added the needs-qa label Apr 29, 2017

@tchaikov

This comment has been minimized.

@tchaikov tchaikov merged commit 45d03d8 into ceph:master Apr 30, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@tchaikov tchaikov deleted the tchaikov:wip-19801 branch Apr 30, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment