summaryrefslogtreecommitdiffstats
path: root/fs/cifs/inode.c
diff options
context:
space:
mode:
authorSteve French <sfrench@us.ibm.com>2011-05-20 17:00:01 +0000
committerSteve French <sfrench@us.ibm.com>2011-05-20 17:00:01 +0000
commit156ecb2d8b06589098f6ce3012e6a10fef07c416 (patch)
tree4008ea00d8c49ff2361ebe784a1e757c8cad9986 /fs/cifs/inode.c
parentceec1e0fae6eecac876ec28cfb97786f87dfb1c6 (diff)
downloadop-kernel-dev-156ecb2d8b06589098f6ce3012e6a10fef07c416.zip
op-kernel-dev-156ecb2d8b06589098f6ce3012e6a10fef07c416.tar.gz
[CIFS] Fix to problem with getattr caused by invalidate simplification patch
Fix to earlier "Simplify invalidate part (try #6)" patch That patch caused problems with connectathon test 5. Reviewed-by: Jeff Layton <jlayton@samba.org> Signed-off-by: Pavel Shilovsky <piastry@etersoft.ru> Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs/cifs/inode.c')
-rw-r--r--fs/cifs/inode.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/fs/cifs/inode.c b/fs/cifs/inode.c
index 0cc7edd..de02ed5 100644
--- a/fs/cifs/inode.c
+++ b/fs/cifs/inode.c
@@ -1805,8 +1805,10 @@ int cifs_getattr(struct vfsmount *mnt, struct dentry *dentry,
if (!CIFS_I(inode)->clientCanCacheRead && inode->i_mapping &&
inode->i_mapping->nrpages != 0) {
rc = filemap_fdatawait(inode->i_mapping);
- mapping_set_error(inode->i_mapping, rc);
- return rc;
+ if (rc) {
+ mapping_set_error(inode->i_mapping, rc);
+ return rc;
+ }
}
rc = cifs_revalidate_dentry_attr(dentry);
OpenPOWER on IntegriCloud