From: Igor Fedotov <ifedotov@suse.com>
Date: Mon, 5 Mar 2018 18:09:36 +0000 (+0300)
Subject: os/bluestore: add debug option to disable per pool statistics collection
X-Git-Tag: v14.1.0~692^2~15
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=6a01f368536de6e3539d0914211e2e24966e79ad;p=ceph.git

os/bluestore: add debug option to disable per pool statistics collection

Signed-off-by: Igor Fedotov <ifedotov@suse.com>
---

diff --git a/src/common/legacy_config_opts.h b/src/common/legacy_config_opts.h
index 2ee8f85936c8..9e1a06082304 100644
--- a/src/common/legacy_config_opts.h
+++ b/src/common/legacy_config_opts.h
@@ -1073,6 +1073,7 @@ OPTION(bluestore_debug_permit_any_bdev_label, OPT_BOOL)
 OPTION(bluestore_debug_random_read_err, OPT_DOUBLE)
 OPTION(bluestore_debug_inject_bug21040, OPT_BOOL)
 OPTION(bluestore_debug_inject_csum_err_probability, OPT_FLOAT)
+OPTION(bluestore_debug_no_per_pool_stats, OPT_BOOL)
 
 OPTION(kstore_max_ops, OPT_U64)
 OPTION(kstore_max_bytes, OPT_U64)
diff --git a/src/common/options.cc b/src/common/options.cc
index 3d03c32b6e5d..19c771000691 100644
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -4385,6 +4385,10 @@ std::vector<Option> get_global_options() {
     .set_default(0.0)
     .set_description("inject crc verification errors into bluestore device reads"),
 
+    Option("bluestore_debug_no_per_pool_stats", Option::TYPE_BOOL, Option::LEVEL_DEV)
+    .set_default(false)
+    .set_description(""),
+    
     // -----------------------------------------
     // kstore
 
diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc
index 1494af6f2bfc..4f57253b982d 100644
--- a/src/os/bluestore/BlueStore.cc
+++ b/src/os/bluestore/BlueStore.cc
@@ -7648,7 +7648,8 @@ int BlueStore::statfs(struct store_statfs_t *buf)
 int BlueStore::pool_statfs(uint64_t pool_id, struct store_statfs_t *buf)
 {
   dout(20) << __func__ << " pool " << pool_id<< dendl;
-  if (!per_pool_stat_collection) {
+  if (!per_pool_stat_collection ||
+      cct->_conf->bluestore_debug_no_per_pool_stats) {
     dout(20) << __func__ << " not supported in a legacy mode " << dendl;
     return -ENOTSUP;
   }