diff options
| author |  Josef Bacik <josef@toxicpanda.com>
| 2021-10-05 16:35:25 -0400 |
|---|---|---|
| committer |  David Sterba <dsterba@suse.com>
| 2021-10-26 19:08:05 +0200 |
| commit | 849615394515cce02add9c5b931179162e251aab (patch) | |
| tree | 4b068cdce0d0cc751bb3a157b763919487c2dcf0 /fs/btrfs/scrub.c | |
| parent | btrfs: change error handling for btrfs_delete_*_in_log (diff) | |
| download | linux-849615394515cce02add9c5b931179162e251aab.tar.gz linux-849615394515cce02add9c5b931179162e251aab.tar.bz2 linux-849615394515cce02add9c5b931179162e251aab.zip | |
btrfs: add a BTRFS_FS_ERROR helper
We have a few flags that are inconsistently used to describe the fs in
different states of failure. As of 5963ffcaf383 ("btrfs: always abort
the transaction if we abort a trans handle") we will always set
BTRFS_FS_STATE_ERROR if we abort, so we don't have to check both ABORTED
and ERROR to see if things have gone wrong. Add a helper to check
BTRFS_FS_STATE_ERROR and then convert all checkers of FS_STATE_ERROR to
use the helper.
The TRANS_ABORTED bit check was added in af7227338135 ("Btrfs: clean up
resources during umount after trans is aborted") but is not actually
specific.
Reviewed-by: Anand Jain <anand.jain@oracle.com>
Reviewed-by: Nikolay Borisov <nborisov@suse.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/scrub.c')
| -rw-r--r-- | fs/btrfs/scrub.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/scrub.c b/fs/btrfs/scrub.c index bd3cd7427391..b1c26a90243b 100644 --- a/fs/btrfs/scrub.c +++ b/fs/btrfs/scrub.c @@ -3955,7 +3955,7 @@ static noinline_for_stack int scrub_supers(struct scrub_ctx *sctx, int ret; struct btrfs_fs_info *fs_info = sctx->fs_info; - if (test_bit(BTRFS_FS_STATE_ERROR, &fs_info->fs_state)) + if (BTRFS_FS_ERROR(fs_info)) return -EROFS; /* Seed devices of a new filesystem has their own generation. */ |