diff options
author | Pranith Kumar K <pkarampu@redhat.com> | 2014-08-13 11:11:17 +0530 |
---|---|---|
committer | Pranith Kumar Karampuri <pkarampu@redhat.com> | 2014-08-22 02:31:57 -0700 |
commit | 3b70b160a46b22b77a8ad1897440ec1346795a0f (patch) | |
tree | 5aa5047d58491c5fd6f97e5eab9e59df359608a9 /xlators/cluster/afr/src/afr-self-heal.h | |
parent | 60f12dfbc87818831a65ac80ad8ba2fe166a29e2 (diff) |
cluster/afr: Perform gfid heal inside locks.
Problem:
Allowing lookup with 'gfid-req' will lead to assigning gfid at posix layer.
When two mounts perform lookup in parallel that can lead to both bricks getting
different gfids leading to gfid-mismatch/EIO for the lookup.
Fix:
Perform gfid heal inside lock.
BUG: 1129529
Change-Id: I20c6c5e25ee27eeb906bff2f4c8ad0da18d00090
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
Reviewed-on: http://review.gluster.org/8512
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Krutika Dhananjay <kdhananj@redhat.com>
Diffstat (limited to 'xlators/cluster/afr/src/afr-self-heal.h')
-rw-r--r-- | xlators/cluster/afr/src/afr-self-heal.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/xlators/cluster/afr/src/afr-self-heal.h b/xlators/cluster/afr/src/afr-self-heal.h index a1b972ac35d..8556b2bb335 100644 --- a/xlators/cluster/afr/src/afr-self-heal.h +++ b/xlators/cluster/afr/src/afr-self-heal.h @@ -85,7 +85,8 @@ int afr_selfheal (xlator_t *this, uuid_t gfid); int -afr_selfheal_name (xlator_t *this, uuid_t gfid, const char *name); +afr_selfheal_name (xlator_t *this, uuid_t gfid, const char *name, + void *gfid_req); int afr_selfheal_data (call_frame_t *frame, xlator_t *this, inode_t *inode); |