From 35a4b5072f36df92391b7514eafdea498e42f1dd Mon Sep 17 00:00:00 2001
From: Gu Zhongyan <guzhongyan@360.cn>
Date: Wed, 11 Apr 2018 12:26:35 +0800
Subject: [PATCH] osd: drop unused osd_disk_tp related options

disk_tp has been killed. So those are unused.

Signed-off-by: Gu Zhongyan <guzhongyan@360.cn>
---
 src/common/options.cc      |  4 ----
 src/osd/OSD.cc             |  3 ---
 src/test/test_workqueue.cc | 12 ++++++------
 3 files changed, 6 insertions(+), 13 deletions(-)

diff --git a/src/common/options.cc b/src/common/options.cc
index fb6914d0dda67..bfdbe46ca5a79 100644
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -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(""),
diff --git a/src/osd/OSD.cc b/src/osd/OSD.cc
index f86b3c25f60eb..adced6ae8373e 100644
--- a/src/osd/OSD.cc
+++ b/src/osd/OSD.cc
@@ -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",
diff --git a/src/test/test_workqueue.cc b/src/test/test_workqueue.cc
index 8bf6562f4ffe1..f43e267bd6053 100644
--- a/src/test/test_workqueue.cc
+++ b/src/test/test_workqueue.cc
@@ -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());
-- 
2.39.5