diff options
author | David Sterba <dsterba@suse.com> | 2024-01-19 22:15:41 +0300 |
---|---|---|
committer | David Sterba <dsterba@suse.com> | 2024-03-04 18:24:48 +0300 |
commit | 51d4be540054be32d7ce28b63ea9b84ac6ff1db2 (patch) | |
tree | caab4d0d46034ccbb17dfd9926b7c47d2fc9f71e | |
parent | be73f4448b607e6b7ce41cd8ef2214fdf6e7986f (diff) | |
download | linux-51d4be540054be32d7ce28b63ea9b84ac6ff1db2.tar.xz |
btrfs: defrag: change BUG_ON to assertion in btrfs_defrag_leaves()
The BUG_ON verifies a condition that should be guaranteed by the correct
use of the path search (with keep_locks and lowest_level set), an
assertion is the suitable check.
Reviewed-by: Josef Bacik <josef@toxicpanda.com>
Reviewed-by: Anand Jain <anand.jain@oracle.com>
Signed-off-by: David Sterba <dsterba@suse.com>
-rw-r--r-- | fs/btrfs/defrag.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/defrag.c b/fs/btrfs/defrag.c index eb62ff490c48..6f49de3ec92f 100644 --- a/fs/btrfs/defrag.c +++ b/fs/btrfs/defrag.c @@ -520,7 +520,7 @@ static int btrfs_defrag_leaves(struct btrfs_trans_handle *trans, * keep_locks set and lowest_level is 1, regardless of the value of * path->slots[1]. */ - BUG_ON(path->locks[1] == 0); + ASSERT(path->locks[1] != 0); ret = btrfs_realloc_node(trans, root, path->nodes[1], 0, &last_ret, |