Noah:
Branch wip-messenger contains the patch you want (plus one or two
cleanups), or apply the below. Should handle it, although I'd like to
do a more thorough cleanup of this problem at a later time.
-Greg
>From 8c4f4748e8b683f5b4ea939295793421c0ab7b61 Mon Sep 17 00:00:00 2001
From: Greg Farnum <gregory.farnum@xxxxxxxxxxxxx>
Date: Wed, 7 Dec 2011 14:19:23 -0800
Subject: [PATCH] mds: mark_disposable when closing a Client connection.
This is causing issues since the Client's ack of the MClientSession
is somehow not getting back to the MDS. We should make the
Client/SimpleMessenger be better-behaved about shutdown, but for
now this solution matches how we're working elsewhere in our codebase.
Signed-off-by: Greg Farnum <gregory.farnum@xxxxxxxxxxxxx>
---
src/mds/Server.cc | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/src/mds/Server.cc b/src/mds/Server.cc
index a9ef64f..d44bd7e 100644
--- a/src/mds/Server.cc
+++ b/src/mds/Server.cc
@@ -289,6 +289,7 @@ void Server::_session_logged(Session *session,
uint64_t state_seq, bool open, ve
// reset session
mds->send_message_client(new
MClientSession(CEPH_SESSION_CLOSE), session);
mds->sessionmap.set_state(session, Session::STATE_CLOSED);
+ mds->messenger->mark_disposable(session->connection);
session->clear();
} else if (session->is_killing()) {
// destroy session, close connection
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
