]> git.ipfire.org Git - people/ms/linux.git/commit
btrfs: add missing inode updates on each iteration when replacing extents
authorFilipe Manana <fdmanana@suse.com>
Mon, 6 Jun 2022 09:41:18 +0000 (10:41 +0100)
committerDavid Sterba <dsterba@suse.com>
Tue, 21 Jun 2022 12:43:21 +0000 (14:43 +0200)
commit983d8209c6803345c9958f4cc358d1155f93a099
tree8189d30d3831151d570731432997524a05b84b4d
parentd4597898ba7b9d467b94a9aafd65ec408a75041f
btrfs: add missing inode updates on each iteration when replacing extents

When replacing file extents, called during fallocate, hole punching,
clone and deduplication, we may not be able to replace/drop all the
target file extent items with a single transaction handle. We may get
-ENOSPC while doing it, in which case we release the transaction handle,
balance the dirty pages of the btree inode, flush delayed items and get
a new transaction handle to operate on what's left of the target range.

By dropping and replacing file extent items we have effectively modified
the inode, so we should bump its iversion and update its mtime/ctime
before we update the inode item. This is because if the transaction
we used for partially modifying the inode gets committed by someone after
we release it and before we finish the rest of the range, a power failure
happens, then after mounting the filesystem our inode has an outdated
iversion and mtime/ctime, corresponding to the values it had before we
changed it.

So add the missing iversion and mtime/ctime updates.

Reviewed-by: Boris Burkov <boris@bur.io>
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/ctree.h
fs/btrfs/file.c
fs/btrfs/inode.c
fs/btrfs/reflink.c