crimson/osd/object_context_loader: cleanup `loaded`

Signed-off-by: Matan Breizman <mbreizma@redhat.com>
(cherry picked from commit 5b5d2ea4aebcbbe3fec349377337fe38f5a2f27a)

diff --git a/src/crimson/osd/object_context_loader.cc b/src/crimson/osd/object_context_loader.cc
index 9af927806d568f22f3b4e3c700f2d8a47faf5038..79185a9460e9bfcd9ca8dde2f6cfdbb50ab716f7 100644 (file)
--- a/src/crimson/osd/object_context_loader.cc
+++ b/src/crimson/osd/object_context_loader.cc
@@ -88,8 +88,8 @@ using crimson::common::local_conf;
       [existed=existed, clone=std::move(clone),
        func=std::move(func), head=std::move(head), this]() mutable
       -> load_obc_iertr::future<> {
-      auto loaded = get_or_load_obc<State>(clone, existed);
-      return loaded.safe_then_interruptible(
+      return get_or_load_obc<State>(clone, existed
+      ).safe_then_interruptible(
         [func = std::move(func), head=std::move(head)](auto clone) mutable {
         clone->set_clone_ssc(head->ssc);
         return std::move(func)(std::move(head), std::move(clone));
@@ -151,20 +151,17 @@ using crimson::common::local_conf;
              obc->invalidated_by_interval_change);
     return interruptor::with_lock(obc->loading_mutex,
     [this, obc, existed, FNAME] {
-      auto loaded =
-        load_obc_iertr::make_ready_future<ObjectContextRef>(obc);
       if (existed) {
         ceph_assert(obc->is_valid() && obc->is_loaded());
         DEBUGDPP("cache hit on {}", dpp, obc->get_oid());
+        return load_obc_iertr::make_ready_future<ObjectContextRef>(obc);
       } else {
         DEBUGDPP("cache miss on {}", dpp, obc->get_oid());
-        loaded =
-          obc->template with_promoted_lock<State, IOInterruptCondition>(
+        return obc->template with_promoted_lock<State, IOInterruptCondition>(
           [obc, this] {
           return load_obc(obc);
         });
       }
-      return loaded;
     });
   }