diff options
author | Kinglong Mee <kinglongmee@gmail.com> | 2018-07-16 16:39:02 +0800 |
---|---|---|
committer | N Balachandran <nbalacha@redhat.com> | 2018-07-18 08:16:01 +0000 |
commit | c63979fdd8589483f242a8f450a130d269e907ed (patch) | |
tree | 0da2cbadecc795851795bb50ffcf46ebb01ef02d /xlators/cluster/dht/src/dht-rename.c | |
parent | 6857d80a50823f5bd1be495edf6899d9bcba9f20 (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.c | 34 |
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; |