From: Sage Weil <sage@redhat.com>
Date: Tue, 8 Aug 2017 16:14:07 +0000 (-0400)
Subject: os/bluestore: default to bluestore_allocator = stupid
X-Git-Tag: v12.2.1~85^2~3
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=3e06813fe92c0ad07ac033aab14d088136b996f9;p=ceph.git

os/bluestore: default to bluestore_allocator = stupid

Performance looks better, and code is simpler.

Also fix config option annotations.

Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit 3750cf9cafabd3296c22b8b8dff35e44d18b9dd8)
---

diff --git a/src/common/options.cc b/src/common/options.cc
index 6166ef3f8d50..f19ca2d5c130 100644
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -3396,10 +3396,10 @@ std::vector<Option> get_global_options() {
     .add_tag("mkfs")
     .set_description("Key value database to use for bluestore"),
 
-    Option("bluestore_allocator", Option::TYPE_STR, Option::LEVEL_DEV)
-    .set_default("bitmap")
-    .add_tag("mkfs")
-    .set_description(""),
+    Option("bluestore_allocator", Option::TYPE_STR, Option::LEVEL_ADVANCED)
+    .set_default("stupid")
+    .set_enum_allowed({"bitmap", "stupid"})
+    .set_description("Allocator policy"),
 
     Option("bluestore_freelist_blocks_per_key", Option::TYPE_INT, Option::LEVEL_DEV)
     .set_default(128)