RepoMirrors/ceph
1
0
Fork 0
mirror of https://github.com/ceph/ceph synced 2024-12-28 22:43:29 +00:00
Code Issues Packages Projects Releases Wiki Activity

config: Change bluestore_cache_kv_max to type INT64

Browse Source 
If bluestore_cache_kv_max is negative, bluestore won't calculate meta
ratio and kv ratio automatically. So change the type from UINT to INT to
make this bluestore logic work.

Signed-off-by: Zhi Zhang <zhangz.david@outlook.com>
This commit is contained in:
Zhi Zhang 2018-02-02 16:02:05 +08:00
parent 7dabdf526d
commit a42a4d405c
2 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/common/legacy_config_opts.h
View File

@ -1029,7 +1029,7 @@ OPTION(bluestore_cache_size_hdd, OPT_U64)
OPTION(bluestore_cache_size_ssd, OPT_U64)
OPTION(bluestore_cache_meta_ratio, OPT_DOUBLE)
OPTION(bluestore_cache_kv_ratio, OPT_DOUBLE)
OPTION(bluestore_cache_kv_max, OPT_U64) // limit the maximum amount of cache for the kv store
OPTION(bluestore_cache_kv_max, OPT_INT) // limit the maximum amount of cache for the kv store
OPTION(bluestore_kvbackend, OPT_STR)
OPTION(bluestore_allocator, OPT_STR) // stupid | bitmap
OPTION(bluestore_freelist_blocks_per_key, OPT_INT)

7
src/common/options.cc
View File

@ -3733,9 +3733,12 @@ std::vector<Option> get_global_options() {
.set_default(.99)
.set_description("Ratio of bluestore cache to devote to kv database (rocksdb)"),
Option("bluestore_cache_kv_max", Option::TYPE_UINT, Option::LEVEL_ADVANCED)
Option("bluestore_cache_kv_max", Option::TYPE_INT, Option::LEVEL_ADVANCED)
.set_default(512_M)
.set_description("Max memory (bytes) to devote to kv database (rocksdb)"),
.set_description("Max memory (bytes) to devote to kv database (rocksdb)")
.set_long_description("A negative value means using bluestore_cache_meta_ratio "
"and bluestore_cache_kv_ratio instead of calculating these ratios using "
"bluestore_cache_size_* and bluestore_cache_kv_max."),
Option("bluestore_kvbackend", Option::TYPE_STR, Option::LEVEL_DEV)
.set_default("rocksdb")