From: Mark Nelson <mnelson@redhat.com>
Date: Fri, 2 Jul 2021 14:19:30 +0000 (+0000)
Subject: common/options: Set osd_client_message_cap to 256.
X-Git-Tag: v16.2.6~45^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=6ce3062a28c2c08592cd182f5cc3f3025a8eba88;p=ceph.git

common/options: Set osd_client_message_cap to 256.

This seems like a reasonable default value based on testing results here:
https://docs.google.com/spreadsheets/d/1dwKcxFKpAOWzDPekgojrJhfiCtPgiIf8CGGMG1rboRU/edit?usp=sharing

Eventually we may want to rethink how the throttles and even how flow control
works, but this at least gives us some basic limits now ( a little higher than
the old value of 100 that we used for many years).

Signed-off-by: Mark Nelson <mnelson@redhat.com>
(cherry picked from commit ac8cf275a6d191d71c104f6822b62ba67a0a4fcd)

 Conflicts:
	src/common/options/osd.yaml.in - file does not exist in pacific
---

diff --git a/src/common/options.cc b/src/common/options.cc
index 96bbf56c1a023..41c1e69eac9fe 100644
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -2678,7 +2678,7 @@ std::vector<Option> get_global_options() {
     .set_long_description("If this value is exceeded, the OSD will not read any new client data off of the network until memory is freed."),
 
     Option("osd_client_message_cap", Option::TYPE_UINT, Option::LEVEL_ADVANCED)
-    .set_default(0)
+    .set_default(256)
     .set_description("maximum number of in-flight client requests"),
 
     Option("osd_crush_update_weight_set", Option::TYPE_BOOL, Option::LEVEL_ADVANCED)