]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
osd: drop unused osd_disk_tp related options 21339/head
authorGu Zhongyan <guzhongyan@360.cn>
Wed, 11 Apr 2018 04:26:35 +0000 (12:26 +0800)
committerGu Zhongyan <guzhongyan@360.cn>
Thu, 26 Apr 2018 03:40:45 +0000 (11:40 +0800)
disk_tp has been killed. So those are unused.

Signed-off-by: Gu Zhongyan <guzhongyan@360.cn>
src/common/options.cc
src/osd/OSD.cc
src/test/test_workqueue.cc

index fb6914d0dda679f3ede491af561ccdb15c1d0ac4..bfdbe46ca5a795072ddd2321f6fd7cc467f961dc 100644 (file)
@@ -2275,10 +2275,6 @@ std::vector<Option> get_global_options() {
     .set_default(65536)
     .set_description(""),
 
-    Option("osd_disk_threads", Option::TYPE_INT, Option::LEVEL_ADVANCED)
-    .set_default(1)
-    .set_description(""),
-
     Option("osd_recover_clone_overlap", Option::TYPE_BOOL, Option::LEVEL_ADVANCED)
     .set_default(true)
     .set_description(""),
index f86b3c25f60eb4e113b987068ef527a8b085031f..adced6ae8373e9c2382f70ef6fdcd4a06d8b86da 100644 (file)
@@ -51,7 +51,6 @@
 #include "common/ceph_argparse.h"
 #include "common/ceph_time.h"
 #include "common/version.h"
-#include "common/io_priority.h"
 #include "common/pick_address.h"
 #include "common/SubProcess.h"
 #include "common/PluginRegistry.h"
@@ -8960,8 +8959,6 @@ const char** OSD::get_tracked_conf_keys() const
     "osd_map_cache_size",
     "osd_pg_epoch_max_lag_factor",
     "osd_pg_epoch_persisted_max_stale",
-    "osd_disk_thread_ioprio_class",
-    "osd_disk_thread_ioprio_priority",
     // clog & admin clog
     "clog_to_monitors",
     "clog_to_syslog",
index 8bf6562f4ffe11f12d40335bcfacbb91a39ac834..f43e267bd6053089cdc5660d9991c73b0b8a3be8 100644 (file)
@@ -18,34 +18,34 @@ TEST(WorkQueue, StartStop)
 
 TEST(WorkQueue, Resize)
 {
-  ThreadPool tp(g_ceph_context, "bar", "tp_bar", 2, "osd_disk_threads");
+  ThreadPool tp(g_ceph_context, "bar", "tp_bar", 2, "filestore_op_threads");
   
   tp.start();
 
   sleep(1);
   ASSERT_EQ(2, tp.get_num_threads());
 
-  g_conf->set_val("osd disk threads", "5");
+  g_conf->set_val("filestore op threads", "5");
   g_conf->apply_changes(&cout);
   sleep(1);
   ASSERT_EQ(5, tp.get_num_threads());
 
-  g_conf->set_val("osd disk threads", "3");
+  g_conf->set_val("filestore op threads", "3");
   g_conf->apply_changes(&cout);
   sleep(1);
   ASSERT_EQ(3, tp.get_num_threads());
 
-  g_conf->set_val("osd disk threads", "0");
+  g_conf->set_val("filestore op threads", "0");
   g_conf->apply_changes(&cout);
   sleep(1);
   ASSERT_EQ(0, tp.get_num_threads());
 
-  g_conf->set_val("osd disk threads", "15");
+  g_conf->set_val("filestore op threads", "15");
   g_conf->apply_changes(&cout);
   sleep(1);
   ASSERT_EQ(15, tp.get_num_threads());
 
-  g_conf->set_val("osd disk threads", "-1");
+  g_conf->set_val("filestore op threads", "-1");
   g_conf->apply_changes(&cout);
   sleep(1);
   ASSERT_EQ(15, tp.get_num_threads());