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
msgr: set close on exec flag #9772
Conversation
lgtm |
@@ -63,6 +63,11 @@ int Accepter::bind(const entity_addr_t &bind_addr, const set<int>& avoid_ports) | |||
return -errno; | |||
} | |||
|
|||
if (set_close_on_exec(listen_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.
please move this change to another commit.
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.
done.
2d3a014
to
10002c8
Compare
@ukernel do you think if mds "respawn" is an unusual case or not? if yes, i will file a tracker ticket and backport this changeset to jewel after it gets merged. |
10002c8
to
90069ce
Compare
90069ce
to
516413c
Compare
mds execv() when handling the "respawn" command, to avoid fd leakage, and enormous CLOSE_WAIT connections after respawning, we need to set FD_CLOEXEC flag for the socket fds. Fixes: http://tracker.ceph.com/issues/16390 Signed-off-by: Kefu Chai <kchai@redhat.com>
mds execv() when handling the "respawn" command, to avoid fd leakage, and enormous CLOSE_WAIT connections after respawning, we need to set FD_CLOEXEC flag for the socket fds. Fixes: http://tracker.ceph.com/issues/16390 Signed-off-by: Kefu Chai <kchai@redhat.com>
516413c
to
f019ad5
Compare
rebased against master |
mds execv() when handling the "respawn" command, to avoid fd leakage,
and enormous CLOSE_WAIT connections after respawning, we need to set
FD_CLOEXEC flag for the socket fds.
Signed-off-by: Kefu Chai kchai@redhat.com