diff options
author | Pranith Kumar K <pkarampu@redhat.com> | 2015-06-04 09:52:51 +0530 |
---|---|---|
committer | Vijay Bellur <vbellur@redhat.com> | 2015-06-06 22:35:43 -0700 |
commit | 3373379303afa575c0616482c8ab8c3c4a08cc22 (patch) | |
tree | c6bf853d8a0520d63223df28e07edab270cb0a96 /tools | |
parent | 8f12fae9cb9349af2e3eeb338b5d85112655dc29 (diff) |
cluster/ec: Don't handle EC_XATTR_DIRTY in response
Problem:
ec_update_size_version expects all the keys it did xattrop with to come in
response so that it can set the values again in ec_update_size_version_done.
But EC_XATTR_DIRTY is not combined so the value won't be present in the
response. So ctx->post/pre_dirty are not updated in
ec_update_size_version_done. So these values are still non-zero. When
ec_unlock_now is called as part of flush's unlock phase it again tries to
perform same xattrop for EC_XATTR_DIRTY. But ec_update_size_version is not
expected to be called in unlock phase of flush because ec_flush_size_version
should have reset everything to zero and unlock is never invoked from
ec_update_size_version_done for flush/fsync/fsyncdir. This leads to stale lock
which leads to hang.
Fix:
EC_XATTR_DIRTY is removed in ex_xattrop_cbk and is never combined with other
answers. So remove handling of this in the response.
Change-Id: If0ea3efec3235a6e312465d8838585fbe752c7ea
BUG: 1227654
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
Reviewed-on: http://review.gluster.org/11078
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'tools')
0 files changed, 0 insertions, 0 deletions