diff options
author | Christoph Hellwig <hch@infradead.org> | 2010-07-18 17:51:21 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2010-08-09 16:48:48 -0400 |
commit | 1b9474635e21eef0f3e69fd1c7b1b9598ffdddd3 (patch) | |
tree | 89c63dc57f7c197a8f48633c5e05d9c1f3b87679 /fs/affs/super.c | |
parent | e566d48c9bd56f57e25e855a21e06ca2c2525795 (diff) | |
download | op-kernel-dev-1b9474635e21eef0f3e69fd1c7b1b9598ffdddd3.zip op-kernel-dev-1b9474635e21eef0f3e69fd1c7b1b9598ffdddd3.tar.gz |
cifs: truncate fallout
Remove the calls to inode_newsize_ok given that we already did it as
part of inode_change_ok in the beginning of cifs_setattr_(no)unix.
No need to call ->truncate if cifs doesn't have one, so remove the
explicit call in cifs_vmtruncate, and replace the calls to vmtruncate
with truncate_setsize which is vmtruncate minus inode_newsize_ok
and the call to ->truncate.
Rename cifs_vmtruncate to cifs_setsize to match the new calling conventions.
Question 1: why does cifs do the pagecache munging and i_size update twice
for each setattr call, once opencoded in cifs_vmtruncate, and once
using the VFS helpers?
Question 2: what is supposed to be protected by i_lock in cifs_vmtruncate?
Do we need it around the call to inode_change_ok?
[AV: fixed build breakage]
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/affs/super.c')
0 files changed, 0 insertions, 0 deletions