]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
os/bluestore: bluestore_cache_kv_max -> bluestore_cache_kv_min
authorSage Weil <sage@redhat.com>
Thu, 22 Feb 2018 19:15:17 +0000 (13:15 -0600)
committerSage Weil <sage@redhat.com>
Wed, 28 Feb 2018 14:27:02 +0000 (08:27 -0600)
commit7a73d9b155bdd9e8cfffbf4968e57e4fbd95ac65
tree403a304bee4be0b03e64e0f7ae5e98cb9b76c18f
parentfbb62621ac5727dcb6a140aa05cb0c38285b0576
os/bluestore: bluestore_cache_kv_max -> bluestore_cache_kv_min

Currently, the default tunables are
  cache_meta_ratio = .01
  cache_kv_ratio = .99
  cache_kv_max = 512MB

We currently have no doubts that, for low memory situations, we want to
give all of the memory to rocksdb.  The .99/.01 values are an indirect
way to communicating that.  However, we do see problems with high-memory
situations where rocksdb needs more than 512MB (due to, we believe,
bloom filters). And it is very reasonable to think that there are
situations and reasons why we might want more than 512MB for rocksdb.

So, we change kv_max -> kv_min.  For low memory situations, we give *all*
memory to rocksdb.  (If you don't want this, set kv_min = 0.)  Above that,
respect the kv_ratio and meta_ratio settings.

I don't see much reason to have an absolute kv_max on the rocksdb memory.

We adjust the kv_ratio and meta_ratio defaults in this patch so that all
high memory goes to bluestore and there is no effective change in
default behavior for this patch.

Signed-off-by: Sage Weil <sage@redhat.com>
src/common/legacy_config_opts.h
src/common/options.cc
src/os/bluestore/BlueStore.cc