aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChristoph Hellwig <hch@lst.de>2018-02-21 10:54:48 -0500
committerAl Viro <viro@zeniv.linux.org.uk>2018-03-28 01:39:02 -0400
commitf35562549ff9ecb9114f380843b3ac778c2a781e (patch)
treed28ff6ffd0dddb4e091e6acd29eba8c7fdefb6e1
parent2c2acd2d193595f2919583fd6e32b09ca8e85e94 (diff)
ubifs: fix bogus __mark_inode_dirty(I_DIRTY_SYNC | I_DIRTY_DATASYNC) call
I_DIRTY_DATASYNC is a strict superset of I_DIRTY_SYNC semantics, as in mark dirty to be written out by fdatasync as well. So dirtying for both flags makes no sense. Signed-off-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
-rw-r--r--fs/ubifs/file.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ubifs/file.c b/fs/ubifs/file.c
index cf348ba99238..1acb2ff505e6 100644
--- a/fs/ubifs/file.c
+++ b/fs/ubifs/file.c
@@ -1256,7 +1256,7 @@ static int do_setattr(struct ubifs_info *c, struct inode *inode,
1256 * Inode length changed, so we have to make sure 1256 * Inode length changed, so we have to make sure
1257 * @I_DIRTY_DATASYNC is set. 1257 * @I_DIRTY_DATASYNC is set.
1258 */ 1258 */
1259 __mark_inode_dirty(inode, I_DIRTY_SYNC | I_DIRTY_DATASYNC); 1259 __mark_inode_dirty(inode, I_DIRTY_DATASYNC);
1260 else 1260 else
1261 mark_inode_dirty_sync(inode); 1261 mark_inode_dirty_sync(inode);
1262 mutex_unlock(&ui->ui_mutex); 1262 mutex_unlock(&ui->ui_mutex);