crimson/os/seastore: remove unecessary public interfaces

Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
(cherry picked from commit 785634262b6c39df44f3e88ab326421f75b06c98)

diff --git a/src/crimson/os/seastore/cache.h b/src/crimson/os/seastore/cache.h
index 5093c79b81c1f1a7a37df1df420a7bedcd50d54d..86e63aa6c11e901c667556934536542760ea1089 100644 (file)
--- a/src/crimson/os/seastore/cache.h
+++ b/src/crimson/os/seastore/cache.h
@@ -1343,6 +1343,7 @@ public:
     return stats.omap_tree_depth;
   }
 
+private:
   /// Update lru for access to ref
   void touch_extent(
       CachedExtent &ext,
@@ -1355,7 +1356,6 @@ public:
     }
   }
 
-private:
   ExtentPlacementManager& epm;
   RootBlockRef root;               ///< ref to current root
   ExtentIndex extents;             ///< set of live extents

diff --git a/src/crimson/os/seastore/cached_extent.h b/src/crimson/os/seastore/cached_extent.h
index 4cb7e2b490cbdbf6a6fd11c20c99d44f148f8f70..35753d101bd25c19a48ac3b0967644270a03ee1e 100644 (file)
--- a/src/crimson/os/seastore/cached_extent.h
+++ b/src/crimson/os/seastore/cached_extent.h
@@ -1244,8 +1244,6 @@ public:
 
   std::ostream &_print_detail(std::ostream &out) const final;
 
-  void on_replace_prior(Transaction &t) final;
-
   struct modified_region_t {
     extent_len_t offset;
     extent_len_t len;
@@ -1257,9 +1255,12 @@ public:
   virtual void clear_modified_region() {}
 
   virtual ~LogicalCachedExtent();
+
 protected:
+  void on_replace_prior(Transaction &t) final;
 
   virtual void apply_delta(const ceph::bufferlist &bl) = 0;
+
   virtual std::ostream &print_detail_l(std::ostream &out) const {
     return out;
   }