btrfs: rename set_extent_bit() to include a btrfs prefix
authorFilipe Manana <fdmanana@suse.com>
Thu, 3 Apr 2025 14:00:26 +0000 (15:00 +0100)
committerDavid Sterba <dsterba@suse.com>
Thu, 15 May 2025 12:30:44 +0000 (14:30 +0200)
This is an exported function so it should have a 'btrfs_' prefix by
convention, to make it clear it's btrfs specific and to avoid collisions
with functions from elsewhere in the kernel.

So rename it to btrfs_set_extent_bit().

Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/block-group.c
fs/btrfs/defrag.c
fs/btrfs/dev-replace.c
fs/btrfs/extent-io-tree.c
fs/btrfs/extent-io-tree.h
fs/btrfs/extent-tree.c
fs/btrfs/file-item.c
fs/btrfs/inode.c
fs/btrfs/relocation.c
fs/btrfs/tests/extent-io-tests.c
fs/btrfs/volumes.c

index a5c587cf42a739d92bb4ab68fda2023ee686e751..944c355bf051017f9d3249a9a28d02bb86fca74f 100644 (file)
@@ -2216,9 +2216,9 @@ static int exclude_super_stripes(struct btrfs_block_group *cache)
        if (cache->start < BTRFS_SUPER_INFO_OFFSET) {
                stripe_len = BTRFS_SUPER_INFO_OFFSET - cache->start;
                cache->bytes_super += stripe_len;
-               ret = set_extent_bit(&fs_info->excluded_extents, cache->start,
-                                    cache->start + stripe_len - 1,
-                                    EXTENT_DIRTY, NULL);
+               ret = btrfs_set_extent_bit(&fs_info->excluded_extents, cache->start,
+                                          cache->start + stripe_len - 1,
+                                          EXTENT_DIRTY, NULL);
                if (ret)
                        return ret;
        }
@@ -2244,9 +2244,9 @@ static int exclude_super_stripes(struct btrfs_block_group *cache)
                                cache->start + cache->length - logical[nr]);
 
                        cache->bytes_super += len;
-                       ret = set_extent_bit(&fs_info->excluded_extents, logical[nr],
-                                            logical[nr] + len - 1,
-                                            EXTENT_DIRTY, NULL);
+                       ret = btrfs_set_extent_bit(&fs_info->excluded_extents,
+                                                  logical[nr], logical[nr] + len - 1,
+                                                  EXTENT_DIRTY, NULL);
                        if (ret) {
                                kfree(logical);
                                return ret;
@@ -3736,8 +3736,8 @@ int btrfs_update_block_group(struct btrfs_trans_handle *trans,
                spin_unlock(&cache->lock);
                spin_unlock(&space_info->lock);
 
-               set_extent_bit(&trans->transaction->pinned_extents, bytenr,
-                              bytenr + num_bytes - 1, EXTENT_DIRTY, NULL);
+               btrfs_set_extent_bit(&trans->transaction->pinned_extents, bytenr,
+                                    bytenr + num_bytes - 1, EXTENT_DIRTY, NULL);
        }
 
        spin_lock(&trans->transaction->dirty_bgs_lock);
index ee00250776bb311fe772b15c3e63dac23da4e8ad..587f2504a57012540d4c3c9e5f44778e0feab8c3 100644 (file)
@@ -1173,8 +1173,8 @@ static int defrag_one_locked_target(struct btrfs_inode *inode,
        btrfs_clear_extent_bit(&inode->io_tree, start, start + len - 1,
                               EXTENT_DELALLOC | EXTENT_DO_ACCOUNTING |
                               EXTENT_DEFRAG, cached_state);
-       set_extent_bit(&inode->io_tree, start, start + len - 1,
-                      EXTENT_DELALLOC | EXTENT_DEFRAG, cached_state);
+       btrfs_set_extent_bit(&inode->io_tree, start, start + len - 1,
+                            EXTENT_DELALLOC | EXTENT_DEFRAG, cached_state);
 
        /* Update the page status */
        for (i = start_index - first_index; i <= last_index - first_index; i++) {
index 53d7d85cb4bea2c42f7701754f367211831d00b7..db94c7f2246124c9d17b9d85568660abbe25b24c 100644 (file)
@@ -797,8 +797,8 @@ static int btrfs_set_target_alloc_state(struct btrfs_device *srcdev,
        while (find_first_extent_bit(&srcdev->alloc_state, start,
                                     &found_start, &found_end,
                                     CHUNK_ALLOCATED, &cached_state)) {
-               ret = set_extent_bit(&tgtdev->alloc_state, found_start,
-                                    found_end, CHUNK_ALLOCATED, NULL);
+               ret = btrfs_set_extent_bit(&tgtdev->alloc_state, found_start,
+                                          found_end, CHUNK_ALLOCATED, NULL);
                if (ret)
                        break;
                start = found_end + 1;
index 50b4d0b294688865d02f1cde93f2787b6b13ce47..49a9202dc5d0880e50ebecfd471830ebf0b710cc 100644 (file)
@@ -1256,8 +1256,8 @@ out:
 
 }
 
-int set_extent_bit(struct extent_io_tree *tree, u64 start, u64 end,
-                  u32 bits, struct extent_state **cached_state)
+int btrfs_set_extent_bit(struct extent_io_tree *tree, u64 start, u64 end,
+                        u32 bits, struct extent_state **cached_state)
 {
        return __set_extent_bit(tree, start, end, bits, NULL, NULL,
                                cached_state, NULL);
index 6c2148ed0bb65f9f3a66a1221f3ea6ade61e227d..e092d776a7557bf9a2f9956a102671900a612523 100644 (file)
@@ -199,8 +199,8 @@ static inline int btrfs_clear_extent_bits(struct extent_io_tree *tree, u64 start
 
 int set_record_extent_bits(struct extent_io_tree *tree, u64 start, u64 end,
                           u32 bits, struct extent_changeset *changeset);
-int set_extent_bit(struct extent_io_tree *tree, u64 start, u64 end,
-                  u32 bits, struct extent_state **cached_state);
+int btrfs_set_extent_bit(struct extent_io_tree *tree, u64 start, u64 end,
+                        u32 bits, struct extent_state **cached_state);
 
 static inline int btrfs_clear_extent_dirty(struct extent_io_tree *tree, u64 start,
                                           u64 end, struct extent_state **cached)
index 841f538a940ea8408cab2e6e7f6f1e8331b80826..06eb0d724f3dd8a52b952d10fb31778357886909 100644 (file)
@@ -2598,8 +2598,8 @@ static int pin_down_extent(struct btrfs_trans_handle *trans,
        spin_unlock(&cache->lock);
        spin_unlock(&cache->space_info->lock);
 
-       set_extent_bit(&trans->transaction->pinned_extents, bytenr,
-                      bytenr + num_bytes - 1, EXTENT_DIRTY, NULL);
+       btrfs_set_extent_bit(&trans->transaction->pinned_extents, bytenr,
+                            bytenr + num_bytes - 1, EXTENT_DIRTY, NULL);
        return 0;
 }
 
@@ -5065,17 +5065,17 @@ btrfs_init_new_buffer(struct btrfs_trans_handle *trans, struct btrfs_root *root,
                 * EXTENT bit to differentiate dirty pages.
                 */
                if (buf->log_index == 0)
-                       set_extent_bit(&root->dirty_log_pages, buf->start,
-                                      buf->start + buf->len - 1,
-                                      EXTENT_DIRTY, NULL);
+                       btrfs_set_extent_bit(&root->dirty_log_pages, buf->start,
+                                            buf->start + buf->len - 1,
+                                            EXTENT_DIRTY, NULL);
                else
-                       set_extent_bit(&root->dirty_log_pages, buf->start,
-                                      buf->start + buf->len - 1,
-                                      EXTENT_NEW, NULL);
+                       btrfs_set_extent_bit(&root->dirty_log_pages, buf->start,
+                                            buf->start + buf->len - 1,
+                                            EXTENT_NEW, NULL);
        } else {
                buf->log_index = -1;
-               set_extent_bit(&trans->transaction->dirty_pages, buf->start,
-                              buf->start + buf->len - 1, EXTENT_DIRTY, NULL);
+               btrfs_set_extent_bit(&trans->transaction->dirty_pages, buf->start,
+                                    buf->start + buf->len - 1, EXTENT_DIRTY, NULL);
        }
        /* this returns a buffer locked for blocking */
        return buf;
@@ -6430,8 +6430,8 @@ static int btrfs_trim_free_extents(struct btrfs_device *device, u64 *trimmed)
                ret = btrfs_issue_discard(device->bdev, start, len,
                                          &bytes);
                if (!ret)
-                       set_extent_bit(&device->alloc_state, start,
-                                      start + bytes - 1, CHUNK_TRIMMED, NULL);
+                       btrfs_set_extent_bit(&device->alloc_state, start,
+                                            start + bytes - 1, CHUNK_TRIMMED, NULL);
                mutex_unlock(&fs_info->chunk_mutex);
 
                if (ret)
index aa840ba9463468c26db704a5fe515ac30ff6872a..87e956cd35202987f2889834f99855419d7f4b11 100644 (file)
@@ -91,8 +91,8 @@ int btrfs_inode_set_file_extent_range(struct btrfs_inode *inode, u64 start,
 
        ASSERT(IS_ALIGNED(start + len, inode->root->fs_info->sectorsize));
 
-       return set_extent_bit(inode->file_extent_tree, start, start + len - 1,
-                             EXTENT_DIRTY, NULL);
+       return btrfs_set_extent_bit(inode->file_extent_tree, start, start + len - 1,
+                                   EXTENT_DIRTY, NULL);
 }
 
 /*
@@ -430,9 +430,9 @@ blk_status_t btrfs_lookup_bio_sums(struct btrfs_bio *bbio)
                        if (btrfs_root_id(inode->root) == BTRFS_DATA_RELOC_TREE_OBJECTID) {
                                u64 file_offset = bbio->file_offset + bio_offset;
 
-                               set_extent_bit(&inode->io_tree, file_offset,
-                                              file_offset + sectorsize - 1,
-                                              EXTENT_NODATASUM, NULL);
+                               btrfs_set_extent_bit(&inode->io_tree, file_offset,
+                                                    file_offset + sectorsize - 1,
+                                                    EXTENT_NODATASUM, NULL);
                        } else {
                                btrfs_warn_rl(fs_info,
                        "csum hole found for disk bytenr range [%llu, %llu)",
index 6fd904ee67ab468397d36931035237aea90bad70..b65e17dfaf177af583e094a285c967dbeaad7018 100644 (file)
@@ -2682,9 +2682,9 @@ static int btrfs_find_new_delalloc_bytes(struct btrfs_inode *inode,
                if (em_len > search_len)
                        em_len = search_len;
 
-               ret = set_extent_bit(&inode->io_tree, search_start,
-                                    search_start + em_len - 1,
-                                    EXTENT_DELALLOC_NEW, cached_state);
+               ret = btrfs_set_extent_bit(&inode->io_tree, search_start,
+                                          search_start + em_len - 1,
+                                          EXTENT_DELALLOC_NEW, cached_state);
 next:
                search_start = extent_map_end(em);
                free_extent_map(em);
@@ -2717,8 +2717,8 @@ int btrfs_set_extent_delalloc(struct btrfs_inode *inode, u64 start, u64 end,
                        return ret;
        }
 
-       return set_extent_bit(&inode->io_tree, start, end,
-                             EXTENT_DELALLOC | extra_bits, cached_state);
+       return btrfs_set_extent_bit(&inode->io_tree, start, end,
+                                   EXTENT_DELALLOC | extra_bits, cached_state);
 }
 
 /* see btrfs_writepage_start_hook for details on why this is required */
@@ -4901,8 +4901,8 @@ again:
        btrfs_unlock_extent(io_tree, block_start, block_end, &cached_state);
 
        if (only_release_metadata)
-               set_extent_bit(&inode->io_tree, block_start, block_end,
-                              EXTENT_NORESERVE, NULL);
+               btrfs_set_extent_bit(&inode->io_tree, block_start, block_end,
+                                    EXTENT_NORESERVE, NULL);
 
 out_unlock:
        if (ret) {
index c46b6386b5fe58ebc576db2ae87e35f9c7b2f95c..ec31cefd0dc1783bd0bd935dfc57e9e753b141db 100644 (file)
@@ -178,8 +178,9 @@ static void mark_block_processed(struct reloc_control *rc,
            in_range(node->bytenr, rc->block_group->start,
                     rc->block_group->length)) {
                blocksize = rc->extent_root->fs_info->nodesize;
-               set_extent_bit(&rc->processed_blocks, node->bytenr,
-                              node->bytenr + blocksize - 1, EXTENT_DIRTY, NULL);
+               btrfs_set_extent_bit(&rc->processed_blocks, node->bytenr,
+                                    node->bytenr + blocksize - 1, EXTENT_DIRTY,
+                                    NULL);
        }
        node->processed = 1;
 }
@@ -2929,9 +2930,9 @@ again:
                        u64 boundary_end = boundary_start +
                                           fs_info->sectorsize - 1;
 
-                       set_extent_bit(&BTRFS_I(inode)->io_tree,
-                                      boundary_start, boundary_end,
-                                      EXTENT_BOUNDARY, NULL);
+                       btrfs_set_extent_bit(&BTRFS_I(inode)->io_tree,
+                                            boundary_start, boundary_end,
+                                            EXTENT_BOUNDARY, NULL);
                }
                btrfs_unlock_extent(&BTRFS_I(inode)->io_tree, clamped_start, clamped_end,
                                    &cached_state);
index f715987c6f58f46b2ab0ca98d8f57c55a21fe44a..2d56c1ff715edf7bbcf5a52a0850b2f526ba2cd4 100644 (file)
@@ -176,7 +176,7 @@ static int test_find_delalloc(u32 sectorsize, u32 nodesize)
         * |--- delalloc ---|
         * |---  search  ---|
         */
-       set_extent_bit(tmp, 0, sectorsize - 1, EXTENT_DELALLOC, NULL);
+       btrfs_set_extent_bit(tmp, 0, sectorsize - 1, EXTENT_DELALLOC, NULL);
        start = 0;
        end = start + PAGE_SIZE - 1;
        found = find_lock_delalloc_range(inode, page_folio(locked_page), &start,
@@ -207,7 +207,7 @@ static int test_find_delalloc(u32 sectorsize, u32 nodesize)
                test_err("couldn't find the locked page");
                goto out_bits;
        }
-       set_extent_bit(tmp, sectorsize, max_bytes - 1, EXTENT_DELALLOC, NULL);
+       btrfs_set_extent_bit(tmp, sectorsize, max_bytes - 1, EXTENT_DELALLOC, NULL);
        start = test_start;
        end = start + PAGE_SIZE - 1;
        found = find_lock_delalloc_range(inode, page_folio(locked_page), &start,
@@ -262,7 +262,7 @@ static int test_find_delalloc(u32 sectorsize, u32 nodesize)
         *
         * We are re-using our test_start from above since it works out well.
         */
-       set_extent_bit(tmp, max_bytes, total_dirty - 1, EXTENT_DELALLOC, NULL);
+       btrfs_set_extent_bit(tmp, max_bytes, total_dirty - 1, EXTENT_DELALLOC, NULL);
        start = test_start;
        end = start + PAGE_SIZE - 1;
        found = find_lock_delalloc_range(inode, page_folio(locked_page), &start,
@@ -578,8 +578,8 @@ static int test_find_first_clear_extent_bit(void)
         * Set 1M-4M alloc/discard and 32M-64M thus leaving a hole between
         * 4M-32M
         */
-       set_extent_bit(&tree, SZ_1M, SZ_4M - 1,
-                      CHUNK_TRIMMED | CHUNK_ALLOCATED, NULL);
+       btrfs_set_extent_bit(&tree, SZ_1M, SZ_4M - 1,
+                            CHUNK_TRIMMED | CHUNK_ALLOCATED, NULL);
 
        find_first_clear_extent_bit(&tree, SZ_512K, &start, &end,
                                    CHUNK_TRIMMED | CHUNK_ALLOCATED);
@@ -591,8 +591,8 @@ static int test_find_first_clear_extent_bit(void)
        }
 
        /* Now add 32M-64M so that we have a hole between 4M-32M */
-       set_extent_bit(&tree, SZ_32M, SZ_64M - 1,
-                      CHUNK_TRIMMED | CHUNK_ALLOCATED, NULL);
+       btrfs_set_extent_bit(&tree, SZ_32M, SZ_64M - 1,
+                            CHUNK_TRIMMED | CHUNK_ALLOCATED, NULL);
 
        /*
         * Request first hole starting at 12M, we should get 4M-32M
@@ -623,7 +623,7 @@ static int test_find_first_clear_extent_bit(void)
         * Set 64M-72M with CHUNK_ALLOC flag, then search for CHUNK_TRIMMED flag
         * being unset in this range, we should get the entry in range 64M-72M
         */
-       set_extent_bit(&tree, SZ_64M, SZ_64M + SZ_8M - 1, CHUNK_ALLOCATED, NULL);
+       btrfs_set_extent_bit(&tree, SZ_64M, SZ_64M + SZ_8M - 1, CHUNK_ALLOCATED, NULL);
        find_first_clear_extent_bit(&tree, SZ_64M + SZ_1M, &start, &end,
                                    CHUNK_TRIMMED);
 
index 99856569abc9a3924fea2c25acf09d16a80349d9..f3b22cefb359efbb1bc332e8881699035c7847f3 100644 (file)
@@ -5407,9 +5407,9 @@ static void chunk_map_device_set_bits(struct btrfs_chunk_map *map, unsigned int
                struct btrfs_io_stripe *stripe = &map->stripes[i];
                struct btrfs_device *device = stripe->dev;
 
-               set_extent_bit(&device->alloc_state, stripe->physical,
-                              stripe->physical + map->stripe_size - 1,
-                              bits | EXTENT_NOWAIT, NULL);
+               btrfs_set_extent_bit(&device->alloc_state, stripe->physical,
+                                    stripe->physical + map->stripe_size - 1,
+                                    bits | EXTENT_NOWAIT, NULL);
        }
 }