diff options
author | Ravishankar N <ravishankar@redhat.com> | 2018-12-11 15:32:03 +0530 |
---|---|---|
committer | Amar Tumballi <amarts@redhat.com> | 2018-12-12 23:02:56 +0000 |
commit | 52d3f82db2f032eae1b60ffa2f283109858ce3df (patch) | |
tree | 2fa19f5f3838a8a3343a7a4b928007016960328d /xlators/cluster/afr/src/afr-self-heald.c | |
parent | 607bbd935f102602810fdeb61079b85ecb58a118 (diff) |
afr: some minor itable related cleanups
- this->itable always needs to be allocated, hence move it outside
afr_selfheal_daemon_init().
- Invoke afr_selfheal_daemon_init() only for self-heal daemon case.
- remove redundant itable allocation in afr_discover().
- destroy itable in fini.
Updates bz#1193929
Change-Id: Ib28b50b607386f5a5aa7d2f743c8b506ccb10eae
Signed-off-by: Ravishankar N <ravishankar@redhat.com>
Diffstat (limited to 'xlators/cluster/afr/src/afr-self-heald.c')
-rw-r--r-- | xlators/cluster/afr/src/afr-self-heald.c | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/xlators/cluster/afr/src/afr-self-heald.c b/xlators/cluster/afr/src/afr-self-heald.c index 8d6d383e872..c6966147777 100644 --- a/xlators/cluster/afr/src/afr-self-heald.c +++ b/xlators/cluster/afr/src/afr-self-heald.c @@ -16,7 +16,6 @@ #include "afr-messages.h" #include <glusterfs/byte-order.h> -#define SHD_INODE_LRU_LIMIT 2048 #define AFR_EH_SPLIT_BRAIN_LIMIT 1024 #define AFR_STATISTICS_HISTORY_SIZE 50 @@ -722,9 +721,9 @@ afr_shd_ta_unset_xattrs(xlator_t *this, loc_t *loc, dict_t **xdata, int healer) } ret = dict_set_bin(xattr, priv->pending_key[i], raw, - AFR_NUM_CHANGE_LOGS * sizeof (int)); + AFR_NUM_CHANGE_LOGS * sizeof(int)); if (ret) { - GF_FREE (raw); + GF_FREE(raw); goto out; } @@ -1210,10 +1209,6 @@ afr_selfheal_daemon_init(xlator_t *this) priv = this->private; shd = &priv->shd; - this->itable = inode_table_new(SHD_INODE_LRU_LIMIT, this); - if (!this->itable) - goto out; - shd->index_healers = GF_CALLOC(sizeof(*shd->index_healers), priv->child_count, gf_afr_mt_subvol_healer_t); |