libceph: fix messenger retry
authorSage Weil <sage@inktank.com>
Tue, 10 Jul 2012 18:53:34 +0000 (11:53 -0700)
committerSage Weil <sage@inktank.com>
Wed, 18 Jul 2012 02:35:59 +0000 (19:35 -0700)
commit5bdca4e0768d3e0f4efa43d9a2cc8210aeb91ab9
treede2a46ca2bc95e84737f3fe65e715d602b3b9356
parenta018540141a931f5299a866907b27886916b4374
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>
include/linux/ceph/messenger.h
net/ceph/messenger.c