librbd/plugin: delete field m_image_ctx

librbd::plugin::ParentCache may be shared by more than one images.

Signed-off-by: Li, Xiaoyan <xiaoyan.li@intel.com>

diff --git a/src/librbd/plugin/ParentCache.cc b/src/librbd/plugin/ParentCache.cc
index 0c50c8e7197055d6cd6f91a767ad6e91d818a868..cdbe8ff9b3cda11e16db42afa24aa0b0a94c03cf 100644 (file)
--- a/src/librbd/plugin/ParentCache.cc
+++ b/src/librbd/plugin/ParentCache.cc
@@ -35,20 +35,19 @@ namespace plugin {
 template <typename I>
 void ParentCache<I>::init(I* image_ctx, Api<I>& api, HookPoints* hook_points,
                           Context* on_finish) {
-  m_image_ctx = image_ctx;
-  bool parent_cache_enabled = m_image_ctx->config.template get_val<bool>(
+  bool parent_cache_enabled = image_ctx->config.template get_val<bool>(
     "rbd_parent_cache_enabled");
-  if (m_image_ctx->child == nullptr || !parent_cache_enabled ||
-      !m_image_ctx->data_ctx.is_valid()) {
+  if (image_ctx->child == nullptr || !parent_cache_enabled ||
+      !image_ctx->data_ctx.is_valid()) {
     on_finish->complete(0);
     return;
   }
 
-  auto cct = m_image_ctx->cct;
+  auto cct = image_ctx->cct;
   ldout(cct, 5) << dendl;
 
   auto parent_cache = cache::ParentCacheObjectDispatch<I>::create(
-    m_image_ctx, api);
+    image_ctx, api);
   on_finish = new LambdaContext([this, on_finish, parent_cache](int r) {
       if (r < 0) {
         // the object dispatcher will handle cleanup if successfully initialized
@@ -62,7 +61,6 @@ void ParentCache<I>::init(I* image_ctx, Api<I>& api, HookPoints* hook_points,
 
 template <typename I>
 void ParentCache<I>::handle_init_parent_cache(int r, Context* on_finish) {
-  auto cct = m_image_ctx->cct;
   ldout(cct, 5) << "r=" << r << dendl;
 
   if (r < 0) {

diff --git a/src/librbd/plugin/ParentCache.h b/src/librbd/plugin/ParentCache.h
index e456c5ac3d806de17155063fb1dbd31829118a19..276170b258cbc23cf2ba048d74019a3ad4bfe9ae 100644 (file)
--- a/src/librbd/plugin/ParentCache.h
+++ b/src/librbd/plugin/ParentCache.h
@@ -23,9 +23,8 @@ public:
             Context* on_finish) override;
 
 private:
-  ImageCtxT* m_image_ctx = nullptr;
-
   void handle_init_parent_cache(int r, Context* on_finish);
+  using ceph::Plugin::cct;
 
 };