diff options
author | Susant Palai <spalai@redhat.com> | 2018-04-11 23:14:02 +0530 |
---|---|---|
committer | Raghavendra G <rgowdapp@redhat.com> | 2018-04-18 15:04:09 +0000 |
commit | 87bcdd9465b140e0b9d33dadf3384e28b7b6ed9f (patch) | |
tree | f4ed87c44555cb1839aa69537dcea1fad89044b1 /xlators/cluster/dht/src | |
parent | d9cf6d25a4719c4f6fb4d88325f08e49fca18e6b (diff) |
fuse: do fd_resolve in fuse_getattr if fd is received
problem: With the current code, post graph switch the old fd is received for
fuse_getattr and since it is associated with old inode, it does not
have the inode ctx across xlators in new graph. Hence, dht
errored out saying "no layout" for fstat call. Hence the EINVAL.
Solution: if fd is passed, init and resolve fd to carry on getattr
test case:
- Created a single brick distributed volume
- Started untar
- Added a new-brick
Without this fix, untar used to abort with ERROR.
Change-Id: I5805c463fb9a04ba5c24829b768127097ff8b9f9
fixes: bz#1566207
Signed-off-by: Susant Palai <spalai@redhat.com>
Diffstat (limited to 'xlators/cluster/dht/src')
-rw-r--r-- | xlators/cluster/dht/src/dht-inode-read.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xlators/cluster/dht/src/dht-inode-read.c b/xlators/cluster/dht/src/dht-inode-read.c index fa63fefb903..d1895eb2abb 100644 --- a/xlators/cluster/dht/src/dht-inode-read.c +++ b/xlators/cluster/dht/src/dht-inode-read.c @@ -400,8 +400,8 @@ dht_fstat (call_frame_t *frame, xlator_t *this, fd_t *fd, dict_t *xdata) layout = local->layout; if (!layout) { - gf_msg_debug (this->name, 0, - "no layout for fd=%p", fd); + gf_msg (this->name, GF_LOG_ERROR, 0, 0, + "no layout for fd=%p", fd); op_errno = EINVAL; goto err; } |