]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
btrfs: don't ignore inode missing when replaying log tree
authorFilipe Manana <fdmanana@suse.com>
Fri, 11 Jul 2025 19:21:28 +0000 (20:21 +0100)
committerDavid Sterba <dsterba@suse.com>
Mon, 21 Jul 2025 23:13:40 +0000 (01:13 +0200)
commit7ebf381a69421a88265d3c49cd0f007ba7336c9d
treef11d9c5afe7cf28652f6762ecb69acfcda18136b
parent041c39da53c25f94a4705466b7f980c979215767
btrfs: don't ignore inode missing when replaying log tree

During log replay, at add_inode_ref(), we return -ENOENT if our current
inode isn't found on the subvolume tree or if a parent directory isn't
found. The error comes from btrfs_iget_logging() <- btrfs_iget() <-
btrfs_read_locked_inode().

The single caller of add_inode_ref(), replay_one_buffer(), ignores an
-ENOENT error because it expects that error to mean only that a parent
directory wasn't found and that is ok.

Before commit 5f61b961599a ("btrfs: fix inode lookup error handling during
log replay") we were converting any error when getting a parent directory
to -ENOENT and any error when getting the current inode to -EIO, so our
caller would fail log replay in case we can't find the current inode.
After that commit however in case the current inode is not found we return
-ENOENT to the caller and therefore it ignores the critical fact that the
current inode was not found in the subvolume tree.

Fix this by converting -ENOENT to 0 when we don't find a parent directory,
returning -ENOENT when we don't find the current inode and making the
caller, replay_one_buffer(), not ignore -ENOENT anymore.

Fixes: 5f61b961599a ("btrfs: fix inode lookup error handling during log replay")
CC: stable@vger.kernel.org # 6.16
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/tree-log.c