summaryrefslogtreecommitdiffstats
path: root/xlators/cluster/dht/src/dht-rename.c
diff options
context:
space:
mode:
authorKinglong Mee <kinglongmee@gmail.com>2018-07-16 16:39:02 +0800
committerN Balachandran <nbalacha@redhat.com>2018-07-18 08:16:01 +0000
commitc63979fdd8589483f242a8f450a130d269e907ed (patch)
tree0da2cbadecc795851795bb50ffcf46ebb01ef02d /xlators/cluster/dht/src/dht-rename.c
parent6857d80a50823f5bd1be495edf6899d9bcba9f20 (diff)
dht: remove useless argument from dht_iatt_merge
The last using of the subvol argument has been removed at 4e1ec35ef4f7 ("core: fill 'ia_ino' from 'ia_gfid' in 'storage/posix' ......") 7 years ago (2011-06-16). Change-Id: I9788d79e2e40cc153cf2960e28c7c1c1033dc8f7 fixes: bz#1601683 Signed-off-by: Kinglong Mee <mijinlong@open-fs.com>
Diffstat (limited to 'xlators/cluster/dht/src/dht-rename.c')
-rw-r--r--xlators/cluster/dht/src/dht-rename.c34
1 files changed, 16 insertions, 18 deletions
diff --git a/xlators/cluster/dht/src/dht-rename.c b/xlators/cluster/dht/src/dht-rename.c
index 1d0c2bbc7ed..62b72b7dc29 100644
--- a/xlators/cluster/dht/src/dht-rename.c
+++ b/xlators/cluster/dht/src/dht-rename.c
@@ -153,11 +153,11 @@ dht_rename_dir_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
* FIXME: is this the correct way to build stbuf and
* parent bufs?
*/
- dht_iatt_merge (this, &local->stbuf, stbuf, prev);
- dht_iatt_merge (this, &local->preoldparent, preoldparent, prev);
- dht_iatt_merge (this, &local->postoldparent, postoldparent, prev);
- dht_iatt_merge (this, &local->preparent, prenewparent, prev);
- dht_iatt_merge (this, &local->postparent, postnewparent, prev);
+ dht_iatt_merge (this, &local->stbuf, stbuf);
+ dht_iatt_merge (this, &local->preoldparent, preoldparent);
+ dht_iatt_merge (this, &local->postoldparent, postoldparent);
+ dht_iatt_merge (this, &local->preparent, prenewparent);
+ dht_iatt_merge (this, &local->postparent, postnewparent);
unwind:
this_call_cnt = dht_frame_return (frame);
@@ -248,11 +248,11 @@ dht_rename_hashed_dir_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
* FIXME: is this the correct way to build stbuf and
* parent bufs?
*/
- dht_iatt_merge (this, &local->stbuf, stbuf, prev);
- dht_iatt_merge (this, &local->preoldparent, preoldparent, prev);
- dht_iatt_merge (this, &local->postoldparent, postoldparent, prev);
- dht_iatt_merge (this, &local->preparent, prenewparent, prev);
- dht_iatt_merge (this, &local->postparent, postnewparent, prev);
+ dht_iatt_merge (this, &local->stbuf, stbuf);
+ dht_iatt_merge (this, &local->preoldparent, preoldparent);
+ dht_iatt_merge (this, &local->postoldparent, postoldparent);
+ dht_iatt_merge (this, &local->preparent, prenewparent);
+ dht_iatt_merge (this, &local->postparent, postnewparent);
call_cnt = local->call_cnt = conf->subvolume_cnt - 1;
@@ -1071,13 +1071,11 @@ err:
/* Merge attrs only from src_cached. In case there of src_cached !=
* dst_hashed, this ignores linkfile attrs. */
if (prev == src_cached) {
- dht_iatt_merge (this, &local->stbuf, stbuf, prev);
- dht_iatt_merge (this, &local->preoldparent, preoldparent,
- prev);
- dht_iatt_merge (this, &local->postoldparent, postoldparent,
- prev);
- dht_iatt_merge (this, &local->preparent, prenewparent, prev);
- dht_iatt_merge (this, &local->postparent, postnewparent, prev);
+ dht_iatt_merge (this, &local->stbuf, stbuf);
+ dht_iatt_merge (this, &local->preoldparent, preoldparent);
+ dht_iatt_merge (this, &local->postoldparent, postoldparent);
+ dht_iatt_merge (this, &local->preparent, prenewparent);
+ dht_iatt_merge (this, &local->postparent, postnewparent);
}
@@ -1245,7 +1243,7 @@ dht_rename_link_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
local->op_errno = op_errno;
local->added_link = _gf_false;
} else
- dht_iatt_merge (this, &local->stbuf, stbuf, prev);
+ dht_iatt_merge (this, &local->stbuf, stbuf);
if (local->op_ret == -1)
goto cleanup;