simplemessenger: Fix num_threads bug printout.

Also add documentation to get_num_threads since its contract changed significantly. Signed-off-by: Greg Farnum <[email protected]>
sahid · Feb 16, 2011 · 29b5ae0 · 29b5ae0
1 parent 03e56b7
commit 29b5ae0
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/src/common/Thread.cc b/src/common/Thread.cc
@@ -22,6 +22,13 @@
 #include "common/debug.h"
 #include "common/Thread.h"
 
+/**
+ * Return the number of threads in this process. The data is
+ * retrieved from /proc and includes all threads, not just
+ * "child" threads.
+ * Behavior changed in 6fb416b083d518e5f524359cc3cacb66ccc63dca
+ * to support eventual elimination of global variables.
+ */
 int Thread::get_num_threads(void)
 {
   std::ostringstream oss;

diff --git a/src/msg/SimpleMessenger.cc b/src/msg/SimpleMessenger.cc
@@ -2429,8 +2429,8 @@ int SimpleMessenger::start(bool nodaemon)
   if (g_conf.daemonize && !nodaemon) {
     int num_threads = Thread::get_num_threads();
     if (num_threads > 1) {
-      derr << "messenger.start BUG: there are " << num_threads << " threads"
-	   << " already started that will now die!  call messenger.start() sooner."
+      derr << "messenger.start BUG: there are " << num_threads - 1
+           << " child threads already started that will now die!  call messenger.start() sooner."
 	   << dendl;
     }