f2fs: fix wrong AUTO_RECOVER condition

Jaegeuk Kim · pundiramit · commit 4935b166e960 · 2017-10-15T23:28:17.000+05:30
commit 97dd26ad834739d4e4ea35fd7ab5f92824de4cbb upstream.

If i_size is not aligned to the f2fs's block size, we should not skip inode
update during fsync.

Signed-off-by: Jaegeuk Kim &lt;jaegeuk@kernel.org&gt;
diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h
@@ -1774,7 +1774,8 @@ static inline bool f2fs_skip_inode_update(struct inode *inode, int dsync)
 		spin_unlock(&sbi->inode_lock[DIRTY_META]);
 		return ret;
 	}
-	if (!is_inode_flag_set(inode, FI_AUTO_RECOVER))
+	if (!is_inode_flag_set(inode, FI_AUTO_RECOVER) ||
+			i_size_read(inode) & PAGE_MASK)
 		return false;
 	return F2FS_I(inode)->last_disk_size == i_size_read(inode);
 }

Original file line number	Diff line number	Diff line change
`@@ -1774,7 +1774,8 @@ static inline bool f2fs_skip_inode_update(struct inode *inode, int dsync)`
`1774`	`1774`	`spin_unlock(&sbi->inode_lock[DIRTY_META]);`
`1775`	`1775`	`return ret;`
`1776`	`1776`	`}`
`1777`		`- if (!is_inode_flag_set(inode, FI_AUTO_RECOVER))`
	`1777`	`+ if (!is_inode_flag_set(inode, FI_AUTO_RECOVER) \|\|`
	`1778`	`+ i_size_read(inode) & PAGE_MASK)`
`1778`	`1779`	`return false;`
`1779`	`1780`	`return F2FS_I(inode)->last_disk_size == i_size_read(inode);`
`1780`	`1781`	`}`