From f4073002c48a2533b24f38fd41ddacf3160bdfef Mon Sep 17 00:00:00 2001 From: Igor Fedotov Date: Wed, 1 Nov 2017 07:14:25 -0700 Subject: [PATCH] os/bluestore: do not release empty bluefs_extents_reclaiming. This saves us one mutex lock attempt in Stupid Allocator. Signed-off-by: Igor Fedotov --- src/os/bluestore/BlueStore.cc | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc index 717777f9088..95d970dbc3f 100644 --- a/src/os/bluestore/BlueStore.cc +++ b/src/os/bluestore/BlueStore.cc @@ -8708,10 +8708,12 @@ void BlueStore::_kv_sync_thread() if (!bluefs_gift_extents.empty()) { _commit_bluefs_freespace(bluefs_gift_extents); } - dout(20) << __func__ << " releasing old bluefs 0x" << std::hex - << bluefs_extents_reclaiming << std::dec << dendl; - alloc->release(bluefs_extents_reclaiming); - bluefs_extents_reclaiming.clear(); + if (!bluefs_extents_reclaiming.empty()) { + dout(0) << __func__ << " releasing old bluefs 0x" << std::hex + << bluefs_extents_reclaiming << std::dec << dendl; + alloc->release(bluefs_extents_reclaiming); + bluefs_extents_reclaiming.clear(); + } } l.lock();