diff options
| author | Anand Avati <avati@gluster.com> | 2011-06-09 02:32:22 +0000 |
|---|---|---|
| committer | Anand Avati <avati@gluster.com> | 2011-06-09 01:35:30 -0700 |
| commit | 1a06583bc6ad298c97bdbafbcc0246bddc0ce301 (patch) | |
| tree | fd90d15bb1babea3138f874875ce4e84f9958842 | |
| parent | e8e2079d82d08daa600086895de3c2633b909188 (diff) | |
cluster/dht: use GFID returned from hashed subvolume
.. to perform lookups on remaining subvolumes. This way, if there is a
race between two clients to 'fix' GFIDs with gfid-req, then the hashed
subvolume will arbitrate and return the winner in stbuf->ia_gfid. This
patch uses the returned gfid as the new gfid-req thereby preventing
mismatching GFIDs on other servers due to further races.
Signed-off-by: Anand Avati <avati@gluster.com>
BUG: 2522 ([glusterfs-3.1.3qa8]: rm -rf shows invalid argument)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2522
| -rw-r--r-- | xlators/cluster/dht/src/dht-common.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/xlators/cluster/dht/src/dht-common.c b/xlators/cluster/dht/src/dht-common.c index ac713384abe..5583bd5592d 100644 --- a/xlators/cluster/dht/src/dht-common.c +++ b/xlators/cluster/dht/src/dht-common.c @@ -699,6 +699,7 @@ dht_lookup_directory (call_frame_t *frame, xlator_t *this, loc_t *loc) int i = 0; dht_conf_t *conf = NULL; dht_local_t *local = NULL; + int ret = 0; conf = this->private; local = frame->local; @@ -716,6 +717,10 @@ dht_lookup_directory (call_frame_t *frame, xlator_t *this, loc_t *loc) goto unwind; } + if (!uuid_is_null (local->gfid)) + ret = dict_set_static_bin (local->xattr_req, "gfid-req", + local->gfid, 16); + for (i = 0; i < call_cnt; i++) { STACK_WIND (frame, dht_lookup_dir_cbk, conf->subvolumes[i], |