From 8fb37b930016ad726f649e00cbaeace2a4fd0d9d Mon Sep 17 00:00:00 2001 From: Radoslaw Zarzynski Date: Tue, 12 Jul 2022 16:55:47 +0000 Subject: [PATCH] crimson/osd: improve debugs along stopping OperationRegistry Signed-off-by: Radoslaw Zarzynski --- src/crimson/common/operation.h | 2 ++ src/crimson/osd/osd_operation.cc | 1 + src/crimson/osd/shard_services.h | 1 + 3 files changed, 4 insertions(+) diff --git a/src/crimson/common/operation.h b/src/crimson/common/operation.h index 60b5d59e28dd8..220cc64d1341d 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 c2f6908c82511..dec37d95433d4 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 39a635a070b52..917c06303865c 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(); } -- 2.39.5