From: Sage Weil <sage@inktank.com>
Date: Tue, 30 Oct 2012 20:19:30 +0000 (-0700)
Subject: msg/SimpleMessenger: start accepter in ready()
X-Git-Tag: v0.55~130^2~42
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=184a676e64cdcd125f06904aa0fb93727381f1a5;p=ceph.git

msg/SimpleMessenger: start accepter in ready()

Start the accepter thread when the first dispatcher is ready.  This ensures
that there will be someone around to verify authorizers for incoming
connections, and means we have a bit less failure noise on the monitors
as a result.

Signed-off-by: Sage Weil <sage@inktank.com>
---

diff --git a/src/msg/SimpleMessenger.cc b/src/msg/SimpleMessenger.cc
index 6db721113fff1..6ae3af7c9ee84 100644
--- a/src/msg/SimpleMessenger.cc
+++ b/src/msg/SimpleMessenger.cc
@@ -73,6 +73,11 @@ void SimpleMessenger::ready()
 {
   ldout(cct,10) << "ready " << get_myaddr() << dendl;
   dispatch_queue.start();
+
+  lock.Lock();
+  if (did_bind)
+    accepter.start();
+  lock.Unlock();
 }
 
 
@@ -287,9 +292,6 @@ int SimpleMessenger::start()
 
   lock.Unlock();
 
-  if (did_bind)
-    accepter.start();
-
   reaper_started = true;
   reaper_thread.create();
   return 0;