btrfs: convert BUG_ON(EXTENT_BIT_LOCKED) checks to ASSERT's

We only call these functions from the qgroup code which doesn't call
with EXTENT_BIT_LOCKED.  These are BUG_ON()'s that exist to keep us
developers from using these functions with EXTENT_BIT_LOCKED, so convert
them to ASSERT()'s.

Signed-off-by: Josef Bacik <josef@toxicpanda.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>

diff --git a/fs/btrfs/extent_io.c b/fs/btrfs/extent_io.c
index a6d7d5c1f10817bbe58278cf50ebf1567a126b73..c600428edcc61b3b7b72981d6c6a9a0325dc16cb 100644 (file)
--- a/fs/btrfs/extent_io.c
+++ b/fs/btrfs/extent_io.c
@@ -1293,7 +1293,7 @@ int set_record_extent_bits(struct extent_io_tree *tree, u64 start, u64 end,
         * either fail with -EEXIST or changeset will record the whole
         * range.
         */
-       BUG_ON(bits & EXTENT_LOCKED);
+       ASSERT(!(bits & EXTENT_LOCKED));
 
        return set_extent_bit(tree, start, end, bits, 0, NULL, NULL, GFP_NOFS,
                              changeset);
@@ -1321,7 +1321,7 @@ int clear_record_extent_bits(struct extent_io_tree *tree, u64 start, u64 end,
         * Don't support EXTENT_LOCKED case, same reason as
         * set_record_extent_bits().
         */
-       BUG_ON(bits & EXTENT_LOCKED);
+       ASSERT(!(bits & EXTENT_LOCKED));
 
        return __clear_extent_bit(tree, start, end, bits, 0, 0, NULL, GFP_NOFS,
                                  changeset);