diff options
| author | Christoph Hellwig <hch@infradead.org> | 2013-11-19 07:17:07 -0800 |
|---|---|---|
| committer | Ben Myers <bpm@sgi.com> | 2013-12-05 16:36:21 -0600 |
| commit | dff6efc326a4d5f305797d4a6bba14f374fdd633 (patch) | |
| tree | c0370871b238354313a3809b8911d7f633ed0c3a /fs/attr.c | |
| parent | b7d961b35b3ab69609aeea93f870269cb6e7ba4d (diff) | |
| download | lwn-dff6efc326a4d5f305797d4a6bba14f374fdd633.tar.gz lwn-dff6efc326a4d5f305797d4a6bba14f374fdd633.zip | |
fs: fix iversion handling
Currently notify_change directly updates i_version for size updates,
which not only is counter to how all other fields are updated through
struct iattr, but also breaks XFS, which need inode updates to happen
under its own lock, and synchronized to the structure that gets written
to the log.
Remove the update in the common code, and it to btrfs and ext4,
XFS already does a proper updaste internally and currently gets a
double update with the existing code.
IMHO this is 3.13 and -stable material and should go in through the XFS
tree.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Andreas Dilger <adilger@dilger.ca>
Acked-by: Jan Kara <jack@suse.cz>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Chris Mason <clm@fb.com>
Signed-off-by: Ben Myers <bpm@sgi.com>
Diffstat (limited to 'fs/attr.c')
| -rw-r--r-- | fs/attr.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/fs/attr.c b/fs/attr.c index 267968d94673..5d4e59d56e85 100644 --- a/fs/attr.c +++ b/fs/attr.c @@ -202,11 +202,6 @@ int notify_change(struct dentry * dentry, struct iattr * attr, struct inode **de return -EPERM; } - if ((ia_valid & ATTR_SIZE) && IS_I_VERSION(inode)) { - if (attr->ia_size != inode->i_size) - inode_inc_iversion(inode); - } - if ((ia_valid & ATTR_MODE)) { umode_t amode = attr->ia_mode; /* Flag setting protected by i_mutex */ |