diff options
| author | Kaushal M <kaushal@redhat.com> | 2014-06-16 18:56:18 +0530 |
|---|---|---|
| committer | Krishnan Parthasarathi <kparthas@redhat.com> | 2014-06-16 19:36:18 -0700 |
| commit | 84d370774cdbc6847f4f2f64a7f47abb27a7471b (patch) | |
| tree | c7c7e7a7f77c2ef03aedf47f03163bcc01462666 /xlators/mgmt/glusterd/src/glusterd-utils.h | |
| parent | d3bd3319afb208acfa905696fb0be5545f5883f6 (diff) | |
glusterd: Use blocking quotad start only on quota enable
Having quotad always being using the blocking runner variant is
problematic. In some cases where quotad was started from the epoll
thread, it lead to a deadlock which lead to glusterd becoming
unresponsive.
This patch makes the default quotad_start function use the non-blocking
runner_nowait variant. The blocking start is used only when quotad is
started on quota enable command.
Change-Id: Ib27042748d69ea28e68badcfaddf61589aae4eba
BUG: 1109872
Signed-off-by: Kaushal M <kaushal@redhat.com>
Reviewed-on: http://review.gluster.org/8082
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Krishnan Parthasarathi <kparthas@redhat.com>
Tested-by: Krishnan Parthasarathi <kparthas@redhat.com>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-utils.h')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-utils.h | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-utils.h b/xlators/mgmt/glusterd/src/glusterd-utils.h index aae91cdff8d..320bc20cdd2 100644 --- a/xlators/mgmt/glusterd/src/glusterd-utils.h +++ b/xlators/mgmt/glusterd/src/glusterd-utils.h @@ -223,6 +223,9 @@ int32_t glusterd_quotad_start (); int32_t +glusterd_quotad_start_wait (); + +int32_t glusterd_quotad_stop (); void @@ -268,6 +271,9 @@ int glusterd_check_generate_start_quotad (void); int +glusterd_check_generate_start_quotad_wait (void); + +int glusterd_nodesvcs_handle_graph_change (glusterd_volinfo_t *volinfo); int |