From 39bf8e7f50ac9a58a5159e25f65190a350601208 Mon Sep 17 00:00:00 2001
From: Patrick Donnelly <pdonnell@redhat.com>
Date: Mon, 14 Dec 2020 13:27:08 -0800
Subject: [PATCH] mds: update defaults for recall configs

This is based on informal experience reportings from production
clusters.

Fixes: https://tracker.ceph.com/issues/48403
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 8df2388b9fb66e1606f47c095ecf0b5c71a1941e)

Conflicts:
	src/common/options.cc
- nautilus does not have FLAG_RUNTIME
---
 src/common/options.cc | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/common/options.cc b/src/common/options.cc
index d921bf1c8aa68..50020c455522b 100644
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -7775,7 +7775,7 @@ std::vector<Option> get_mds_options() {
     .set_description("decay rate for trimming MDS cache throttle"),
 
     Option("mds_cache_trim_threshold", Option::TYPE_SIZE, Option::LEVEL_ADVANCED)
-    .set_default(64_K)
+    .set_default(256_K)
     .set_description("threshold for number of dentries that can be trimmed"),
 
     Option("mds_max_file_recover", Option::TYPE_UINT, Option::LEVEL_ADVANCED)
@@ -7823,23 +7823,23 @@ std::vector<Option> get_mds_options() {
     .set_description("number of omap keys to read from the SessionMap in one operation"),
 
     Option("mds_recall_max_caps", Option::TYPE_SIZE, Option::LEVEL_ADVANCED)
-    .set_default(5000)
+    .set_default(30000)
     .set_description("maximum number of caps to recall from client session in single recall"),
 
     Option("mds_recall_max_decay_rate", Option::TYPE_FLOAT, Option::LEVEL_ADVANCED)
-    .set_default(2.5)
+    .set_default(1.5)
     .set_description("decay rate for throttle on recalled caps on a session"),
 
     Option("mds_recall_max_decay_threshold", Option::TYPE_SIZE, Option::LEVEL_ADVANCED)
-    .set_default(16_K)
+    .set_default(128_K)
     .set_description("decay threshold for throttle on recalled caps on a session"),
 
     Option("mds_recall_global_max_decay_threshold", Option::TYPE_SIZE, Option::LEVEL_ADVANCED)
-    .set_default(64_K)
+    .set_default(128_K)
     .set_description("decay threshold for throttle on recalled caps globally"),
 
     Option("mds_recall_warning_threshold", Option::TYPE_SIZE, Option::LEVEL_ADVANCED)
-    .set_default(32_K)
+    .set_default(256_K)
     .set_description("decay threshold for warning on slow session cap recall"),
 
     Option("mds_recall_warning_decay_rate", Option::TYPE_FLOAT, Option::LEVEL_ADVANCED)
-- 
2.39.5