diff options
author | Ashish Pandey <aspandey@redhat.com> | 2015-10-12 13:14:08 +0530 |
---|---|---|
committer | Pranith Kumar Karampuri <pkarampu@redhat.com> | 2015-11-16 00:19:58 -0800 |
commit | 92e3bbbad803688a4dbcbab6bcd35867aa055da1 (patch) | |
tree | 17e6a1bcc997f58a0d8e360119d595a423f7ed63 | |
parent | d4aaa00d77d7a10748a1ca9af82265ad9ac42598 (diff) |
cluster/afr : Remove stale indices
Change-Id: Iba23338a452b49dc9fe6ae7b4ca108ebc377fe42
BUG: 1270668
Signed-off-by: Ashish Pandey <aspandey@redhat.com>
Reviewed-on: http://review.gluster.org/12336
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Pranith Kumar Karampuri <pkarampu@redhat.com>
-rw-r--r-- | xlators/cluster/afr/src/afr-self-heal-common.c | 23 |
1 files changed, 22 insertions, 1 deletions
diff --git a/xlators/cluster/afr/src/afr-self-heal-common.c b/xlators/cluster/afr/src/afr-self-heal-common.c index a7633c92094..cd2dc10be27 100644 --- a/xlators/cluster/afr/src/afr-self-heal-common.c +++ b/xlators/cluster/afr/src/afr-self-heal-common.c @@ -54,6 +54,27 @@ afr_selfheal_post_op (call_frame_t *frame, xlator_t *this, inode_t *inode, return 0; } +int +afr_check_stale_error (struct afr_reply *replies, afr_private_t *priv) +{ + int i = 0; + int op_errno = 0; + int tmp_errno = 0; + int stale_count = 0; + + for (i = 0; i < priv->child_count; i++) { + tmp_errno = replies[i].op_errno; + if (tmp_errno == ENOENT || tmp_errno == ESTALE) { + op_errno = afr_higher_errno (op_errno, tmp_errno); + stale_count++; + } + } + if (stale_count != priv->child_count) + return -ENOTCONN; + else + return -op_errno; +} + dict_t * afr_selfheal_output_xattr (xlator_t *this, afr_transaction_type type, @@ -1196,7 +1217,7 @@ afr_selfheal_unlocked_inspect (call_frame_t *frame, xlator_t *this, goto out; } } else if (valid_cnt < 2) { - ret = -ENOTCONN; + ret = afr_check_stale_error (replies, priv); goto out; } |