Btrfs: fix race when checking if we can skip fsync'ing an inode
If we're about to do a fast fsync for an inode and btrfs_inode_in_log() returns false, it's possible that we had an ordered extent in progress (btrfs_finish_ordered_io() not run yet) when we noticed that the inode's last_trans field was not greater than the id of the last committed transaction, but shortly after, before we checked if there were any ongoing ordered extents, the ordered extent had just completed and removed itself from the inode's ordered tree, in which case we end up not logging the inode, losing some data if a power failure or crash happens after the fsync handler returns and before the transaction is committed. Fix this by checking first if there are any ongoing ordered extents before comparing the inode's last_trans with the id of the last committed transaction - when it completes, an ordered extent always updates the inode's last_trans before it removes itself from the inode's ordered tree (at btrfs_finish_ordered_io()). Signed-off-by: Filipe Manana <fdmanana@suse.com> Signed-off-by: Chris Mason <clm@fb.com>
This commit is contained in:
parent
daac7ba61a
commit
affc0ff902
|
@ -2017,10 +2017,11 @@ int btrfs_sync_file(struct file *file, loff_t start, loff_t end, int datasync)
|
||||||
*/
|
*/
|
||||||
smp_mb();
|
smp_mb();
|
||||||
if (btrfs_inode_in_log(inode, root->fs_info->generation) ||
|
if (btrfs_inode_in_log(inode, root->fs_info->generation) ||
|
||||||
(BTRFS_I(inode)->last_trans <=
|
(full_sync && BTRFS_I(inode)->last_trans <=
|
||||||
root->fs_info->last_trans_committed &&
|
root->fs_info->last_trans_committed) ||
|
||||||
(full_sync ||
|
(!btrfs_have_ordered_extents_in_range(inode, start, len) &&
|
||||||
!btrfs_have_ordered_extents_in_range(inode, start, len)))) {
|
BTRFS_I(inode)->last_trans
|
||||||
|
<= root->fs_info->last_trans_committed)) {
|
||||||
/*
|
/*
|
||||||
* We'v had everything committed since the last time we were
|
* We'v had everything committed since the last time we were
|
||||||
* modified so clear this flag in case it was set for whatever
|
* modified so clear this flag in case it was set for whatever
|
||||||
|
|
Loading…
Reference in a new issue