nfsd: special case truncates some more
authorChristoph Hellwig <hch@lst.de>
Tue, 24 Jan 2017 08:22:41 +0000 (09:22 +0100)
committerJ. Bruce Fields <bfields@redhat.com>
Tue, 31 Jan 2017 17:29:24 +0000 (12:29 -0500)
commit41f53350a0f36a7b8e31bec0d0ca907e028ab4cd
tree086931a2815c160d45576b05254ded0aa8c2cbf4
parentd19fb70dd68c4e960e2ac09b0b9c79dfdeefa726
nfsd: special case truncates some more

Both the NFS protocols and the Linux VFS use a setattr operation with a
bitmap of attributs to set to set various file attributes including the
file size and the uid/gid.

The Linux syscalls never mixes size updates with unrelated updates like
the uid/gid, and some file systems like XFS and GFS2 rely on the fact
that truncates might not update random other attributes, and many other
file systems handle the case but do not update the different attributes
in the same transaction.  NFSD on the other hand passes the attributes
it gets on the wire more or less directly through to the VFS, leading to
updates the file systems don't expect.  XFS at least has an assert on
the allowed attributes, which caught an unusual NFS client setting the
size and group at the same time.

To handle this issue properly this switches nfsd to call vfs_truncate
for size changes, and then handle all other attributes through
notify_change.  As a side effect this also means less boilerplace code
around the size change as we can now reuse the VFS code.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
fs/nfsd/vfs.c