]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
common/options: bluestore 64k min_alloc_size for HDD
authorSage Weil <sage@redhat.com>
Thu, 23 Jan 2020 17:23:38 +0000 (11:23 -0600)
committerSage Weil <sage@redhat.com>
Thu, 23 Jan 2020 17:24:36 +0000 (11:24 -0600)
Until we understand the performance regression and allocator behavior,
go back to 64k.  This will continue to have a high space overhead for
small objects and EC, but will preserve the current performance levels
for all workloads.

This partially reverts 0ec75c99ddb103c664f66c04450003ed4c407708

Signed-off-by: Sage Weil <sage@redhat.com>
src/common/options.cc

index 75213f11a0ca442c4be5efe67d017f7e84ac4d83..a4a73833378cc87a64eae6cba4bf8e9c7f84d0ad 100644 (file)
@@ -4161,7 +4161,7 @@ std::vector<Option> get_global_options() {
     .set_long_description("A smaller allocation size generally means less data is read and then rewritten when a copy-on-write operation is triggered (e.g., when writing to something that was recently snapshotted).  Similarly, less data is journaled before performing an overwrite (writes smaller than min_alloc_size must first pass through the BlueStore journal).  Larger values of min_alloc_size reduce the amount of metadata required to describe the on-disk layout and reduce overall fragmentation."),
 
     Option("bluestore_min_alloc_size_hdd", Option::TYPE_SIZE, Option::LEVEL_ADVANCED)
-    .set_default(4_K)
+    .set_default(64_K)
     .set_flag(Option::FLAG_CREATE)
     .set_description("Default min_alloc_size value for rotational media")
     .add_see_also("bluestore_min_alloc_size"),