diff options
author | Venky Shankar <vshankar@redhat.com> | 2015-04-26 16:25:08 +0530 |
---|---|---|
committer | Vijay Bellur <vbellur@redhat.com> | 2015-04-30 03:10:26 -0700 |
commit | 0c5910a39b8daad2b131c1f7f48b0bf3b8f46030 (patch) | |
tree | ddb8abe84db48bda43d5306a65c7ee0edfdb1f85 /xlators | |
parent | cb8ce71560a496db5392d322b7cc663054a8457a (diff) |
features/bitrot: Use global timer wheel
Change-Id: I761927ea263b4144b851881f25791fda5b794f59
BUG: 1170075
Signed-off-by: Venky Shankar <vshankar@redhat.com>
Reviewed-on: http://review.gluster.org/10381
Tested-by: NetBSD Build System
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Raghavendra Bhat <raghavendra@redhat.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'xlators')
-rw-r--r-- | xlators/features/bit-rot/src/bitd/bit-rot.c | 31 | ||||
-rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-bitd-svc.c | 18 |
2 files changed, 25 insertions, 24 deletions
diff --git a/xlators/features/bit-rot/src/bitd/bit-rot.c b/xlators/features/bit-rot/src/bitd/bit-rot.c index 1202ca260eb..a5c5ab76964 100644 --- a/xlators/features/bit-rot/src/bitd/bit-rot.c +++ b/xlators/features/bit-rot/src/bitd/bit-rot.c @@ -25,6 +25,8 @@ #include "bit-rot-scrub.h" #include <pthread.h> +#include "tw.h" + static int br_find_child_index (xlator_t *this, xlator_t *child) { @@ -604,6 +606,8 @@ br_add_object_to_queue (struct gf_tw_timer_list *timer, this = object->this; priv = this->private; + THIS = this; + pthread_mutex_lock (&priv->lock); { list_add_tail (&object->list, &priv->obj_queue->objects); @@ -1292,23 +1296,17 @@ static inline void br_fini_signer (xlator_t *this, br_private_t *priv) { int i = 0; - int ret = 0; for (; i < BR_WORKERS; i++) { (void) gf_thread_cleanup_xint (priv->obj_queue->workers[i]); } pthread_cond_destroy (&priv->object_cond); - ret = gf_tw_cleanup_timers (priv->timer_wheel); - if (ret == 0) { - priv->timer_wheel = NULL; - } } static inline int32_t br_init_signer (xlator_t *this, br_private_t *priv) { - int rc = 0; int i = 0; int32_t ret = -1; @@ -1317,10 +1315,10 @@ br_init_signer (xlator_t *this, br_private_t *priv) if (ret) goto out; - priv->timer_wheel = gf_tw_init_timers (); + priv->timer_wheel = glusterfs_global_timer_wheel (this); if (!priv->timer_wheel) { gf_log (this->name, GF_LOG_ERROR, - "failed to initialize the timer wheel"); + "global timer wheel unavailable"); goto out; } @@ -1329,7 +1327,7 @@ br_init_signer (xlator_t *this, br_private_t *priv) priv->obj_queue = GF_CALLOC (1, sizeof (*priv->obj_queue), gf_br_mt_br_ob_n_wk_t); if (!priv->obj_queue) - goto cleanup_timer; + goto cleanup_cond; INIT_LIST_HEAD (&priv->obj_queue->objects); for (i = 0; i < BR_WORKERS; i++) { @@ -1352,14 +1350,9 @@ br_init_signer (xlator_t *this, br_private_t *priv) GF_FREE (priv->obj_queue); - cleanup_timer: + cleanup_cond: /* that's explicit */ pthread_cond_destroy (&priv->object_cond); - rc = gf_tw_cleanup_timers (priv->timer_wheel); - if (rc == 0) { - priv->timer_wheel = NULL; - } - out: return -1; } @@ -1460,7 +1453,6 @@ init (xlator_t *this) void fini (xlator_t *this) { - int ret = 0; br_private_t *priv = this->private; if (!priv) @@ -1469,13 +1461,6 @@ fini (xlator_t *this) if (!priv->iamscrubber) br_fini_signer (this, priv); br_free_children (this); - if (priv->timer_wheel) { - ret = gf_tw_cleanup_timers (priv->timer_wheel); - - if (ret == 0) { - priv->timer_wheel = NULL; - } - } this->private = NULL; GF_FREE (priv); diff --git a/xlators/mgmt/glusterd/src/glusterd-bitd-svc.c b/xlators/mgmt/glusterd/src/glusterd-bitd-svc.c index 8bd6d8a7276..002210a90cf 100644 --- a/xlators/mgmt/glusterd/src/glusterd-bitd-svc.c +++ b/xlators/mgmt/glusterd/src/glusterd-bitd-svc.c @@ -106,7 +106,23 @@ out: int glusterd_bitdsvc_start (glusterd_svc_t *svc, int flags) { - return glusterd_svc_start (svc, flags, NULL); + int ret = -1; + dict_t *cmdict = NULL; + + cmdict = dict_new (); + if (!cmdict) + goto error_return; + + ret = dict_set_str (cmdict, "cmdarg0", "--global-timer-wheel"); + if (ret) + goto dealloc_dict; + + ret = glusterd_svc_start (svc, flags, cmdict); + + dealloc_dict: + dict_unref (cmdict); + error_return: + return ret; } int |