diff options
| author | Sage Weil <sage@inktank.com> | 2012-07-10 20:53:34 +0200 |
|---|---|---|
| committer | Sage Weil <sage@inktank.com> | 2012-07-18 04:35:59 +0200 |
| commit | 5bdca4e0768d3e0f4efa43d9a2cc8210aeb91ab9 (patch) | |
| tree | de2a46ca2bc95e84737f3fe65e715d602b3b9356 /net/netlink | |
| parent | Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/net (diff) | |
| download | linux-5bdca4e0768d3e0f4efa43d9a2cc8210aeb91ab9.tar.xz linux-5bdca4e0768d3e0f4efa43d9a2cc8210aeb91ab9.zip | |
libceph: fix messenger retry
In ancient times, the messenger could both initiate and accept connections.
An artifact if that was data structures to store/process an incoming
ceph_msg_connect request and send an outgoing ceph_msg_connect_reply.
Sadly, the negotiation code was referencing those structures and ignoring
important information (like the peer's connect_seq) from the correct ones.
Among other things, this fixes tight reconnect loops where the server sends
RETRY_SESSION and we (the client) retries with the same connect_seq as last
time. This bug pretty easily triggered by injecting socket failures on the
MDS and running some fs workload like workunits/direct_io/test_sync_io.
Signed-off-by: Sage Weil <sage@inktank.com>
Diffstat (limited to 'net/netlink')
0 files changed, 0 insertions, 0 deletions