From: Samuel Just <sjust@redhat.com>
Date: Wed, 15 Oct 2025 21:58:34 +0000 (+0000)
Subject: crimson/.../btree_lba_manager: convert get_end_mapping to return LBACursorRef
X-Git-Tag: v21.0.0~99^2~16
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=31443e0f794c294234c35dd2c42dd19153565f3c;p=ceph.git

crimson/.../btree_lba_manager: convert get_end_mapping to return LBACursorRef

Signed-off-by: Samuel Just <sjust@redhat.com>
---

diff --git a/src/crimson/os/seastore/lba/btree_lba_manager.cc b/src/crimson/os/seastore/lba/btree_lba_manager.cc
index b9274679d26a..2c5fcb750681 100644
--- a/src/crimson/os/seastore/lba/btree_lba_manager.cc
+++ b/src/crimson/os/seastore/lba/btree_lba_manager.cc
@@ -1134,7 +1134,7 @@ BtreeLBAManager::get_end_mapping(
   auto c = get_context(t);
   auto btree = co_await get_btree<LBABtree>(cache, c);
   auto iter = co_await btree.end(c);
-  co_return LBAMapping::create_direct(iter.get_cursor(c));
+  co_return iter.get_cursor(c);
 }
 #endif
 
diff --git a/src/crimson/os/seastore/lba_manager.h b/src/crimson/os/seastore/lba_manager.h
index a4c001eb9d3b..0ec0c58c140f 100644
--- a/src/crimson/os/seastore/lba_manager.h
+++ b/src/crimson/os/seastore/lba_manager.h
@@ -57,7 +57,7 @@ public:
 
 #ifdef UNIT_TESTS_BUILT
   using get_end_mapping_iertr = base_iertr;
-  using get_end_mapping_ret = get_end_mapping_iertr::future<LBAMapping>;
+  using get_end_mapping_ret = get_end_mapping_iertr::future<LBACursorRef>;
   virtual get_end_mapping_ret get_end_mapping(Transaction &t) = 0;
 #endif
 
diff --git a/src/test/crimson/seastore/test_transaction_manager.cc b/src/test/crimson/seastore/test_transaction_manager.cc
index 137dbe0d0d18..ffd8ea55c158 100644
--- a/src/test/crimson/seastore/test_transaction_manager.cc
+++ b/src/test/crimson/seastore/test_transaction_manager.cc
@@ -726,9 +726,9 @@ struct transaction_manager_test_t :
   }
 
   LBAMapping get_end(test_transaction_t &t) {
-    return with_trans_intr(*(t.t), [&](auto &trans) {
+    return LBAMapping::create_direct(with_trans_intr(*(t.t), [&](auto &trans) {
       return lba_manager->get_end_mapping(trans);
-    }).unsafe_get();
+    }).unsafe_get());
   }
 
   std::optional<LBAMapping> try_get_pin(