]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr: Mutex::Locker -> std::lock_guard 24623/head
authorSage Weil <sage@redhat.com>
Tue, 16 Oct 2018 19:11:33 +0000 (14:11 -0500)
committerSage Weil <sage@redhat.com>
Tue, 16 Oct 2018 19:11:33 +0000 (14:11 -0500)
Signed-off-by: Sage Weil <sage@redhat.com>
16 files changed:
src/mgr/ActivePyModules.cc
src/mgr/ClusterState.cc
src/mgr/ClusterState.h
src/mgr/DaemonServer.cc
src/mgr/DaemonState.h
src/mgr/Mgr.cc
src/mgr/MgrClient.cc
src/mgr/MgrClient.h
src/mgr/MgrStandby.cc
src/mgr/OSDPerfMetricCollector.cc
src/mgr/PyModule.cc
src/mgr/PyModule.h
src/mgr/PyModuleRegistry.cc
src/mgr/PyModuleRegistry.h
src/mgr/StandbyPyModules.cc
src/mgr/StandbyPyModules.h

index cb635ab12217cee0203c1fef3d8e20541007d66e..81dbe30705ce5ef3d16aa58b360a8260de2213e3 100644 (file)
@@ -60,7 +60,7 @@ void ActivePyModules::dump_server(const std::string &hostname,
   std::string ceph_version;
 
   for (const auto &i : dmc) {
-    Mutex::Locker l(i.second->lock);
+    std::lock_guard l(i.second->lock);
     const auto &key = i.first;
     const std::string &str_type = key.first;
     const std::string &svc_name = key.second;
@@ -88,7 +88,7 @@ void ActivePyModules::dump_server(const std::string &hostname,
 PyObject *ActivePyModules::get_server_python(const std::string &hostname)
 {
   PyThreadState *tstate = PyEval_SaveThread();
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   PyEval_RestoreThread(tstate);
   dout(10) << " (" << hostname << ")" << dendl;
 
@@ -103,7 +103,7 @@ PyObject *ActivePyModules::get_server_python(const std::string &hostname)
 PyObject *ActivePyModules::list_servers_python()
 {
   PyThreadState *tstate = PyEval_SaveThread();
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   PyEval_RestoreThread(tstate);
   dout(10) << " >" << dendl;
 
@@ -132,7 +132,7 @@ PyObject *ActivePyModules::get_metadata_python(
     Py_RETURN_NONE;
   }
 
-  Mutex::Locker l(metadata->lock);
+  std::lock_guard l(metadata->lock);
   PyFormatter f;
   f.dump_string("hostname", metadata->hostname);
   for (const auto &i : metadata->metadata) {
@@ -152,7 +152,7 @@ PyObject *ActivePyModules::get_daemon_status_python(
     Py_RETURN_NONE;
   }
 
-  Mutex::Locker l(metadata->lock);
+  std::lock_guard l(metadata->lock);
   PyFormatter f;
   for (const auto &i : metadata->service_status) {
     f.dump_string(i.first.c_str(), i.second);
@@ -163,7 +163,7 @@ PyObject *ActivePyModules::get_daemon_status_python(
 PyObject *ActivePyModules::get_python(const std::string &what)
 {
   PyThreadState *tstate = PyEval_SaveThread();
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   PyEval_RestoreThread(tstate);
 
   if (what == "fs_map") {
@@ -219,7 +219,7 @@ PyObject *ActivePyModules::get_python(const std::string &what)
     PyFormatter f;
     auto dmc = daemon_state.get_by_service("osd");
     for (const auto &i : dmc) {
-      Mutex::Locker l(i.second->lock);
+      std::lock_guard l(i.second->lock);
       f.open_object_section(i.first.second.c_str());
       f.dump_string("hostname", i.second->hostname);
       for (const auto &j : i.second->metadata) {
@@ -373,7 +373,7 @@ PyObject *ActivePyModules::get_python(const std::string &what)
 
 int ActivePyModules::start_one(PyModuleRef py_module)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   ceph_assert(modules.count(py_module->get_name()) == 0);
 
@@ -396,7 +396,7 @@ int ActivePyModules::start_one(PyModuleRef py_module)
 
 void ActivePyModules::shutdown()
 {
-  Mutex::Locker locker(lock);
+  std::lock_guard locker(lock);
 
   // Signal modules to drop out of serve() and/or tear down resources
   for (auto &i : modules) {
@@ -426,7 +426,7 @@ void ActivePyModules::shutdown()
 void ActivePyModules::notify_all(const std::string &notify_type,
                      const std::string &notify_id)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   dout(10) << __func__ << ": notify_all " << notify_type << dendl;
   for (auto& i : modules) {
@@ -441,7 +441,7 @@ void ActivePyModules::notify_all(const std::string &notify_type,
 
 void ActivePyModules::notify_all(const LogEntry &log_entry)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   dout(10) << __func__ << ": notify_all (clog)" << dendl;
   for (auto& i : modules) {
@@ -462,7 +462,7 @@ bool ActivePyModules::get_store(const std::string &module_name,
     const std::string &key, std::string *val) const
 {
   PyThreadState *tstate = PyEval_SaveThread();
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   PyEval_RestoreThread(tstate);
 
   const std::string global_key = PyModule::config_prefix
@@ -496,7 +496,7 @@ bool ActivePyModules::get_config(const std::string &module_name,
     const std::string &key, std::string *val) const
 {
   PyThreadState *tstate = PyEval_SaveThread();
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   PyEval_RestoreThread(tstate);
 
   const std::string global_key = PyModule::config_prefix
@@ -504,7 +504,7 @@ bool ActivePyModules::get_config(const std::string &module_name,
 
   dout(4) << __func__ << " key: " << global_key << dendl;
 
-  Mutex::Locker lock(module_config.lock);
+  std::lock_guard lock(module_config.lock);
   
   auto i = module_config.config.find(global_key);
   if (i != module_config.config.end()) {
@@ -519,7 +519,7 @@ PyObject *ActivePyModules::get_store_prefix(const std::string &module_name,
     const std::string &prefix) const
 {
   PyThreadState *tstate = PyEval_SaveThread();
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   PyEval_RestoreThread(tstate);
 
   const std::string base_prefix = PyModule::config_prefix
@@ -529,7 +529,7 @@ PyObject *ActivePyModules::get_store_prefix(const std::string &module_name,
 
   PyFormatter f;
   
-  Mutex::Locker lock(module_config.lock);
+  std::lock_guard lock(module_config.lock);
   
   for (auto p = store_cache.lower_bound(global_prefix);
        p != store_cache.end() && p->first.find(global_prefix) == 0;
@@ -548,7 +548,7 @@ void ActivePyModules::set_store(const std::string &module_name,
   Command set_cmd;
   {
     PyThreadState *tstate = PyEval_SaveThread();
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
     PyEval_RestoreThread(tstate);
 
     if (val) {
@@ -593,7 +593,7 @@ void ActivePyModules::set_config(const std::string &module_name,
 std::map<std::string, std::string> ActivePyModules::get_services() const
 {
   std::map<std::string, std::string> result;
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   for (const auto& i : modules) {
     const auto &module = i.second.get();
     std::string svc_str = module->get_uri();
@@ -612,7 +612,7 @@ PyObject* ActivePyModules::with_perf_counters(
     const std::string &path) const
 {
   PyThreadState *tstate = PyEval_SaveThread();
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   PyEval_RestoreThread(tstate);
 
   PyFormatter f;
@@ -620,7 +620,7 @@ PyObject* ActivePyModules::with_perf_counters(
 
   auto metadata = daemon_state.get(DaemonKey(svc_name, svc_id));
   if (metadata) {
-    Mutex::Locker l2(metadata->lock);
+    std::lock_guard l2(metadata->lock);
     if (metadata->perf_counters.instances.count(path)) {
       auto counter_instance = metadata->perf_counters.instances.at(path);
       auto counter_type = metadata->perf_counters.types.at(path);
@@ -702,7 +702,7 @@ PyObject* ActivePyModules::get_perf_schema_python(
     const std::string &svc_id)
 {
   PyThreadState *tstate = PyEval_SaveThread();
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   PyEval_RestoreThread(tstate);
 
   DaemonStateCollection daemons;
@@ -730,7 +730,7 @@ PyObject* ActivePyModules::get_perf_schema_python(
       daemon_name << key.first << "." << key.second;
       f.open_object_section(daemon_name.str().c_str());
 
-      Mutex::Locker l(state->lock);
+      std::lock_guard l(state->lock);
       for (auto ctr_inst_iter : state->perf_counters.instances) {
         const auto &counter_name = ctr_inst_iter.first;
        f.open_object_section(counter_name.c_str());
@@ -756,7 +756,7 @@ PyObject* ActivePyModules::get_perf_schema_python(
 PyObject *ActivePyModules::get_context()
 {
   PyThreadState *tstate = PyEval_SaveThread();
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   PyEval_RestoreThread(tstate);
 
   // Construct a capsule containing ceph context.
@@ -814,7 +814,7 @@ PyObject *construct_with_capsule(
 PyObject *ActivePyModules::get_osdmap()
 {
   PyThreadState *tstate = PyEval_SaveThread();
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   PyEval_RestoreThread(tstate);
 
   OSDMap *newmap = new OSDMap;
@@ -874,7 +874,7 @@ int ActivePyModules::handle_command(
 
 void ActivePyModules::get_health_checks(health_check_map_t *checks)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   for (auto& p : modules) {
     p.second->get_health_checks(checks);
   }
@@ -883,7 +883,7 @@ void ActivePyModules::get_health_checks(health_check_map_t *checks)
 void ActivePyModules::set_uri(const std::string& module_name,
                         const std::string &uri)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   dout(4) << " module " << module_name << " set URI '" << uri << "'" << dendl;
 
@@ -908,7 +908,7 @@ void ActivePyModules::remove_osd_perf_query(OSDPerfMetricQueryID query_id)
 void ActivePyModules::cluster_log(const std::string &channel, clog_type prio,
   const std::string &message)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   if (channel == "audit") {
     audit_clog->do_log(prio, message);
index 436b53f8f9d4b120dd6c76f60acfb6fe34f33058..ed741eeecf58d6ee1641dbd804fc9167a8c19a45 100644 (file)
@@ -34,27 +34,27 @@ ClusterState::ClusterState(
 
 void ClusterState::set_objecter(Objecter *objecter_)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   objecter = objecter_;
 }
 
 void ClusterState::set_fsmap(FSMap const &new_fsmap)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   fsmap = new_fsmap;
 }
 
 void ClusterState::set_mgr_map(MgrMap const &new_mgrmap)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   mgr_map = new_mgrmap;
 }
 
 void ClusterState::set_service_map(ServiceMap const &new_service_map)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   servicemap = new_service_map;
 }
 
@@ -66,7 +66,7 @@ void ClusterState::load_digest(MMgrDigest *m)
 
 void ClusterState::ingest_pgstats(MPGStats *stats)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   const int from = stats->get_orig_source().num();
 
@@ -135,7 +135,7 @@ void ClusterState::update_delta_stats()
 
 void ClusterState::notify_osdmap(const OSDMap &osd_map)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   pending_inc.stamp = ceph_clock_now();
   pending_inc.version = pg_map.version + 1; // to make apply_incremental happy
index a22c058dec74066c2fbf943e1ebd2836549b698c..69fc92bfd026a19ff52a704ef4f3b77249f02ca5 100644 (file)
@@ -70,28 +70,28 @@ public:
   void notify_osdmap(const OSDMap &osd_map);
 
   bool have_fsmap() const {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
     return fsmap.get_epoch() > 0;
   }
 
   template<typename Callback, typename...Args>
   void with_servicemap(Callback&& cb, Args&&...args) const
   {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
     std::forward<Callback>(cb)(servicemap, std::forward<Args>(args)...);
   }
 
   template<typename Callback, typename...Args>
   void with_fsmap(Callback&& cb, Args&&...args) const
   {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
     std::forward<Callback>(cb)(fsmap, std::forward<Args>(args)...);
   }
 
   template<typename Callback, typename...Args>
   void with_mgrmap(Callback&& cb, Args&&...args) const
   {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
     std::forward<Callback>(cb)(mgr_map, std::forward<Args>(args)...);
   }
 
@@ -99,7 +99,7 @@ public:
   auto with_pgmap(Callback&& cb, Args&&...args) const ->
     decltype(cb(pg_map, std::forward<Args>(args)...))
   {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
     return std::forward<Callback>(cb)(pg_map, std::forward<Args>(args)...);
   }
 
@@ -107,14 +107,14 @@ public:
   auto with_mutable_pgmap(Callback&& cb, Args&&...args) ->
     decltype(cb(pg_map, std::forward<Args>(args)...))
   {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
     return std::forward<Callback>(cb)(pg_map, std::forward<Args>(args)...);
   }
 
   template<typename... Args>
   void with_monmap(Args &&... args) const
   {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
     ceph_assert(monc != nullptr);
     monc->with_monmap(std::forward<Args>(args)...);
   }
index 3fbf791b69ee897627cda89cacb3341eac3006de..e9373bbea72b08ab8e322a35da8e1387be495a82 100644 (file)
@@ -147,7 +147,7 @@ int DaemonServer::init(uint64_t gid, entity_addrvec_t client_addrs)
 
   started_at = ceph_clock_now();
 
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   timer.init();
 
   schedule_tick_locked(
@@ -207,7 +207,7 @@ int DaemonServer::ms_handle_authentication(Connection *con)
   }
 
   if (con->get_peer_type() == CEPH_ENTITY_TYPE_OSD) {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
     s->osd_id = atoi(s->entity_name.get_id().c_str());
     dout(10) << "registering osd." << s->osd_id << " session "
             << s << " con " << con << dendl;
@@ -244,7 +244,7 @@ bool DaemonServer::ms_handle_reset(Connection *con)
     if (!session) {
       return false;
     }
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
     dout(10) << "unregistering osd." << session->osd_id
             << "  session " << session << " con " << con << dendl;
     osd_cons[session->osd_id].erase(con);
@@ -294,7 +294,7 @@ void DaemonServer::maybe_ready(int32_t osd_id)
     // Fast path: we don't need to take lock because pgmap_ready
     // is already set
   } else {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
 
     if (reported_osds.find(osd_id) == reported_osds.end()) {
       dout(4) << "initial report from osd " << osd_id << dendl;
@@ -360,7 +360,7 @@ void DaemonServer::schedule_tick_locked(double delay_sec)
 
 void DaemonServer::schedule_tick(double delay_sec)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   schedule_tick_locked(delay_sec);
 }
 
@@ -371,7 +371,7 @@ void DaemonServer::handle_osd_perf_metric_query_updated()
   // Send a fresh MMgrConfigure to all clients, so that they can follow
   // the new policy for transmitting stats
   finisher.queue(new FunctionContext([this](int r) {
-        Mutex::Locker l(lock);
+        std::lock_guard l(lock);
         for (auto &c : daemon_connections) {
           if (c->peer_is_osd()) {
             _send_configure(c);
@@ -387,7 +387,7 @@ void DaemonServer::shutdown()
   msgr->wait();
   dout(10) << "done" << dendl;
 
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   shutting_down = true;
   timer.shutdown();
 }
@@ -419,7 +419,7 @@ static bool key_from_string(
 
 bool DaemonServer::handle_open(MMgrOpen *m)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   DaemonKey key = key_from_service(m->service_name,
                                   m->get_connection()->get_peer_type(),
@@ -445,7 +445,7 @@ bool DaemonServer::handle_open(MMgrOpen *m)
   }
   if (daemon) {
     dout(20) << "updating existing DaemonState for " << m->daemon_name << dendl;
-    Mutex::Locker l(daemon->lock);
+    std::lock_guard l(daemon->lock);
     daemon->perf_counters.clear();
 
     if (m->service_daemon) {
@@ -494,7 +494,7 @@ bool DaemonServer::handle_open(MMgrOpen *m)
 
 bool DaemonServer::handle_close(MMgrClose *m)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   DaemonKey key = key_from_service(m->service_name,
                                   m->get_connection()->get_peer_type(),
@@ -505,7 +505,7 @@ bool DaemonServer::handle_close(MMgrClose *m)
     DaemonStatePtr daemon = daemon_state.get(key);
     daemon_state.rm(key);
     {
-      Mutex::Locker l(daemon->lock);
+      std::lock_guard l(daemon->lock);
       if (daemon->service_daemon) {
        pending_service_map.rm_daemon(m->service_name, m->daemon_name);
        pending_service_map_dirty = pending_service_map.epoch;
@@ -574,7 +574,7 @@ bool DaemonServer::handle_report(MMgrReport *m)
     }
     
     {
-      Mutex::Locker l(lock);
+      std::lock_guard l(lock);
       // kill session
       auto priv = m->get_connection()->get_priv();
       auto session = static_cast<MgrSession*>(priv.get());
@@ -602,7 +602,7 @@ bool DaemonServer::handle_report(MMgrReport *m)
   // Update the DaemonState
   ceph_assert(daemon != nullptr);
   {
-    Mutex::Locker l(daemon->lock);
+    std::lock_guard l(daemon->lock);
     auto &daemon_counters = daemon->perf_counters;
     daemon_counters.update(m);
 
@@ -778,7 +778,7 @@ public:
 
 bool DaemonServer::handle_command(MCommand *m)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   std::shared_ptr<CommandContext> cmdctx = std::make_shared<CommandContext>(m);
   try {
     return _handle_command(m, cmdctx);
@@ -911,7 +911,7 @@ bool DaemonServer::_handle_command(
        DaemonKey key(p.first, q.first);
        ceph_assert(daemon_state.exists(key));
        auto daemon = daemon_state.get(key);
-       Mutex::Locker l(daemon->lock);
+       std::lock_guard l(daemon->lock);
        f->dump_stream("status_stamp") << daemon->service_status_stamp;
        f->dump_stream("last_beacon") << daemon->last_service_beacon;
        f->open_object_section("status");
@@ -1608,7 +1608,7 @@ bool DaemonServer::_handle_command(
        }
       }
     } else if (daemon->config_defaults_bl.length() > 0) {
-      Mutex::Locker l(daemon->lock);
+      std::lock_guard l(daemon->lock);
       TextTable tbl;
       if (f) {
        f->open_array_section("config");
@@ -2110,7 +2110,7 @@ void DaemonServer::_prune_pending_service_map()
        continue;
       }
       auto daemon = daemon_state.get(key);
-      Mutex::Locker l(daemon->lock);
+      std::lock_guard l(daemon->lock);
       if (daemon->last_service_beacon == utime_t()) {
        // we must have just restarted; assume they are alive now.
        daemon->last_service_beacon = ceph_clock_now();
@@ -2190,7 +2190,7 @@ void DaemonServer::send_report()
   for (auto service : {"osd", "mon"} ) {
     auto daemons = daemon_state.get_by_service(service);
     for (const auto& [key,state] : daemons) {
-      Mutex::Locker l{state->lock};
+      std::lock_guard l{state->lock};
       for (const auto& metric : state->daemon_health_metrics) {
         auto acc = accumulated.find(metric.get_type());
         if (acc == accumulated.end()) {
@@ -2408,7 +2408,7 @@ void DaemonServer::adjust_pgs()
 
 void DaemonServer::got_service_map()
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   cluster_state.with_servicemap([&](const ServiceMap& service_map) {
       if (pending_service_map.epoch == 0) {
@@ -2447,7 +2447,7 @@ void DaemonServer::got_service_map()
 
 void DaemonServer::got_mgr_map()
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   set<std::string> have;
   cluster_state.with_mgrmap([&](const MgrMap& mgrmap) {
       auto md_update = [&] (DaemonKey key) {
index 5732178390ae71f746066e6b3137bf894ffc38d8..f706aab3080d814a2cfdd1289f09bc887862c7bf 100644 (file)
@@ -333,7 +333,7 @@ public:
                             std::set<std::string> *ls) {
     auto m = get_by_server(server);
     for (auto& i : m) {
-      Mutex::Locker l(i.second->lock);
+      std::lock_guard l(i.second->lock);
       for (auto& j : i.second->devices) {
        ls->insert(j.first);
       }
index 329a1ade2d1a7eaa156502053d9075cab069809d..d6b57b9d283212523bbf3f60be132384556a3289 100644 (file)
@@ -94,7 +94,7 @@ void MetadataUpdate::finish(int r)
       DaemonStatePtr state;
       if (daemon_state.exists(key)) {
         state = daemon_state.get(key);
-       Mutex::Locker l(state->lock);
+       std::lock_guard l(state->lock);
         if (key.first == "mds" || key.first == "mgr") {
           daemon_meta.erase("name");
         } else if (key.first == "osd") {
@@ -139,7 +139,7 @@ void MetadataUpdate::finish(int r)
 
 void Mgr::background_init(Context *completion)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   ceph_assert(!initializing);
   ceph_assert(!initialized);
   initializing = true;
@@ -214,7 +214,7 @@ std::map<std::string, std::string> Mgr::load_store()
 
 void Mgr::init()
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   ceph_assert(initializing);
   ceph_assert(!initialized);
 
@@ -389,7 +389,7 @@ void Mgr::shutdown()
 {
   finisher.queue(new FunctionContext([&](int) {
     {
-      Mutex::Locker l(lock);
+      std::lock_guard l(lock);
       monc->sub_unwant("log-info");
       monc->sub_unwant("mgrdigest");
       monc->sub_unwant("fsmap");
@@ -436,7 +436,7 @@ void Mgr::handle_osd_map()
 
       if (daemon_state.exists(k)) {
         auto metadata = daemon_state.get(k);
-       Mutex::Locker l(metadata->lock);
+       std::lock_guard l(metadata->lock);
         auto addr_iter = metadata->metadata.find("front_addr");
         if (addr_iter != metadata->metadata.end()) {
           const std::string &metadata_addr = addr_iter->second;
@@ -496,7 +496,7 @@ void Mgr::handle_service_map(MServiceMap *m)
 bool Mgr::ms_dispatch(Message *m)
 {
   dout(4) << *m << dendl;
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   switch (m->get_type()) {
     case MSG_MGR_DIGEST:
@@ -573,7 +573,7 @@ void Mgr::handle_fs_map(MFSMap* m)
     bool update = false;
     if (daemon_state.exists(k)) {
       auto metadata = daemon_state.get(k);
-      Mutex::Locker l(metadata->lock);
+      std::lock_guard l(metadata->lock);
       if (metadata->metadata.empty() ||
          metadata->metadata.count("addr") == 0) {
         update = true;
@@ -610,7 +610,7 @@ void Mgr::handle_fs_map(MFSMap* m)
 
 bool Mgr::got_mgr_map(const MgrMap& m)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   dout(10) << m << dendl;
 
   set<string> old_modules;
@@ -652,7 +652,7 @@ void Mgr::handle_mgr_digest(MMgrDigest* m)
 
 std::map<std::string, std::string> Mgr::get_services() const
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   return py_module_registry->get_services();
 }
index c436bb9fee1adf116af8bdcf67c0aa143882c2f8..4bddbe2a19ce65ba2f4cce9a2a86a6d1737aec1d 100644 (file)
@@ -39,7 +39,7 @@ MgrClient::MgrClient(CephContext *cct_, Messenger *msgr_)
 
 void MgrClient::init()
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   ceph_assert(msgr != nullptr);
 
@@ -48,7 +48,7 @@ void MgrClient::init()
 
 void MgrClient::shutdown()
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   ldout(cct, 10) << dendl;
 
   if (connect_retry_callback) {
@@ -83,7 +83,7 @@ void MgrClient::shutdown()
 
 bool MgrClient::ms_dispatch(Message *m)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   switch(m->get_type()) {
   case MSG_MGR_MAP:
@@ -219,7 +219,7 @@ bool MgrClient::handle_mgr_map(MMgrMap *m)
 
 bool MgrClient::ms_handle_reset(Connection *con)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   if (session && con == session->con) {
     ldout(cct, 4) << __func__ << " con " << con << dendl;
     reconnect();
@@ -359,7 +359,7 @@ void MgrClient::_send_report()
 
 void MgrClient::send_pgstats()
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   _send_pgstats();
 }
 
@@ -415,7 +415,7 @@ int MgrClient::start_command(const vector<string>& cmd, const bufferlist& inbl,
                   bufferlist *outbl, string *outs,
                   Context *onfinish)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   ldout(cct, 20) << "cmd: " << cmd << dendl;
 
@@ -479,7 +479,7 @@ int MgrClient::service_daemon_register(
   const std::string& name,
   const std::map<std::string,std::string>& metadata)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   if (service == "osd" ||
       service == "mds" ||
       service == "client" ||
@@ -509,7 +509,7 @@ int MgrClient::service_daemon_register(
 int MgrClient::service_daemon_update_status(
   std::map<std::string,std::string>&& status)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   ldout(cct,10) << status << dendl;
   daemon_status = std::move(status);
   daemon_dirty_status = true;
@@ -518,7 +518,7 @@ int MgrClient::service_daemon_update_status(
 
 void MgrClient::update_daemon_health(std::vector<DaemonHealthMetric>&& metrics)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   daemon_health_metrics = std::move(metrics);
 }
 
index ffeb586450a84a3f060fa9a09bcb4a21eb249531..5b561d26170d7cec1053d60d7a9a6f0450832d8f 100644 (file)
@@ -124,7 +124,7 @@ public:
           std::function<void(OSDPerfMetricReport *)> cb_get)
 
   {
-      Mutex::Locker l(lock);
+      std::lock_guard l(lock);
       set_perf_queries_cb = cb_set;
       get_perf_report_cb = cb_get;
   }
@@ -133,7 +133,7 @@ public:
   void send_pgstats();
   void set_pgstats_cb(std::function<MPGStats*()>&& cb_)
   {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
     pgstats_cb = std::move(cb_);
   }
 
index da358d868fb3cccb42f17bd920aaaa72be865923..9265c4ff1d5ef4f654075d592cebd59ba89ec731 100644 (file)
@@ -105,7 +105,7 @@ int MgrStandby::init()
   init_async_signal_handler();
   register_async_signal_handler(SIGHUP, sighup_handler);
 
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   // Initialize Messenger
   client_messenger->add_dispatcher_tail(this);
@@ -248,7 +248,7 @@ void MgrStandby::tick()
 
 void MgrStandby::handle_signal(int signum)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   ceph_assert(signum == SIGINT || signum == SIGTERM);
   derr << "*** Got signal " << sig_str(signum) << " ***" << dendl;
   shutdown();
@@ -372,7 +372,7 @@ void MgrStandby::handle_mgr_map(MMgrMap* mmap)
             [this](int r){
               // Advertise our active-ness ASAP instead of waiting for
               // next tick.
-              Mutex::Locker l(lock);
+              std::lock_guard l(lock);
               send_beacon();
             }));
       dout(1) << "I am now activating" << dendl;
@@ -404,7 +404,7 @@ void MgrStandby::handle_mgr_map(MMgrMap* mmap)
 
 bool MgrStandby::ms_dispatch(Message *m)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   dout(4) << state_str() << " " << *m << dendl;
 
   if (m->get_type() == MSG_MGR_MAP) {
index ca28bcca8454474f9eabc977183aa6505e53bcb7..6c4354d7abed4036e5eef01c5d9f82408bba16b9 100644 (file)
@@ -16,7 +16,7 @@ OSDPerfMetricCollector::OSDPerfMetricCollector(Listener &listener)
 }
 
 std::list<OSDPerfMetricQuery> OSDPerfMetricCollector::get_queries() {
-  Mutex::Locker locker(lock);
+  std::lock_guard locker(lock);
 
   std::list<OSDPerfMetricQuery> query_list;
   for (auto &it : queries) {
@@ -31,7 +31,7 @@ int OSDPerfMetricCollector::add_query(const OSDPerfMetricQuery& query) {
   bool notify = false;
 
   {
-    Mutex::Locker locker(lock);
+    std::lock_guard locker(lock);
 
     query_id = next_query_id++;
     auto it = queries.find(query);
@@ -56,7 +56,7 @@ int OSDPerfMetricCollector::remove_query(int query_id) {
   bool notify = false;
 
   {
-    Mutex::Locker locker(lock);
+    std::lock_guard locker(lock);
 
     for (auto it = queries.begin() ; it != queries.end(); it++) {
       auto &ids = it->second;
@@ -92,7 +92,7 @@ void OSDPerfMetricCollector::remove_all_queries() {
   bool notify;
 
   {
-    Mutex::Locker locker(lock);
+    std::lock_guard locker(lock);
 
     notify = !queries.empty();
     queries.clear();
index cd97f4b5286060eef1072e2fa50b68a1a35529e2..9838bbff7c219d84d08d65f5a5291196f22201c3 100644 (file)
@@ -145,7 +145,7 @@ void PyModuleConfig::set_config(
   const std::string global_key = PyModule::config_prefix
                                    + module_name + "/" + key;
   {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
 
     if (val) {
       config[global_key] = *val;
@@ -534,7 +534,7 @@ int PyModule::load_options()
 
 bool PyModule::is_option(const std::string &option_name)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   return options.count(option_name) > 0;
 }
 
index db5c501d375485b523dc342695777a370d0e9c61..84d3a6f62f0ffe5f17eff7e31a3888e70b075fed 100644 (file)
@@ -131,7 +131,7 @@ public:
    */
   void get_commands(std::vector<ModuleCommand> *out) const
   {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
     ceph_assert(out != nullptr);
     out->insert(out->end(), commands.begin(), commands.end());
   }
@@ -143,28 +143,28 @@ public:
    */
   void fail(const std::string &reason)
   {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
     failed = true;
     error_string = reason;
   }
 
   bool is_enabled() const {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
     return enabled || always_on;
   }
 
-  bool is_failed() const { Mutex::Locker l(lock) ; return failed; }
-  bool is_loaded() const { Mutex::Locker l(lock) ; return loaded; }
-  bool is_always_on() const { Mutex::Locker l(lock) ; return always_on; }
+  bool is_failed() const { std::lock_guard l(lock) ; return failed; }
+  bool is_loaded() const { std::lock_guard l(lock) ; return loaded; }
+  bool is_always_on() const { std::lock_guard l(lock) ; return always_on; }
 
   const std::string &get_name() const {
-    Mutex::Locker l(lock) ; return module_name;
+    std::lock_guard l(lock) ; return module_name;
   }
   const std::string &get_error_string() const {
-    Mutex::Locker l(lock) ; return error_string;
+    std::lock_guard l(lock) ; return error_string;
   }
   bool get_can_run() const {
-    Mutex::Locker l(lock) ; return can_run;
+    std::lock_guard l(lock) ; return can_run;
   }
 };
 
index b95b482dd24596741b13defef7ae0b9e62e2d555..7696fb02eb645b2f75e2c5cb385719ab35b7e1ad 100644 (file)
@@ -36,7 +36,7 @@
 
 void PyModuleRegistry::init()
 {
-  Mutex::Locker locker(lock);
+  std::lock_guard locker(lock);
 
   // Set up global python interpreter
 #if PY_MAJOR_VERSION >= 3
@@ -97,7 +97,7 @@ void PyModuleRegistry::init()
 
 bool PyModuleRegistry::handle_mgr_map(const MgrMap &mgr_map_)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   if (mgr_map.epoch == 0) {
     mgr_map = mgr_map_;
@@ -130,7 +130,7 @@ bool PyModuleRegistry::handle_mgr_map(const MgrMap &mgr_map_)
 
 void PyModuleRegistry::standby_start(MonClient &mc)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   ceph_assert(active_modules == nullptr);
   ceph_assert(standby_modules == nullptr);
 
@@ -182,7 +182,7 @@ void PyModuleRegistry::active_start(
             Objecter &objecter_, Client &client_, Finisher &f,
             DaemonServer &server)
 {
-  Mutex::Locker locker(lock);
+  std::lock_guard locker(lock);
 
   dout(4) << "Starting modules in active mode" << dendl;
 
@@ -219,7 +219,7 @@ void PyModuleRegistry::active_start(
 
 void PyModuleRegistry::active_shutdown()
 {
-  Mutex::Locker locker(lock);
+  std::lock_guard locker(lock);
 
   if (active_modules != nullptr) {
     active_modules->shutdown();
@@ -229,7 +229,7 @@ void PyModuleRegistry::active_shutdown()
 
 void PyModuleRegistry::shutdown()
 {
-  Mutex::Locker locker(lock);
+  std::lock_guard locker(lock);
 
   if (standby_modules != nullptr) {
     standby_modules->shutdown();
@@ -311,7 +311,7 @@ int PyModuleRegistry::handle_command(
 
 std::vector<ModuleCommand> PyModuleRegistry::get_py_commands() const
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   std::vector<ModuleCommand> result;
   for (const auto& i : modules) {
@@ -338,7 +338,7 @@ std::vector<MonCommand> PyModuleRegistry::get_commands() const
 
 void PyModuleRegistry::get_health_checks(health_check_map_t *checks)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
 
   // Only the active mgr reports module issues
   if (active_modules) {
@@ -412,7 +412,7 @@ void PyModuleRegistry::get_health_checks(health_check_map_t *checks)
 
 void PyModuleRegistry::handle_config(const std::string &k, const std::string &v)
 {
-  Mutex::Locker l(module_config.lock);
+  std::lock_guard l(module_config.lock);
 
   if (!v.empty()) {
     dout(4) << "Loaded module_config entry " << k << ":" << v << dendl;
index 7e2ae5062d5ed57a9276bb6492ab6bc87fc57e85..d0cbe1881a9ce0e61a822d0bc9ef3d01a695b1ec 100644 (file)
@@ -68,7 +68,7 @@ public:
    */
   std::list<PyModuleRef> get_modules() const
   {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
     std::list<PyModuleRef> modules_out;
     for (const auto &i : modules) {
       modules_out.push_back(i.second);
@@ -117,7 +117,7 @@ public:
    */
   PyModuleRef get_module(const std::string &module_name)
   {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
     return modules.at(module_name);
   }
 
index f2d1eeb45b8d4a01ba6cf65d0f6ba4f9d7ee864b..0682a26811999ef4c77a6502ea8c822a8f511152 100644 (file)
@@ -46,7 +46,7 @@ StandbyPyModules::StandbyPyModules(
 // FIXME: completely identical to ActivePyModules
 void StandbyPyModules::shutdown()
 {
-  Mutex::Locker locker(lock);
+  std::lock_guard locker(lock);
 
   // Signal modules to drop out of serve() and/or tear down resources
   for (auto &i : modules) {
@@ -74,7 +74,7 @@ void StandbyPyModules::shutdown()
 
 int StandbyPyModules::start_one(PyModuleRef py_module)
 {
-  Mutex::Locker l(lock);
+  std::lock_guard l(lock);
   const std::string &module_name = py_module->get_name();
 
   ceph_assert(modules.count(module_name) == 0);
index f443ba67e7947ebeda2ceeb5ae00e279398c048e..02d26800cd1a5ab7343ec80e47125c48625532af 100644 (file)
@@ -46,7 +46,7 @@ public:
 
   void set_mgr_map(const MgrMap &mgr_map_)
   {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
 
     mgr_map = mgr_map_;
   }
@@ -58,14 +58,14 @@ public:
   template<typename Callback, typename...Args>
   void with_mgr_map(Callback&& cb, Args&&...args) const
   {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
     std::forward<Callback>(cb)(mgr_map, std::forward<Args>(args)...);
   }
 
   template<typename Callback, typename...Args>
   auto with_config(Callback&& cb, Args&&... args) const ->
     decltype(cb(module_config, std::forward<Args>(args)...)) {
-    Mutex::Locker l(lock);
+    std::lock_guard l(lock);
 
     return std::forward<Callback>(cb)(module_config, std::forward<Args>(args)...);
   }