diff options
author | Soumya Koduri <skoduri@redhat.com> | 2018-11-28 14:14:00 +0530 |
---|---|---|
committer | Raghavendra G <rgowdapp@redhat.com> | 2018-12-04 05:23:03 +0000 |
commit | b3d88a0904131f6851f4185e43f815ecc3353ab5 (patch) | |
tree | d68fc655153a13fc3addfdee01e357a3e89ce52c /xlators/performance/io-cache/src/io-cache.c | |
parent | 9fc6cf898bcb7dc0f3f671e734678616722d0721 (diff) |
io-cache: xdata needs to be passed for readv operations
io-cache xlator has been skipping xdata references when the
date needs to be read into page cache. This patch fixes the same.
Note: similar changes may be needed for other fops as well
which are handled by io-cache.
Change-Id: I28d73d4ba471d13eb55d0fd0b5197d222df77a2a
updates: bz#1648768
Signed-off-by: Soumya Koduri <skoduri@redhat.com>
Diffstat (limited to 'xlators/performance/io-cache/src/io-cache.c')
-rw-r--r-- | xlators/performance/io-cache/src/io-cache.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/xlators/performance/io-cache/src/io-cache.c b/xlators/performance/io-cache/src/io-cache.c index 6705128087e..31e3bb0df96 100644 --- a/xlators/performance/io-cache/src/io-cache.c +++ b/xlators/performance/io-cache/src/io-cache.c @@ -413,6 +413,7 @@ ioc_cache_validate_cbk(call_frame_t *frame, void *cookie, xlator_t *this, * fd_ref on fd, safe to unref validate frame's private copy */ fd_unref(local->fd); + dict_unref(local->xattr_req); STACK_DESTROY(frame->root); @@ -495,10 +496,13 @@ ioc_cache_validate(call_frame_t *frame, ioc_inode_t *ioc_inode, fd_t *fd, validate_local->fd = fd_ref(fd); validate_local->inode = ioc_inode; + if (local && local->xattr_req) + validate_local->xattr_req = dict_ref(local->xattr_req); validate_frame->local = validate_local; STACK_WIND(validate_frame, ioc_cache_validate_cbk, FIRST_CHILD(frame->this), - FIRST_CHILD(frame->this)->fops->fstat, fd, NULL); + FIRST_CHILD(frame->this)->fops->fstat, fd, + validate_local->xattr_req); out: return ret; @@ -1141,6 +1145,7 @@ ioc_readv(call_frame_t *frame, xlator_t *this, fd_t *fd, size_t size, local->offset = offset; local->size = size; local->inode = ioc_inode; + local->xattr_req = dict_ref(xdata); gf_msg_trace(this->name, 0, "NEW REQ (%p) offset " |