diff options
author | Pranith Kumar K <pkarampu@redhat.com> | 2014-09-14 16:51:25 +0530 |
---|---|---|
committer | Vijay Bellur <vbellur@redhat.com> | 2014-09-15 23:02:27 -0700 |
commit | 3719996a2d31d492d8a3b3f7defe57c2108c3f45 (patch) | |
tree | a0aa972affd369f3e42fe91288b19a0588f3e13c /xlators/cluster/afr/src/afr.h | |
parent | cc332a8f1766562b407dcc6b9cd88a7f5adee216 (diff) |
cluster/afr: Handle EAGAIN properly in inodelk
Problem:
When one of the brick is taken down and brough back up in a replica pair, locks
on that brick will be allowed. Afr returns inodelk success even when one of the
bricks already has the lock taken.
Fix:
If any brick returns EAGAIN return failure to parent xlator.
BUG: 1142020
Change-Id: Iee3f5990be75e10f8accec9bc3856e3f76d1593c
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
Reviewed-on: http://review.gluster.org/8744
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'xlators/cluster/afr/src/afr.h')
-rw-r--r-- | xlators/cluster/afr/src/afr.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/xlators/cluster/afr/src/afr.h b/xlators/cluster/afr/src/afr.h index 35d4d545bc9..ddcb2c0e177 100644 --- a/xlators/cluster/afr/src/afr.h +++ b/xlators/cluster/afr/src/afr.h @@ -604,6 +604,11 @@ typedef struct _afr_local { struct iatt postbuf; } zerofill; + struct { + const char *volume; + int32_t cmd; + struct gf_flock flock; + } inodelk; } cont; |