crimson/os/seastore: remove CachedExtent move constructor

I'm not sure why I added this in the first place, there aren't really
any scenarios where invoking it wouldn't be a bug.

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

diff --git a/src/crimson/os/seastore/cached_extent.h b/src/crimson/os/seastore/cached_extent.h
index e242d17499600ab18af2da973113eb7291318c72..c5fedf2adafb34dd0b920342769e57c83835da51 100644 (file)
--- a/src/crimson/os/seastore/cached_extent.h
+++ b/src/crimson/os/seastore/cached_extent.h
@@ -313,7 +313,7 @@ private:
   }
 
 protected:
-  CachedExtent(CachedExtent &&other) = default;
+  CachedExtent(CachedExtent &&other) = delete;
   CachedExtent(ceph::bufferptr &&ptr) : ptr(std::move(ptr)) {}
   CachedExtent(const CachedExtent &other)
     : state(other.state),

diff --git a/src/crimson/os/seastore/onode_manager/simple-fltree/onode_block.h b/src/crimson/os/seastore/onode_manager/simple-fltree/onode_block.h
index 892388be0c48ab927467227ae83d22498a22f570..9f53bef81053214ea4c4b506e9cfd337a46c7113 100644 (file)
--- a/src/crimson/os/seastore/onode_manager/simple-fltree/onode_block.h
+++ b/src/crimson/os/seastore/onode_manager/simple-fltree/onode_block.h
@@ -15,10 +15,7 @@ struct OnodeBlock final : LogicalCachedExtent {
 
   template <typename... T>
   OnodeBlock(T&&... t) : LogicalCachedExtent(std::forward<T>(t)...) {}
-  OnodeBlock(OnodeBlock&& block) noexcept
-    : LogicalCachedExtent{std::move(block)},
-      deltas{std::move(block.deltas)}
-  {}
+  OnodeBlock(OnodeBlock&& block) = delete;
   OnodeBlock(const OnodeBlock& block, CachedExtent::share_buffer_t tag) noexcept
     : LogicalCachedExtent{block, tag},
       share_buffer{true}