os/bluestore: do not release empty bluefs_extents_reclaiming.

This saves us one mutex lock attempt in Stupid Allocator.

Signed-off-by: Igor Fedotov <ifedotov@suse.com>

diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc
index 717777f9088ffd81c83bff84f7b417c2ecc56df9..95d970dbc3f69e8556b05563e5d033353163e150 100644 (file)
--- 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();