]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
crimson/osd/object_context_loader: Hide private methods 51624/head
authorMatan Breizman <mbreizma@redhat.com>
Sun, 5 Mar 2023 10:22:10 +0000 (10:22 +0000)
committerMatan Breizman <mbreizma@redhat.com>
Sun, 21 May 2023 08:12:30 +0000 (08:12 +0000)
Signed-off-by: Matan Breizman <mbreizma@redhat.com>
(cherry picked from commit b1632dbf17d0118b6a6c7aff1da15c62cbec6637)

src/crimson/osd/object_context_loader.h

index ce5ad6475381f6c80f8f5cbbd757d1c43f616164..82e2c1e4dd5449eb61a6d2bb9042509dfb8cc111 100644 (file)
@@ -35,17 +35,15 @@ public:
   using with_both_obc_func_t =
     std::function<load_obc_iertr::future<> (ObjectContextRef, ObjectContextRef)>;
 
+  // Use this variant by default
   template<RWState::State State>
   load_obc_iertr::future<> with_obc(hobject_t oid,
                                     with_obc_func_t&& func);
 
-  template<RWState::State State>
-  load_obc_iertr::future<> with_clone_obc(hobject_t oid,
-                                          with_obc_func_t&& func);
-
   // Use this variant in the case where the head object
-  // obc is already locked. Avoid nesting
-  // with_head_obc() as in using with_clone_obc().
+  // obc is already locked and only the clone obc is needed.
+  // Avoid nesting with_head_obc() calls by using with_clone_obc()
+  // with an already locked head.
   template<RWState::State State>
   load_obc_iertr::future<> with_clone_obc_only(ObjectContextRef head,
                                                hobject_t oid,
@@ -59,6 +57,20 @@ public:
     hobject_t oid,
     with_both_obc_func_t&& func);
 
+  load_obc_iertr::future<> reload_obc(ObjectContext& obc) const;
+
+  void notify_on_change(bool is_primary);
+
+private:
+  ObjectContextRegistry& obc_registry;
+  PGBackend& backend;
+  DoutPrefixProvider& dpp;
+  obc_accessing_list_t obc_set_accessing;
+
+  template<RWState::State State>
+  load_obc_iertr::future<> with_clone_obc(hobject_t oid,
+                                          with_obc_func_t&& func);
+
   template<RWState::State State>
   load_obc_iertr::future<> with_head_obc(ObjectContextRef obc,
                                          bool existed,
@@ -71,15 +83,5 @@ public:
 
   load_obc_iertr::future<ObjectContextRef>
   load_obc(ObjectContextRef obc);
-
-  load_obc_iertr::future<> reload_obc(ObjectContext& obc) const;
-
-  void notify_on_change(bool is_primary);
-
-private:
-  ObjectContextRegistry& obc_registry;
-  PGBackend& backend;
-  DoutPrefixProvider& dpp;
-  obc_accessing_list_t obc_set_accessing;
 };
 }