diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-04-30 10:36:21 -0400 |
---|---|---|
committer | Steve French <sfrench@us.ibm.com> | 2012-05-09 16:16:22 -0400 |
commit | 48a5730e5b71201e226ff06e245bf308feba5f10 (patch) | |
tree | 2018cd2924ed13f736032beec587858c5e688ce1 /fs/cifs/cifsfs.c | |
parent | 7ee94d97aafacf5a019b3578e0eae6daa2e2bcd5 (diff) |
cifs: fix revalidation test in cifs_llseek()
This test is always true so it means we revalidate the length every
time, which generates more network traffic. When it is SEEK_SET or
SEEK_CUR, then we don't need to revalidate.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs/cifs/cifsfs.c')
-rw-r--r-- | fs/cifs/cifsfs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/cifs/cifsfs.c b/fs/cifs/cifsfs.c index ca6a3796a33b..541ef81f6ae8 100644 --- a/fs/cifs/cifsfs.c +++ b/fs/cifs/cifsfs.c | |||
@@ -699,7 +699,7 @@ static loff_t cifs_llseek(struct file *file, loff_t offset, int origin) | |||
699 | * origin == SEEK_END || SEEK_DATA || SEEK_HOLE => we must revalidate | 699 | * origin == SEEK_END || SEEK_DATA || SEEK_HOLE => we must revalidate |
700 | * the cached file length | 700 | * the cached file length |
701 | */ | 701 | */ |
702 | if (origin != SEEK_SET || origin != SEEK_CUR) { | 702 | if (origin != SEEK_SET && origin != SEEK_CUR) { |
703 | int rc; | 703 | int rc; |
704 | struct inode *inode = file->f_path.dentry->d_inode; | 704 | struct inode *inode = file->f_path.dentry->d_inode; |
705 | 705 | ||