crimson/osd: improve debugs along stopping OperationRegistry

Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

diff --git a/src/crimson/common/operation.h b/src/crimson/common/operation.h
index 60b5d59e28dd8e09cbef3735784bb5783d27331d..220cc64d1341d50641e12a54dcaaea7f9fc64522 100644 (file)
--- a/src/crimson/common/operation.h
+++ b/src/crimson/common/operation.h
@@ -20,6 +20,7 @@
 #include "include/utime.h"
 #include "common/Clock.h"
 #include "crimson/common/interruptible_future.h"
+#include "crimson/common/log.h"
 
 namespace ceph {
   class Formatter;
@@ -372,6 +373,7 @@ public:
   }
 
   seastar::future<> stop() {
+    crimson::get_logger(ceph_subsys_osd).info("OperationRegistryI::{}", __func__);
     do_stop();
     shutdown_timer.set_callback([this] {
       if (registries_empty()) {

diff --git a/src/crimson/osd/osd_operation.cc b/src/crimson/osd/osd_operation.cc
index c2f6908c82511217f9497ec0c8cd19b8886781b8..dec37d95433d4e4cad5ef11d220bd8af54907e6d 100644 (file)
--- a/src/crimson/osd/osd_operation.cc
+++ b/src/crimson/osd/osd_operation.cc
@@ -16,6 +16,7 @@ namespace crimson::osd {
 
 void OSDOperationRegistry::do_stop()
 {
+  logger().info("OSDOperationRegistry::{}", __func__);
   // we need to decouple visiting the registry from destructing
   // ops because of the auto-unlink feature of boost::intrusive.
   // the list shouldn't change while iterating due to constrains

diff --git a/src/crimson/osd/shard_services.h b/src/crimson/osd/shard_services.h
index 39a635a070b52ed2cf1d5b14808d9a3471f30f3d..917c06303865ccb069ff0e564684e74caed8b839 100644 (file)
--- a/src/crimson/osd/shard_services.h
+++ b/src/crimson/osd/shard_services.h
@@ -108,6 +108,7 @@ public:
   }
 
   seastar::future<> stop() {
+    crimson::get_logger(ceph_subsys_osd).info("ShardServices::{}", __func__);
     stopping = true;
     return registry.stop();
   }