e3c68b
From bc5e3967864d6f6ea22deb22ba72aedca8367797 Mon Sep 17 00:00:00 2001
e3c68b
From: Mohammed Rafi KC <rkavunga@redhat.com>
e3c68b
Date: Thu, 11 Jul 2019 12:49:41 +0530
e3c68b
Subject: [PATCH 246/255] Revert "glusterd/shd: Keep a ref on volinfo until
e3c68b
 attach rpc execute cbk"
e3c68b
e3c68b
This reverts commit c429d3c63601e6ea15af76aa684c30bbeb746467.
e3c68b
e3c68b
BUG: 1471742
e3c68b
Change-Id: I614e8bdbcc5111dbf407aba047e7d2284bef8ac8
e3c68b
Signed-off-by: Mohammed Rafi KC <rkavunga@redhat.com>
e3c68b
Reviewed-on: https://code.engineering.redhat.com/gerrit/175955
e3c68b
Tested-by: RHGS Build Bot <nigelb@redhat.com>
e3c68b
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
e3c68b
---
e3c68b
 xlators/mgmt/glusterd/src/glusterd-shd-svc.c    | 3 ---
e3c68b
 xlators/mgmt/glusterd/src/glusterd-svc-helper.c | 4 ----
e3c68b
 2 files changed, 7 deletions(-)
e3c68b
e3c68b
diff --git a/xlators/mgmt/glusterd/src/glusterd-shd-svc.c b/xlators/mgmt/glusterd/src/glusterd-shd-svc.c
e3c68b
index a9eab42..19eca9f 100644
e3c68b
--- a/xlators/mgmt/glusterd/src/glusterd-shd-svc.c
e3c68b
+++ b/xlators/mgmt/glusterd/src/glusterd-shd-svc.c
e3c68b
@@ -452,11 +452,8 @@ glusterd_shdsvc_start(glusterd_svc_t *svc, int flags)
e3c68b
     }
e3c68b
 
e3c68b
     if (shd->attached) {
e3c68b
-        glusterd_volinfo_ref(volinfo);
e3c68b
-        /* Unref will happen from glusterd_svc_attach_cbk */
e3c68b
         ret = glusterd_attach_svc(svc, volinfo, flags);
e3c68b
         if (ret) {
e3c68b
-            glusterd_volinfo_unref(volinfo);
e3c68b
             gf_msg("glusterd", GF_LOG_ERROR, 0, GD_MSG_VOLINFO_GET_FAIL,
e3c68b
                    "Failed to attach shd svc(volume=%s) to pid=%d. Starting"
e3c68b
                    "a new process",
e3c68b
diff --git a/xlators/mgmt/glusterd/src/glusterd-svc-helper.c b/xlators/mgmt/glusterd/src/glusterd-svc-helper.c
e3c68b
index f7be394..02945b1 100644
e3c68b
--- a/xlators/mgmt/glusterd/src/glusterd-svc-helper.c
e3c68b
+++ b/xlators/mgmt/glusterd/src/glusterd-svc-helper.c
e3c68b
@@ -695,10 +695,6 @@ out:
e3c68b
     if (flag) {
e3c68b
         GF_FREE(flag);
e3c68b
     }
e3c68b
-
e3c68b
-    if (volinfo)
e3c68b
-        glusterd_volinfo_unref(volinfo);
e3c68b
-
e3c68b
     GF_ATOMIC_DEC(conf->blockers);
e3c68b
     STACK_DESTROY(frame->root);
e3c68b
     return 0;
e3c68b
-- 
e3c68b
1.8.3.1
e3c68b