aboutsummaryrefslogtreecommitdiffstats
path: root/fs/btrfs/inode.c
diff options
context:
space:
mode:
authorJosef Bacik <josef@toxicpanda.com>2021-10-05 16:35:23 -0400
committerDavid Sterba <dsterba@suse.com>2021-10-26 19:08:05 +0200
commitba51e2a11e389a1e928e16d616c1276423c3a89e (patch)
tree563b00976c44f3e0ac606ce957baa39a599fe4be /fs/btrfs/inode.c
parentbtrfs: zoned: use kmemdup() to replace kmalloc + memcpy (diff)
downloadlinux-dev-ba51e2a11e389a1e928e16d616c1276423c3a89e.tar.xz
linux-dev-ba51e2a11e389a1e928e16d616c1276423c3a89e.zip
btrfs: change handle_fs_error in recover_log_trees to aborts
During inspection of the return path for replay I noticed that we don't actually abort the transaction if we get a failure during replay. This isn't a problem necessarily, as we properly return the error and will fail to mount. However we still leave this dangling transaction that could conceivably be committed without thinking there was an error. We were using btrfs_handle_fs_error() here, but that pre-dates the transaction abort code. Simply replace the btrfs_handle_fs_error() calls with transaction aborts, so we still know where exactly things went wrong, and add a few in some other un-handled error cases. Reviewed-by: Nikolay Borisov <nborisov@suse.com> Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com> Signed-off-by: Josef Bacik <josef@toxicpanda.com> Reviewed-by: David Sterba <dsterba@suse.com> Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/inode.c')
0 files changed, 0 insertions, 0 deletions