OSDN Git Service

btrfs: rename btrfs_clean_tree_block to btrfs_clear_buffer_dirty
authorJosef Bacik <josef@toxicpanda.com>
Thu, 26 Jan 2023 21:00:58 +0000 (16:00 -0500)
committerDavid Sterba <dsterba@suse.com>
Wed, 15 Feb 2023 18:38:53 +0000 (19:38 +0100)
btrfs_clean_tree_block is a misnomer, it's just
clear_extent_buffer_dirty with some extra accounting around it.  Rename
this to btrfs_clear_buffer_dirty to make it more clear it belongs with
it's setter, btrfs_mark_buffer_dirty.

Signed-off-by: Josef Bacik <josef@toxicpanda.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/ctree.c
fs/btrfs/disk-io.c
fs/btrfs/disk-io.h
fs/btrfs/extent-tree.c
fs/btrfs/free-space-tree.c
fs/btrfs/ioctl.c
fs/btrfs/qgroup.c
fs/btrfs/tree-log.c

index cae2df9..1184408 100644 (file)
@@ -484,7 +484,7 @@ static noinline int update_ref_for_cow(struct btrfs_trans_handle *trans,
                        if (ret)
                                return ret;
                }
-               btrfs_clean_tree_block(trans, buf);
+               btrfs_clear_buffer_dirty(trans, buf);
                *last_ref = 1;
        }
        return 0;
@@ -1054,7 +1054,7 @@ static noinline int balance_level(struct btrfs_trans_handle *trans,
 
                path->locks[level] = 0;
                path->nodes[level] = NULL;
-               btrfs_clean_tree_block(trans, mid);
+               btrfs_clear_buffer_dirty(trans, mid);
                btrfs_tree_unlock(mid);
                /* once for the path */
                free_extent_buffer(mid);
@@ -1115,7 +1115,7 @@ static noinline int balance_level(struct btrfs_trans_handle *trans,
                if (wret < 0 && wret != -ENOSPC)
                        ret = wret;
                if (btrfs_header_nritems(right) == 0) {
-                       btrfs_clean_tree_block(trans, right);
+                       btrfs_clear_buffer_dirty(trans, right);
                        btrfs_tree_unlock(right);
                        del_ptr(root, path, level + 1, pslot + 1);
                        root_sub_used(root, right->len);
@@ -1161,7 +1161,7 @@ static noinline int balance_level(struct btrfs_trans_handle *trans,
                BUG_ON(wret == 1);
        }
        if (btrfs_header_nritems(mid) == 0) {
-               btrfs_clean_tree_block(trans, mid);
+               btrfs_clear_buffer_dirty(trans, mid);
                btrfs_tree_unlock(mid);
                del_ptr(root, path, level + 1, pslot);
                root_sub_used(root, mid->len);
@@ -3140,7 +3140,7 @@ static noinline int __push_leaf_right(struct btrfs_trans_handle *trans,
        if (left_nritems)
                btrfs_mark_buffer_dirty(left);
        else
-               btrfs_clean_tree_block(trans, left);
+               btrfs_clear_buffer_dirty(trans, left);
 
        btrfs_mark_buffer_dirty(right);
 
@@ -3152,7 +3152,7 @@ static noinline int __push_leaf_right(struct btrfs_trans_handle *trans,
        if (path->slots[0] >= left_nritems) {
                path->slots[0] -= left_nritems;
                if (btrfs_header_nritems(path->nodes[0]) == 0)
-                       btrfs_clean_tree_block(trans, path->nodes[0]);
+                       btrfs_clear_buffer_dirty(trans, path->nodes[0]);
                btrfs_tree_unlock(path->nodes[0]);
                free_extent_buffer(path->nodes[0]);
                path->nodes[0] = right;
@@ -3365,7 +3365,7 @@ static noinline int __push_leaf_left(struct btrfs_trans_handle *trans,
        if (right_nritems)
                btrfs_mark_buffer_dirty(right);
        else
-               btrfs_clean_tree_block(trans, right);
+               btrfs_clear_buffer_dirty(trans, right);
 
        btrfs_item_key(right, &disk_key, 0);
        fixup_low_keys(path, &disk_key, 1);
@@ -4401,7 +4401,7 @@ int btrfs_del_items(struct btrfs_trans_handle *trans, struct btrfs_root *root,
                if (leaf == root->node) {
                        btrfs_set_header_level(leaf, 0);
                } else {
-                       btrfs_clean_tree_block(trans, leaf);
+                       btrfs_clear_buffer_dirty(trans, leaf);
                        btrfs_del_leaf(trans, root, path, leaf);
                }
        } else {
index be9bb89..2ef9e0b 100644 (file)
@@ -868,8 +868,8 @@ struct extent_buffer *read_tree_block(struct btrfs_fs_info *fs_info, u64 bytenr,
 
 }
 
-void btrfs_clean_tree_block(struct btrfs_trans_handle *trans,
-                           struct extent_buffer *buf)
+void btrfs_clear_buffer_dirty(struct btrfs_trans_handle *trans,
+                             struct extent_buffer *buf)
 {
        struct btrfs_fs_info *fs_info = buf->fs_info;
        if (!trans || btrfs_header_generation(buf) == trans->transid) {
@@ -4998,7 +4998,7 @@ static int btrfs_destroy_marked_extents(struct btrfs_fs_info *fs_info,
 
                        btrfs_tree_lock(eb);
                        wait_on_extent_buffer_writeback(eb);
-                       btrfs_clean_tree_block(NULL, eb);
+                       btrfs_clear_buffer_dirty(NULL, eb);
                        btrfs_tree_unlock(eb);
 
                        free_extent_buffer_stale(eb);
index d5466f1..4d57723 100644 (file)
@@ -39,8 +39,8 @@ struct extent_buffer *btrfs_find_create_tree_block(
                                                struct btrfs_fs_info *fs_info,
                                                u64 bytenr, u64 owner_root,
                                                int level);
-void btrfs_clean_tree_block(struct btrfs_trans_handle *trans,
-                           struct extent_buffer *buf);
+void btrfs_clear_buffer_dirty(struct btrfs_trans_handle *trans,
+                             struct extent_buffer *buf);
 void btrfs_clear_oneshot_options(struct btrfs_fs_info *fs_info);
 int btrfs_start_pre_rw_mount(struct btrfs_fs_info *fs_info);
 int btrfs_check_super_csum(struct btrfs_fs_info *fs_info,
index 8465327..824c657 100644 (file)
@@ -4838,7 +4838,7 @@ btrfs_init_new_buffer(struct btrfs_trans_handle *trans, struct btrfs_root *root,
        btrfs_set_buffer_lockdep_class(lockdep_owner, buf, level);
 
        __btrfs_tree_lock(buf, nest);
-       btrfs_clean_tree_block(trans, buf);
+       btrfs_clear_buffer_dirty(trans, buf);
        clear_bit(EXTENT_BUFFER_STALE, &buf->bflags);
        clear_bit(EXTENT_BUFFER_NO_CHECK, &buf->bflags);
 
@@ -5468,12 +5468,12 @@ static noinline int walk_up_proc(struct btrfs_trans_handle *trans,
                                }
                        }
                }
-               /* make block locked assertion in btrfs_clean_tree_block happy */
+               /* Make block locked assertion in btrfs_clear_buffer_dirty happy. */
                if (!path->locks[level]) {
                        btrfs_tree_lock(eb);
                        path->locks[level] = BTRFS_WRITE_LOCK;
                }
-               btrfs_clean_tree_block(trans, eb);
+               btrfs_clear_buffer_dirty(trans, eb);
        }
 
        if (eb == root->node) {
index ab206af..4d155a4 100644 (file)
@@ -1283,7 +1283,7 @@ int btrfs_clear_free_space_tree(struct btrfs_fs_info *fs_info)
        list_del(&free_space_root->dirty_list);
 
        btrfs_tree_lock(free_space_root->node);
-       btrfs_clean_tree_block(trans, free_space_root->node);
+       btrfs_clear_buffer_dirty(trans, free_space_root->node);
        btrfs_tree_unlock(free_space_root->node);
        btrfs_free_tree_block(trans, btrfs_root_id(free_space_root),
                              free_space_root->node, 0, 1);
index 5fdc09f..8ea557e 100644 (file)
@@ -707,7 +707,7 @@ static noinline int create_subvol(struct user_namespace *mnt_userns,
                 * exists).
                 */
                btrfs_tree_lock(leaf);
-               btrfs_clean_tree_block(trans, leaf);
+               btrfs_clear_buffer_dirty(trans, leaf);
                btrfs_tree_unlock(leaf);
                btrfs_free_tree_block(trans, objectid, leaf, 0, 1);
                free_extent_buffer(leaf);
index 450d8dd..52a7d2f 100644 (file)
@@ -1304,7 +1304,7 @@ int btrfs_quota_disable(struct btrfs_fs_info *fs_info)
        list_del(&quota_root->dirty_list);
 
        btrfs_tree_lock(quota_root->node);
-       btrfs_clean_tree_block(trans, quota_root->node);
+       btrfs_clear_buffer_dirty(trans, quota_root->node);
        btrfs_tree_unlock(quota_root->node);
        btrfs_free_tree_block(trans, btrfs_root_id(quota_root),
                              quota_root->node, 0, 1);
index a74acb3..0297379 100644 (file)
@@ -2624,7 +2624,7 @@ static noinline int walk_down_log_tree(struct btrfs_trans_handle *trans,
                                }
 
                                btrfs_tree_lock(next);
-                               btrfs_clean_tree_block(trans, next);
+                               btrfs_clear_buffer_dirty(trans, next);
                                btrfs_wait_tree_block_writeback(next);
                                btrfs_tree_unlock(next);
 
@@ -2693,7 +2693,7 @@ static noinline int walk_up_log_tree(struct btrfs_trans_handle *trans,
                                next = path->nodes[*level];
 
                                btrfs_tree_lock(next);
-                               btrfs_clean_tree_block(trans, next);
+                               btrfs_clear_buffer_dirty(trans, next);
                                btrfs_wait_tree_block_writeback(next);
                                btrfs_tree_unlock(next);
 
@@ -2774,7 +2774,7 @@ static int walk_log_tree(struct btrfs_trans_handle *trans,
                        next = path->nodes[orig_level];
 
                        btrfs_tree_lock(next);
-                       btrfs_clean_tree_block(trans, next);
+                       btrfs_clear_buffer_dirty(trans, next);
                        btrfs_wait_tree_block_writeback(next);
                        btrfs_tree_unlock(next);