From: Radoslaw Zarzynski <rzarzyns@redhat.com>
Date: Tue, 12 Jul 2022 16:55:47 +0000 (+0000)
Subject: crimson/osd: improve debugs along stopping OperationRegistry
X-Git-Tag: v18.0.0~522^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F47064%2Fhead;p=ceph.git

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 60b5d59e28dd..220cc64d1341 100644
--- 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 c2f6908c8251..dec37d95433d 100644
--- 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 39a635a070b5..917c06303865 100644
--- 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();
   }