timer: kill unused g_timer

diff --git a/src/common/Timer.cc b/src/common/Timer.cc
index 4d84b2a3606d135ae04e8579dfc4e92e6cf53fa1..fe8223a9347e4a4d02a41dfca9402796e3f11752 100644 (file)
--- a/src/common/Timer.cc
+++ b/src/common/Timer.cc
@@ -31,10 +31,6 @@
 #include <sys/time.h>
 #include <math.h>
 
-// single global instance
-Timer      g_timer;
-
-
 
 /**** thread solution *****/
 

diff --git a/src/common/Timer.h b/src/common/Timer.h
index bd1543de295866b8dad4826ca3ac657d3c3ce60e..e3611321768d138f248880ad4258fab60a180cf9 100644 (file)
--- a/src/common/Timer.h
+++ b/src/common/Timer.h
@@ -133,11 +133,11 @@ class Timer {
 
 
 /*
- * SafeTimer is a wrapper around the raw Timer (or rather, g_timer, it's global
- * instantiation) that protects event execution with an existing mutex.  It 
- * provides for, among other things, reliable event cancellation on class
- * destruction.  The caller just needs to cancel each event (or cancel_all()),
- * and then call join() to ensure any concurrently exectuting events (in other
+ * SafeTimer is a wrapper around the a Timer that protects event
+ * execution with an existing mutex.  It provides for, among other
+ * things, reliable event cancellation on class destruction.  The
+ * caller just needs to cancel each event (or cancel_all()), and then
+ * call join() to ensure any concurrently exectuting events (in other
  * threads) get flushed.
  */
 class SafeTimer : public Timer {
@@ -170,9 +170,6 @@ public:
 };
 
 
-// single global instance
-extern Timer g_timer;
-
 
 
 #endif

diff --git a/src/cosd.cc b/src/cosd.cc
index 9c01046fac3d15d127949495e11f630fb1553da4..9f9d995da2c77b433579e7f7de36fe13d007b2ee 100644 (file)
--- a/src/cosd.cc
+++ b/src/cosd.cc
@@ -143,7 +143,6 @@ int main(int argc, const char **argv)
        << " " << ((g_conf.osd_journal && g_conf.osd_journal[0]) ? g_conf.osd_journal:"(no journal)")
        << std::endl;
 
-  g_timer.shutdown();
   messenger->register_entity(entity_name_t::OSD(whoami));
   if (!messenger)
     return 1;

diff --git a/src/fakefuse.cc b/src/fakefuse.cc
index 11272d6967c1cf5eb21249fc47ed2e4aa0062833..b715af1aa9a8f1bcd35643158503795b1740b303 100644 (file)
--- a/src/fakefuse.cc
+++ b/src/fakefuse.cc
@@ -53,7 +53,7 @@ class C_Test2 : public Context {
 public:
   void finish(int r) {
     cout << "C_Test2->finish(" << r << ")" << std::endl;
-    g_timer.add_event_after(2, new C_Test);
+    //g_timer.add_event_after(2, new C_Test);
   }
 };
 

diff --git a/src/fakesyn.cc b/src/fakesyn.cc
index 8b6c0c85357fa9944e8738d7aff34661a4278dbe..e575d86fc815c2e28810bfcfccce3ea1bca8c73b 100644 (file)
--- a/src/fakesyn.cc
+++ b/src/fakesyn.cc
@@ -82,8 +82,8 @@ int main(int argc, const char **argv)
   assert(nargs.empty());
 
 
-  if (g_conf.kill_after) 
-    g_timer.add_event_after(g_conf.kill_after, new C_Die);
+  //if (g_conf.kill_after) 
+  //g_timer.add_event_after(g_conf.kill_after, new C_Die);
 
   if (g_conf.clock_tare) g_clock.tare();
 

diff --git a/src/mds/mdstypes.h b/src/mds/mdstypes.h
index aff034095d5339229ace829bc286466ec7e31bf7..1152bd7369ba4b72ebb0c27f6321b20541396907 100644 (file)
--- a/src/mds/mdstypes.h
+++ b/src/mds/mdstypes.h
@@ -23,7 +23,7 @@ using namespace std;
 #define CEPH_FS_ONDISK_MAGIC "ceph fs volume v011"
 
 
-//#define MDS_REF_SET      // define me for improved debug output, sanity checking
+#define MDS_REF_SET      // define me for improved debug output, sanity checking
 //#define MDS_AUTHPIN_SET  // define me for debugging auth pin leaks
 //#define MDS_VERIFY_FRAGSTAT    // do do (slow) sanity checking on frags
 

diff --git a/src/msg/FakeMessenger.cc b/src/msg/FakeMessenger.cc
index df9248473ac1920279963e1960851a83b66532d0..2ee44e74c147d46e7a8d91d20de35782ee398dcd 100644 (file)
--- a/src/msg/FakeMessenger.cc
+++ b/src/msg/FakeMessenger.cc
@@ -126,7 +126,6 @@ int fakemessenger_do_loop()
   fakemessenger_do_loop_2();
   lock.Unlock();
 
-  g_timer.shutdown();
   return 0;
 }
 

diff --git a/src/msg/SimpleMessenger.cc b/src/msg/SimpleMessenger.cc
index 4613ecb8998104f42abef93796aeef0d0dd15556..60c972847ecc59028eb4b15ea1cadf76883c4759 100644 (file)
--- a/src/msg/SimpleMessenger.cc
+++ b/src/msg/SimpleMessenger.cc
@@ -2181,8 +2181,8 @@ int SimpleMessenger::start(bool nodaemon)
   }
 
   // some debug hackery?
-  if (g_conf.kill_after) 
-    g_timer.add_event_after(g_conf.kill_after, new C_Die);
+  //if (g_conf.kill_after) 
+  //qg_timer.add_event_after(g_conf.kill_after, new C_Die);
 
   // go!
   if (did_bind)