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 6db721113fff1e7994ff796337de838a7625c0d7..6ae3af7c9ee843f69ada506c10591c777f4dafa8 100644 (file)
--- 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;