diff options
author | David Sterba <dsterba@suse.com> | 2024-01-23 23:09:18 +0100 |
---|---|---|
committer | David Sterba <dsterba@suse.com> | 2024-03-04 16:24:48 +0100 |
commit | 504a00ac48a714845e4ae0593b4ab715f93ac297 (patch) | |
tree | a41576cdeb2aa8a7b6bebac96d4294761a2e0e1f /fs/btrfs/locking.c | |
parent | 2467d0fead77180d6be8b237683bbb375acf064b (diff) | |
download | linux-stable-504a00ac48a714845e4ae0593b4ab715f93ac297.tar.gz linux-stable-504a00ac48a714845e4ae0593b4ab715f93ac297.tar.bz2 linux-stable-504a00ac48a714845e4ae0593b4ab715f93ac297.zip |
btrfs: change BUG_ON to assertion when verifying lockdep class setup
The BUG_ON in btrfs_set_buffer_lockdep_class() is a sanity check of the
level which is verified in callers, e.g. when initializing an extent
buffer or reading from an eb header. Change it to an assertion as this
would not happen unless things are really bad and would fail elsewhere
too.
Reviewed-by: Josef Bacik <josef@toxicpanda.com>
Reviewed-by: Anand Jain <anand.jain@oracle.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/locking.c')
-rw-r--r-- | fs/btrfs/locking.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/locking.c b/fs/btrfs/locking.c index 286e6aa721c7..99ccab86bb86 100644 --- a/fs/btrfs/locking.c +++ b/fs/btrfs/locking.c @@ -84,7 +84,7 @@ void btrfs_set_buffer_lockdep_class(u64 objectid, struct extent_buffer *eb, int { struct btrfs_lockdep_keyset *ks; - BUG_ON(level >= ARRAY_SIZE(ks->keys)); + ASSERT(level < ARRAY_SIZE(ks->keys)); /* Find the matching keyset, id 0 is the default entry */ for (ks = btrfs_lockdep_keysets; ks->id; ks++) |