crimson/os/seastore: remove unneeded TransactionManager::find_hole

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

diff --git a/src/crimson/os/seastore/lba_manager.h b/src/crimson/os/seastore/lba_manager.h
index 2ad5b4596df2171e9eb7e59bf7ac6d7ff4393864..958ca71ee407cfadddd82311860a56e7e1f5d780 100644 (file)
--- a/src/crimson/os/seastore/lba_manager.h
+++ b/src/crimson/os/seastore/lba_manager.h
@@ -68,18 +68,6 @@ public:
     Transaction &t,
     laddr_t offset) = 0;
 
-  /**
-   * Finds unmapped laddr extent of len len
-   */
-  using find_hole_iertr = base_iertr;
-  using find_hole_ret = find_hole_iertr::future<
-    std::pair<laddr_t, extent_len_t>
-    >;
-  virtual find_hole_ret find_hole(
-    Transaction &t,
-    laddr_t hint,
-    extent_len_t) = 0;
-
   /**
    * Allocates a new mapping referenced by LBARef
    *

diff --git a/src/crimson/os/seastore/lba_manager/btree/btree_lba_manager.cc b/src/crimson/os/seastore/lba_manager/btree/btree_lba_manager.cc
index 0202423c014d90dc53cf0de06fdb60d4d4c3f481..278c0958fa990f9cca25d6e511fcca4dead4d9df 100644 (file)
--- a/src/crimson/os/seastore/lba_manager/btree/btree_lba_manager.cc
+++ b/src/crimson/os/seastore/lba_manager/btree/btree_lba_manager.cc
@@ -117,24 +117,6 @@ BtreeLBAManager::get_mapping(
   });
 }
 
-BtreeLBAManager::find_hole_ret
-BtreeLBAManager::find_hole(
-  Transaction &t,
-  laddr_t hint,
-  extent_len_t len)
-{
-  return get_root(t
-  ).si_then([this, hint, len, &t](auto extent) {
-    return extent->find_hole(
-      get_context(t),
-      hint,
-      L_ADDR_MAX,
-      len);
-  }).si_then([len](auto addr) {
-    return std::make_pair(addr, len);
-  });
-}
-
 BtreeLBAManager::alloc_extent_ret
 BtreeLBAManager::alloc_extent(
   Transaction &t,

diff --git a/src/crimson/os/seastore/lba_manager/btree/btree_lba_manager.h b/src/crimson/os/seastore/lba_manager/btree/btree_lba_manager.h
index b562c8cb62bb3373ce9169228678942d45d933d7..0c16efaca74ba639a76340346062f155d4ddf5b8 100644 (file)
--- a/src/crimson/os/seastore/lba_manager/btree/btree_lba_manager.h
+++ b/src/crimson/os/seastore/lba_manager/btree/btree_lba_manager.h
@@ -62,11 +62,6 @@ public:
     Transaction &t,
     laddr_t offset) final;
 
-  find_hole_ret find_hole(
-    Transaction &t,
-    laddr_t hint,
-    extent_len_t) final;
-
   alloc_extent_ret alloc_extent(
     Transaction &t,
     laddr_t hint,

diff --git a/src/crimson/os/seastore/transaction_manager.h b/src/crimson/os/seastore/transaction_manager.h
index db229b12e23505a970fa4a1488f7a70fc8d3e1cc..03a0d32a8753dedd3dbce69da2fcd98fd826fc87 100644 (file)
--- a/src/crimson/os/seastore/transaction_manager.h
+++ b/src/crimson/os/seastore/transaction_manager.h
@@ -312,20 +312,6 @@ public:
       zero_paddr());
   }
 
-  using find_hole_iertr = LBAManager::find_hole_iertr;
-  using find_hole_ret = LBAManager::find_hole_iertr::future<
-    std::pair<laddr_t, extent_len_t>
-    >;
-  find_hole_ret find_hole(
-    Transaction &t,
-    laddr_t hint,
-    extent_len_t len) {
-    return lba_manager->find_hole(
-      t,
-      hint,
-      len);
-  }
-
   /* alloc_extents
    *
    * allocates more than one new blocks of type T.