From: Mark Nelson <mnelson@redhat.com>
Date: Thu, 17 Oct 2019 13:50:37 +0000 (-0400)
Subject: osd/OSD: Set the number of cache shards independently.  Default to 32
X-Git-Tag: v15.1.0~1172^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F30964%2Fhead;p=ceph.git

osd/OSD: Set the number of cache shards independently.  Default to 32

Signed-off-by: Mark Nelson <mnelson@redhat.com>
---

diff --git a/src/common/options.cc b/src/common/options.cc
index 0ebbe4b3037c..e05aa13c0e77 100644
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -2745,6 +2745,11 @@ std::vector<Option> get_global_options() {
     .set_default(true)
     .set_description(""),
 
+    Option("osd_num_cache_shards", Option::TYPE_SIZE, Option::LEVEL_ADVANCED)
+    .set_default(32)
+    .set_flag(Option::FLAG_STARTUP)
+    .set_description("The number of cache shards to use in the object store."),
+
     Option("osd_op_num_threads_per_shard", Option::TYPE_INT, Option::LEVEL_ADVANCED)
     .set_default(0)
     .set_flag(Option::FLAG_STARTUP)
diff --git a/src/osd/OSD.cc b/src/osd/OSD.cc
index 96aed0b706e3..6840d9ed7e91 100644
--- a/src/osd/OSD.cc
+++ b/src/osd/OSD.cc
@@ -2803,6 +2803,11 @@ int OSD::enable_disable_fuse(bool stop)
   return 0;
 }
 
+size_t OSD::get_num_cache_shards()
+{
+  return cct->_conf.get_val<Option::size_t>("osd_num_cache_shards");
+}
+
 int OSD::get_num_op_shards()
 {
   if (cct->_conf->osd_op_num_shards)
@@ -2896,7 +2901,7 @@ int OSD::init()
   dout(2) << "journal " << journal_path << dendl;
   ceph_assert(store);  // call pre_init() first!
 
-  store->set_cache_shards(get_num_op_shards());
+  store->set_cache_shards(get_num_cache_shards());
 
   int r = store->mount();
   if (r < 0) {
diff --git a/src/osd/OSD.h b/src/osd/OSD.h
index ae2fafd201fb..f3f37d74ef8f 100644
--- a/src/osd/OSD.h
+++ b/src/osd/OSD.h
@@ -2162,6 +2162,7 @@ private:
 
   int init_op_flags(OpRequestRef& op);
 
+  size_t get_num_cache_shards();
   int get_num_op_shards();
   int get_num_op_threads();