|
 |
50dc83 |
From c429d3c63601e6ea15af76aa684c30bbeb746467 Mon Sep 17 00:00:00 2001
|
|
|
50dc83 |
From: Mohammed Rafi KC <rkavunga@redhat.com>
|
|
|
50dc83 |
Date: Tue, 23 Apr 2019 13:03:28 +0530
|
|
|
50dc83 |
Subject: [PATCH 120/124] glusterd/shd: Keep a ref on volinfo until attach rpc
|
|
|
50dc83 |
execute cbk
|
|
|
50dc83 |
|
|
|
50dc83 |
When svc attach execute for multiplexing a daemon, we have to keep
|
|
|
50dc83 |
a ref on volinfo until it finish the execution. Because, if the attach
|
|
|
50dc83 |
is an aysnc call, then a parallel volume delete can lead to free the
|
|
|
50dc83 |
volinfo
|
|
|
50dc83 |
|
|
|
50dc83 |
> upstream patch : https://review.gluster.org/#/c/glusterfs/+/22606/
|
|
|
50dc83 |
|
|
|
50dc83 |
>Change-Id: Ibc02b89557baaed2f63db63d7fb1a7480444ae0d
|
|
|
50dc83 |
>fixes: bz#1702185
|
|
|
50dc83 |
>Signed-off-by: Mohammed Rafi KC <rkavunga@redhat.com>
|
|
|
50dc83 |
|
|
|
50dc83 |
Change-Id: Ibc02b89557baaed2f63db63d7fb1a7480444ae0d
|
|
|
50dc83 |
BUG: 1702240
|
|
|
50dc83 |
Signed-off-by: Mohammed Rafi KC <rkavunga@redhat.com>
|
|
|
50dc83 |
Reviewed-on: https://code.engineering.redhat.com/gerrit/168616
|
|
|
50dc83 |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
50dc83 |
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
|
|
|
50dc83 |
---
|
|
|
50dc83 |
xlators/mgmt/glusterd/src/glusterd-shd-svc.c | 3 +++
|
|
|
50dc83 |
xlators/mgmt/glusterd/src/glusterd-svc-helper.c | 4 ++++
|
|
|
50dc83 |
2 files changed, 7 insertions(+)
|
|
|
50dc83 |
|
|
|
50dc83 |
diff --git a/xlators/mgmt/glusterd/src/glusterd-shd-svc.c b/xlators/mgmt/glusterd/src/glusterd-shd-svc.c
|
|
|
50dc83 |
index 19eca9f..a9eab42 100644
|
|
|
50dc83 |
--- a/xlators/mgmt/glusterd/src/glusterd-shd-svc.c
|
|
|
50dc83 |
+++ b/xlators/mgmt/glusterd/src/glusterd-shd-svc.c
|
|
|
50dc83 |
@@ -452,8 +452,11 @@ glusterd_shdsvc_start(glusterd_svc_t *svc, int flags)
|
|
|
50dc83 |
}
|
|
|
50dc83 |
|
|
|
50dc83 |
if (shd->attached) {
|
|
|
50dc83 |
+ glusterd_volinfo_ref(volinfo);
|
|
|
50dc83 |
+ /* Unref will happen from glusterd_svc_attach_cbk */
|
|
|
50dc83 |
ret = glusterd_attach_svc(svc, volinfo, flags);
|
|
|
50dc83 |
if (ret) {
|
|
|
50dc83 |
+ glusterd_volinfo_unref(volinfo);
|
|
|
50dc83 |
gf_msg("glusterd", GF_LOG_ERROR, 0, GD_MSG_VOLINFO_GET_FAIL,
|
|
|
50dc83 |
"Failed to attach shd svc(volume=%s) to pid=%d. Starting"
|
|
|
50dc83 |
"a new process",
|
|
|
50dc83 |
diff --git a/xlators/mgmt/glusterd/src/glusterd-svc-helper.c b/xlators/mgmt/glusterd/src/glusterd-svc-helper.c
|
|
|
50dc83 |
index 02945b1..f7be394 100644
|
|
|
50dc83 |
--- a/xlators/mgmt/glusterd/src/glusterd-svc-helper.c
|
|
|
50dc83 |
+++ b/xlators/mgmt/glusterd/src/glusterd-svc-helper.c
|
|
|
50dc83 |
@@ -695,6 +695,10 @@ out:
|
|
|
50dc83 |
if (flag) {
|
|
|
50dc83 |
GF_FREE(flag);
|
|
|
50dc83 |
}
|
|
|
50dc83 |
+
|
|
|
50dc83 |
+ if (volinfo)
|
|
|
50dc83 |
+ glusterd_volinfo_unref(volinfo);
|
|
|
50dc83 |
+
|
|
|
50dc83 |
GF_ATOMIC_DEC(conf->blockers);
|
|
|
50dc83 |
STACK_DESTROY(frame->root);
|
|
|
50dc83 |
return 0;
|
|
|
50dc83 |
--
|
|
|
50dc83 |
1.8.3.1
|
|
|
50dc83 |
|