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 c575d5320780fe6e9c355b425649ea2823b1a829..7df77da85817bf46c2de4748d3a29b529ad7ee57 100644 (file)
--- a/src/crimson/os/seastore/lba/btree_lba_manager.cc
+++ b/src/crimson/os/seastore/lba/btree_lba_manager.cc
@@ -1076,7 +1076,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 b8b9a1f373b62a5a537146bc3c08eb45cb36cb81..c48a8f99e5c8badad2bd6fc45810fe0be1a2a991 100644 (file)
--- a/src/crimson/os/seastore/lba_manager.h
+++ b/src/crimson/os/seastore/lba_manager.h
@@ -54,7 +54,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 43b12e447a394b891d909442ea407e7610cb535d..135b99ebe946eb3a6338078962fba7212192bc6a 100644 (file)
--- 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(