btrfs: rename btrfs_leaf_data to BTRFS_LEAF_DATA_OFFSET
Commit 5f39d397df
("Btrfs: Create extent_buffer interface
for large blocksizes") refactored btrfs_leaf_data function to take
extent_buffer rather than struct btrfs_leaf. However, as it turns out the
parameter being passed is never used. Furthermore this function no longer
returns the leaf data but rather the offset to it. So rename the function
to BTRFS_LEAF_DATA_OFFSET to make it consistent with other BTRFS_LEAF_*
helpers and turn it into a macro.
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
[ removed () from the macro ]
Signed-off-by: David Sterba <dsterba@suse.com>
This commit is contained in:
parent
e1ddce71d6
commit
3d9ec8c49a
|
@ -3667,14 +3667,14 @@ static noinline int __push_leaf_right(struct btrfs_fs_info *fs_info,
|
||||||
/* make room in the right data area */
|
/* make room in the right data area */
|
||||||
data_end = leaf_data_end(fs_info, right);
|
data_end = leaf_data_end(fs_info, right);
|
||||||
memmove_extent_buffer(right,
|
memmove_extent_buffer(right,
|
||||||
btrfs_leaf_data(right) + data_end - push_space,
|
BTRFS_LEAF_DATA_OFFSET + data_end - push_space,
|
||||||
btrfs_leaf_data(right) + data_end,
|
BTRFS_LEAF_DATA_OFFSET + data_end,
|
||||||
BTRFS_LEAF_DATA_SIZE(fs_info) - data_end);
|
BTRFS_LEAF_DATA_SIZE(fs_info) - data_end);
|
||||||
|
|
||||||
/* copy from the left data area */
|
/* copy from the left data area */
|
||||||
copy_extent_buffer(right, left, btrfs_leaf_data(right) +
|
copy_extent_buffer(right, left, BTRFS_LEAF_DATA_OFFSET +
|
||||||
BTRFS_LEAF_DATA_SIZE(fs_info) - push_space,
|
BTRFS_LEAF_DATA_SIZE(fs_info) - push_space,
|
||||||
btrfs_leaf_data(left) + leaf_data_end(fs_info, left),
|
BTRFS_LEAF_DATA_OFFSET + leaf_data_end(fs_info, left),
|
||||||
push_space);
|
push_space);
|
||||||
|
|
||||||
memmove_extent_buffer(right, btrfs_item_nr_offset(push_items),
|
memmove_extent_buffer(right, btrfs_item_nr_offset(push_items),
|
||||||
|
@ -3888,9 +3888,9 @@ static noinline int __push_leaf_left(struct btrfs_fs_info *fs_info,
|
||||||
push_space = BTRFS_LEAF_DATA_SIZE(fs_info) -
|
push_space = BTRFS_LEAF_DATA_SIZE(fs_info) -
|
||||||
btrfs_item_offset_nr(right, push_items - 1);
|
btrfs_item_offset_nr(right, push_items - 1);
|
||||||
|
|
||||||
copy_extent_buffer(left, right, btrfs_leaf_data(left) +
|
copy_extent_buffer(left, right, BTRFS_LEAF_DATA_OFFSET +
|
||||||
leaf_data_end(fs_info, left) - push_space,
|
leaf_data_end(fs_info, left) - push_space,
|
||||||
btrfs_leaf_data(right) +
|
BTRFS_LEAF_DATA_OFFSET +
|
||||||
btrfs_item_offset_nr(right, push_items - 1),
|
btrfs_item_offset_nr(right, push_items - 1),
|
||||||
push_space);
|
push_space);
|
||||||
old_left_nritems = btrfs_header_nritems(left);
|
old_left_nritems = btrfs_header_nritems(left);
|
||||||
|
@ -3917,9 +3917,9 @@ static noinline int __push_leaf_left(struct btrfs_fs_info *fs_info,
|
||||||
if (push_items < right_nritems) {
|
if (push_items < right_nritems) {
|
||||||
push_space = btrfs_item_offset_nr(right, push_items - 1) -
|
push_space = btrfs_item_offset_nr(right, push_items - 1) -
|
||||||
leaf_data_end(fs_info, right);
|
leaf_data_end(fs_info, right);
|
||||||
memmove_extent_buffer(right, btrfs_leaf_data(right) +
|
memmove_extent_buffer(right, BTRFS_LEAF_DATA_OFFSET +
|
||||||
BTRFS_LEAF_DATA_SIZE(fs_info) - push_space,
|
BTRFS_LEAF_DATA_SIZE(fs_info) - push_space,
|
||||||
btrfs_leaf_data(right) +
|
BTRFS_LEAF_DATA_OFFSET +
|
||||||
leaf_data_end(fs_info, right), push_space);
|
leaf_data_end(fs_info, right), push_space);
|
||||||
|
|
||||||
memmove_extent_buffer(right, btrfs_item_nr_offset(0),
|
memmove_extent_buffer(right, btrfs_item_nr_offset(0),
|
||||||
|
@ -4069,8 +4069,8 @@ static noinline void copy_for_split(struct btrfs_trans_handle *trans,
|
||||||
nritems * sizeof(struct btrfs_item));
|
nritems * sizeof(struct btrfs_item));
|
||||||
|
|
||||||
copy_extent_buffer(right, l,
|
copy_extent_buffer(right, l,
|
||||||
btrfs_leaf_data(right) + BTRFS_LEAF_DATA_SIZE(fs_info) -
|
BTRFS_LEAF_DATA_OFFSET + BTRFS_LEAF_DATA_SIZE(fs_info) -
|
||||||
data_copy_size, btrfs_leaf_data(l) +
|
data_copy_size, BTRFS_LEAF_DATA_OFFSET +
|
||||||
leaf_data_end(fs_info, l), data_copy_size);
|
leaf_data_end(fs_info, l), data_copy_size);
|
||||||
|
|
||||||
rt_data_off = BTRFS_LEAF_DATA_SIZE(fs_info) - btrfs_item_end_nr(l, mid);
|
rt_data_off = BTRFS_LEAF_DATA_SIZE(fs_info) - btrfs_item_end_nr(l, mid);
|
||||||
|
@ -4607,8 +4607,8 @@ void btrfs_truncate_item(struct btrfs_fs_info *fs_info,
|
||||||
|
|
||||||
/* shift the data */
|
/* shift the data */
|
||||||
if (from_end) {
|
if (from_end) {
|
||||||
memmove_extent_buffer(leaf, btrfs_leaf_data(leaf) +
|
memmove_extent_buffer(leaf, BTRFS_LEAF_DATA_OFFSET +
|
||||||
data_end + size_diff, btrfs_leaf_data(leaf) +
|
data_end + size_diff, BTRFS_LEAF_DATA_OFFSET +
|
||||||
data_end, old_data_start + new_size - data_end);
|
data_end, old_data_start + new_size - data_end);
|
||||||
} else {
|
} else {
|
||||||
struct btrfs_disk_key disk_key;
|
struct btrfs_disk_key disk_key;
|
||||||
|
@ -4634,8 +4634,8 @@ void btrfs_truncate_item(struct btrfs_fs_info *fs_info,
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
memmove_extent_buffer(leaf, btrfs_leaf_data(leaf) +
|
memmove_extent_buffer(leaf, BTRFS_LEAF_DATA_OFFSET +
|
||||||
data_end + size_diff, btrfs_leaf_data(leaf) +
|
data_end + size_diff, BTRFS_LEAF_DATA_OFFSET +
|
||||||
data_end, old_data_start - data_end);
|
data_end, old_data_start - data_end);
|
||||||
|
|
||||||
offset = btrfs_disk_key_offset(&disk_key);
|
offset = btrfs_disk_key_offset(&disk_key);
|
||||||
|
@ -4707,8 +4707,8 @@ void btrfs_extend_item(struct btrfs_fs_info *fs_info, struct btrfs_path *path,
|
||||||
}
|
}
|
||||||
|
|
||||||
/* shift the data */
|
/* shift the data */
|
||||||
memmove_extent_buffer(leaf, btrfs_leaf_data(leaf) +
|
memmove_extent_buffer(leaf, BTRFS_LEAF_DATA_OFFSET +
|
||||||
data_end - data_size, btrfs_leaf_data(leaf) +
|
data_end - data_size, BTRFS_LEAF_DATA_OFFSET +
|
||||||
data_end, old_data - data_end);
|
data_end, old_data - data_end);
|
||||||
|
|
||||||
data_end = old_data;
|
data_end = old_data;
|
||||||
|
@ -4790,8 +4790,8 @@ void setup_items_for_insert(struct btrfs_root *root, struct btrfs_path *path,
|
||||||
(nritems - slot) * sizeof(struct btrfs_item));
|
(nritems - slot) * sizeof(struct btrfs_item));
|
||||||
|
|
||||||
/* shift the data */
|
/* shift the data */
|
||||||
memmove_extent_buffer(leaf, btrfs_leaf_data(leaf) +
|
memmove_extent_buffer(leaf, BTRFS_LEAF_DATA_OFFSET +
|
||||||
data_end - total_data, btrfs_leaf_data(leaf) +
|
data_end - total_data, BTRFS_LEAF_DATA_OFFSET +
|
||||||
data_end, old_data - data_end);
|
data_end, old_data - data_end);
|
||||||
data_end = old_data;
|
data_end = old_data;
|
||||||
}
|
}
|
||||||
|
@ -4983,9 +4983,9 @@ int btrfs_del_items(struct btrfs_trans_handle *trans, struct btrfs_root *root,
|
||||||
if (slot + nr != nritems) {
|
if (slot + nr != nritems) {
|
||||||
int data_end = leaf_data_end(fs_info, leaf);
|
int data_end = leaf_data_end(fs_info, leaf);
|
||||||
|
|
||||||
memmove_extent_buffer(leaf, btrfs_leaf_data(leaf) +
|
memmove_extent_buffer(leaf, BTRFS_LEAF_DATA_OFFSET +
|
||||||
data_end + dsize,
|
data_end + dsize,
|
||||||
btrfs_leaf_data(leaf) + data_end,
|
BTRFS_LEAF_DATA_OFFSET + data_end,
|
||||||
last_off - data_end);
|
last_off - data_end);
|
||||||
|
|
||||||
for (i = slot + nr; i < nritems; i++) {
|
for (i = slot + nr; i < nritems; i++) {
|
||||||
|
|
|
@ -1293,6 +1293,8 @@ static inline u32 BTRFS_LEAF_DATA_SIZE(const struct btrfs_fs_info *info)
|
||||||
return __BTRFS_LEAF_DATA_SIZE(info->nodesize);
|
return __BTRFS_LEAF_DATA_SIZE(info->nodesize);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#define BTRFS_LEAF_DATA_OFFSET offsetof(struct btrfs_leaf, items)
|
||||||
|
|
||||||
static inline u32 BTRFS_MAX_ITEM_SIZE(const struct btrfs_fs_info *info)
|
static inline u32 BTRFS_MAX_ITEM_SIZE(const struct btrfs_fs_info *info)
|
||||||
{
|
{
|
||||||
return BTRFS_LEAF_DATA_SIZE(info) - sizeof(struct btrfs_item);
|
return BTRFS_LEAF_DATA_SIZE(info) - sizeof(struct btrfs_item);
|
||||||
|
@ -2325,10 +2327,6 @@ static inline int btrfs_super_csum_size(struct btrfs_super_block *s)
|
||||||
return btrfs_csum_sizes[t];
|
return btrfs_csum_sizes[t];
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline unsigned long btrfs_leaf_data(struct extent_buffer *l)
|
|
||||||
{
|
|
||||||
return offsetof(struct btrfs_leaf, items);
|
|
||||||
}
|
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* The leaf data grows from end-to-front in the node.
|
* The leaf data grows from end-to-front in the node.
|
||||||
|
@ -2539,11 +2537,11 @@ BTRFS_SETGET_STACK_FUNCS(stack_dev_replace_cursor_right,
|
||||||
|
|
||||||
/* helper function to cast into the data area of the leaf. */
|
/* helper function to cast into the data area of the leaf. */
|
||||||
#define btrfs_item_ptr(leaf, slot, type) \
|
#define btrfs_item_ptr(leaf, slot, type) \
|
||||||
((type *)(btrfs_leaf_data(leaf) + \
|
((type *)(BTRFS_LEAF_DATA_OFFSET + \
|
||||||
btrfs_item_offset_nr(leaf, slot)))
|
btrfs_item_offset_nr(leaf, slot)))
|
||||||
|
|
||||||
#define btrfs_item_ptr_offset(leaf, slot) \
|
#define btrfs_item_ptr_offset(leaf, slot) \
|
||||||
((unsigned long)(btrfs_leaf_data(leaf) + \
|
((unsigned long)(BTRFS_LEAF_DATA_OFFSET + \
|
||||||
btrfs_item_offset_nr(leaf, slot)))
|
btrfs_item_offset_nr(leaf, slot)))
|
||||||
|
|
||||||
static inline bool btrfs_mixed_space_info(struct btrfs_space_info *space_info)
|
static inline bool btrfs_mixed_space_info(struct btrfs_space_info *space_info)
|
||||||
|
|
|
@ -3757,7 +3757,7 @@ static noinline_for_stack int write_one_eb(struct extent_buffer *eb,
|
||||||
* header 0 1 2 .. N ... data_N .. data_2 data_1 data_0
|
* header 0 1 2 .. N ... data_N .. data_2 data_1 data_0
|
||||||
*/
|
*/
|
||||||
start = btrfs_item_nr_offset(nritems);
|
start = btrfs_item_nr_offset(nritems);
|
||||||
end = btrfs_leaf_data(eb) + leaf_data_end(fs_info, eb);
|
end = BTRFS_LEAF_DATA_OFFSET + leaf_data_end(fs_info, eb);
|
||||||
memzero_extent_buffer(eb, start, end - start);
|
memzero_extent_buffer(eb, start, end - start);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue