diff options
author | Christoph Hellwig <hch@lst.de> | 2014-09-07 15:15:52 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2014-09-11 11:12:16 -0400 |
commit | f0c63124a6165792f6e37e4b5983792d009e1ce8 (patch) | |
tree | a7f1bba20c1706707da46372d88c244221e225e3 | |
parent | 027bc41a3eb4759d60641c033c9a4c85be1cfd39 (diff) |
nfsd: update mtime on truncate
This fixes a failure in xfstests generic/313 because nfs doesn't update
mtime on a truncate. The protocol requires this to be done implicity
for a size changing setattr.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
-rw-r--r-- | fs/nfsd/vfs.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c index 89d1ae3bcff7..965cffd17a0c 100644 --- a/fs/nfsd/vfs.c +++ b/fs/nfsd/vfs.c | |||
@@ -445,6 +445,16 @@ nfsd_setattr(struct svc_rqst *rqstp, struct svc_fh *fhp, struct iattr *iap, | |||
445 | if (err) | 445 | if (err) |
446 | goto out; | 446 | goto out; |
447 | size_change = 1; | 447 | size_change = 1; |
448 | |||
449 | /* | ||
450 | * RFC5661, Section 18.30.4: | ||
451 | * Changing the size of a file with SETATTR indirectly | ||
452 | * changes the time_modify and change attributes. | ||
453 | * | ||
454 | * (and similar for the older RFCs) | ||
455 | */ | ||
456 | if (iap->ia_size != i_size_read(inode)) | ||
457 | iap->ia_valid |= ATTR_MTIME; | ||
448 | } | 458 | } |
449 | 459 | ||
450 | iap->ia_valid |= ATTR_CTIME; | 460 | iap->ia_valid |= ATTR_CTIME; |