From 510e33d2122e2e1d78b4bed9b585832bbf84e987 Mon Sep 17 00:00:00 2001 From: locallocal Date: Mon, 6 Dec 2021 10:01:50 +0800 Subject: [PATCH] os/bluestore: don't need separate variable to mark hits when lookup oid. Signed-off-by: locallocal (cherry picked from commit 1428544ec66b498830bc884b4824cd90106053d5) --- src/os/bluestore/BlueStore.cc | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc index 1cc0f8fb5c983..878e2f92a27f0 100644 --- a/src/os/bluestore/BlueStore.cc +++ b/src/os/bluestore/BlueStore.cc @@ -1869,12 +1869,12 @@ BlueStore::OnodeRef BlueStore::OnodeSpace::lookup(const ghobject_t& oid) { ldout(cache->cct, 30) << __func__ << dendl; OnodeRef o; - bool hit = false; { std::lock_guard l(cache->lock); ceph::unordered_map::iterator p = onode_map.find(oid); if (p == onode_map.end()) { + cache->logger->inc(l_bluestore_onode_misses); ldout(cache->cct, 30) << __func__ << " " << oid << " miss" << dendl; cache->logger->inc(l_bluestore_onode_misses); } else { @@ -1886,15 +1886,10 @@ BlueStore::OnodeRef BlueStore::OnodeSpace::lookup(const ghobject_t& oid) // eventually will become unpinned o = p->second; - hit = true; + cache->logger->inc(l_bluestore_onode_hits); } } - if (hit) { - cache->logger->inc(l_bluestore_onode_hits); - } else { - cache->logger->inc(l_bluestore_onode_misses); - } return o; } -- 2.39.5