]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
os/bluestore/BlueRocksEnv: Avoid flushing too much data at once. 36465/head
authorJianpeng Ma <jianpeng.ma@intel.com>
Mon, 10 Aug 2020 07:56:13 +0000 (15:56 +0800)
committerJianpeng Ma <jianpeng.ma@intel.com>
Mon, 10 Aug 2020 07:56:13 +0000 (15:56 +0800)
commit3a1d4ff6dcdcd897d31a85ef82c726599624b66d
treedd2900202fcf6ccb0710573efc90985514ad577a
parentbb2594200b2a743ce61485ce4d2a27343925d85b
os/bluestore/BlueRocksEnv: Avoid flushing too much data at once.

Although, in _flush func we already check length. If length of dirty
is less then bluefs_min_flush_size, we will skip this flush.
But in fact, we found rocksdb can call many times Append() and then
call Flush(). This make flush_data is much larger than
bluefs_min_flush_size.

From my test, w/o this patch, it can reduce 99.99% latency(from
145.753ms to 20.474ms) for 4k randwrite with bluefs_buffered_io=true.

Because Bluefs::flush acquire lock. So we add new api try_flush
to avoid lock contention.

Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>
src/os/bluestore/BlueFS.h
src/os/bluestore/BlueRocksEnv.cc