Skip to content

Commit e7e6572

Browse files
committed
btrfs-progs: kernel-shared: update const of parameters accessors.h
Sync up with kernel and fix warnings reported by -Wcast-qual. eg. Most of the change is due to extent_buffer::data, which is a direct struct member, unlike in kernel where it's an array of pages. The const qualifier cannot be used the same way so it's dropped in affected herlpers. Signed-off-by: David Sterba <dsterba@suse.com>
1 parent 1083f2e commit e7e6572

File tree

5 files changed

+41
-37
lines changed

5 files changed

+41
-37
lines changed

kernel-shared/accessors.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ void btrfs_init_map_token(struct btrfs_map_token *token, struct extent_buffer *e
4040
/*
4141
* MODIFIED:
4242
* - We don't have eb->folios, simply wrap the set/get helpers.
43+
* - no const for extent buffer in btrfs_set_*
4344
*/
4445

4546
/*
@@ -96,7 +97,7 @@ void btrfs_set_token_##bits(struct btrfs_map_token *token, \
9697
ASSERT(check_setget_bounds(token->eb, ptr, off, size)); \
9798
put_unaligned_le##bits(val, token->kaddr + member_offset); \
9899
} \
99-
void btrfs_set_##bits(const struct extent_buffer *eb, void *ptr, \
100+
void btrfs_set_##bits(struct extent_buffer *eb, void *ptr, \
100101
unsigned long off, u##bits val) \
101102
{ \
102103
unsigned long member_offset = (unsigned long)ptr + off; \

kernel-shared/accessors.h

Lines changed: 19 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ void btrfs_init_map_token(struct btrfs_map_token *token, struct extent_buffer *e
3434

3535
static inline u8 get_unaligned_le8(const void *p)
3636
{
37-
return *(u8 *)p;
37+
return *(const u8 *)p;
3838
}
3939

4040
static inline void put_unaligned_le8(u8 val, void *p)
@@ -48,8 +48,8 @@ static inline void put_unaligned_le8(u8 val, void *p)
4848
offsetof(type, member), \
4949
sizeof(((type *)0)->member)))
5050

51-
#define write_eb_member(eb, ptr, type, member, result) (\
52-
write_extent_buffer(eb, (char *)(result), \
51+
#define write_eb_member(eb, ptr, type, member, source) ( \
52+
write_extent_buffer(eb, (const char *)(source), \
5353
((unsigned long)(ptr)) + \
5454
offsetof(type, member), \
5555
sizeof(((type *)0)->member)))
@@ -62,7 +62,7 @@ void btrfs_set_token_##bits(struct btrfs_map_token *token, \
6262
u##bits val); \
6363
u##bits btrfs_get_##bits(const struct extent_buffer *eb, \
6464
const void *ptr, unsigned long off); \
65-
void btrfs_set_##bits(const struct extent_buffer *eb, void *ptr, \
65+
void btrfs_set_##bits(struct extent_buffer *eb, void *ptr, \
6666
unsigned long off, u##bits val);
6767

6868
DECLARE_BTRFS_SETGET_BITS(8)
@@ -77,7 +77,7 @@ static inline u##bits btrfs_##name(const struct extent_buffer *eb, \
7777
_static_assert(sizeof(u##bits) == sizeof(((type *)0))->member); \
7878
return btrfs_get_##bits(eb, s, offsetof(type, member)); \
7979
} \
80-
static inline void btrfs_set_##name(const struct extent_buffer *eb, type *s, \
80+
static inline void btrfs_set_##name(struct extent_buffer *eb, type *s, \
8181
u##bits val) \
8282
{ \
8383
_static_assert(sizeof(u##bits) == sizeof(((type *)0))->member); \
@@ -99,14 +99,16 @@ static inline void btrfs_set_token_##name(struct btrfs_map_token *token,\
9999
/*
100100
* MODIFIED:
101101
* - We have eb->data, not eb->folios[0]
102+
* - no const for extent buffer in btrfs_set_*, buffer is local, in kernel
103+
* it's indirection to pages/folios
102104
*/
103105
#define BTRFS_SETGET_HEADER_FUNCS(name, type, member, bits) \
104106
static inline u##bits btrfs_##name(const struct extent_buffer *eb) \
105107
{ \
106-
const type *p = (type *)eb->data; \
108+
const type *p = (const type *)eb->data; \
107109
return get_unaligned_le##bits(&p->member); \
108110
} \
109-
static inline void btrfs_set_##name(const struct extent_buffer *eb, \
111+
static inline void btrfs_set_##name(struct extent_buffer *eb, \
110112
u##bits val) \
111113
{ \
112114
type *p = (type *)eb->data; \
@@ -133,8 +135,9 @@ static inline u64 btrfs_device_total_bytes(const struct extent_buffer *eb,
133135
/*
134136
* MODIFIED
135137
* - Removed WARN_ON(!IS_ALIGNED(val, eb->fs_info->sectorsize));
138+
* - no const for extent buffer
136139
*/
137-
static inline void btrfs_set_device_total_bytes(const struct extent_buffer *eb,
140+
static inline void btrfs_set_device_total_bytes(struct extent_buffer *eb,
138141
struct btrfs_dev_item *s,
139142
u64 val)
140143
{
@@ -379,9 +382,9 @@ static inline void btrfs_tree_block_key(const struct extent_buffer *eb,
379382
read_eb_member(eb, item, struct btrfs_tree_block_info, key, key);
380383
}
381384

382-
static inline void btrfs_set_tree_block_key(const struct extent_buffer *eb,
385+
static inline void btrfs_set_tree_block_key(struct extent_buffer *eb,
383386
struct btrfs_tree_block_info *item,
384-
struct btrfs_disk_key *key)
387+
const struct btrfs_disk_key *key)
385388
{
386389
write_eb_member(eb, item, struct btrfs_tree_block_info, key, key);
387390
}
@@ -435,7 +438,7 @@ static inline u64 btrfs_node_blockptr(const struct extent_buffer *eb, int nr)
435438
return btrfs_key_blockptr(eb, (struct btrfs_key_ptr *)ptr);
436439
}
437440

438-
static inline void btrfs_set_node_blockptr(const struct extent_buffer *eb,
441+
static inline void btrfs_set_node_blockptr(struct extent_buffer *eb,
439442
int nr, u64 val)
440443
{
441444
unsigned long ptr;
@@ -454,7 +457,7 @@ static inline u64 btrfs_node_ptr_generation(const struct extent_buffer *eb, int
454457
return btrfs_key_generation(eb, (struct btrfs_key_ptr *)ptr);
455458
}
456459

457-
static inline void btrfs_set_node_ptr_generation(const struct extent_buffer *eb,
460+
static inline void btrfs_set_node_ptr_generation(struct extent_buffer *eb,
458461
int nr, u64 val)
459462
{
460463
unsigned long ptr;
@@ -473,8 +476,8 @@ static inline unsigned long btrfs_node_key_ptr_offset(const struct extent_buffer
473476
void btrfs_node_key(const struct extent_buffer *eb,
474477
struct btrfs_disk_key *disk_key, int nr);
475478

476-
static inline void btrfs_set_node_key(const struct extent_buffer *eb,
477-
struct btrfs_disk_key *disk_key, int nr)
479+
static inline void btrfs_set_node_key(struct extent_buffer *eb,
480+
const struct btrfs_disk_key *disk_key, int nr)
478481
{
479482
unsigned long ptr;
480483

@@ -505,7 +508,7 @@ static inline u32 btrfs_item_##member(const struct extent_buffer *eb, int slot)
505508
{ \
506509
return btrfs_raw_item_##member(eb, btrfs_item_nr(eb, slot)); \
507510
} \
508-
static inline void btrfs_set_item_##member(const struct extent_buffer *eb, \
511+
static inline void btrfs_set_item_##member(struct extent_buffer *eb, \
509512
int slot, u32 val) \
510513
{ \
511514
btrfs_set_raw_item_##member(eb, btrfs_item_nr(eb, slot), val); \
@@ -540,7 +543,7 @@ static inline void btrfs_item_key(const struct extent_buffer *eb,
540543
}
541544

542545
static inline void btrfs_set_item_key(struct extent_buffer *eb,
543-
struct btrfs_disk_key *disk_key, int nr)
546+
const struct btrfs_disk_key *disk_key, int nr)
544547
{
545548
struct btrfs_item *item = btrfs_item_nr(eb, nr);
546549

kernel-shared/ctree.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ static unsigned int leaf_data_end(const struct extent_buffer *leaf)
7979
* have to adjust any offsets to account for the header in the leaf. This
8080
* handles that math to simplify the callers.
8181
*/
82-
static inline void memmove_leaf_data(const struct extent_buffer *leaf,
82+
static inline void memmove_leaf_data(struct extent_buffer *leaf,
8383
unsigned long dst_offset,
8484
unsigned long src_offset,
8585
unsigned long len)
@@ -102,7 +102,7 @@ static inline void memmove_leaf_data(const struct extent_buffer *leaf,
102102
* have to adjust any offsets to account for the header in the leaf. This
103103
* handles that math to simplify the callers.
104104
*/
105-
static inline void copy_leaf_data(const struct extent_buffer *dst,
105+
static inline void copy_leaf_data(struct extent_buffer *dst,
106106
const struct extent_buffer *src,
107107
unsigned long dst_offset,
108108
unsigned long src_offset, unsigned long len)
@@ -122,7 +122,7 @@ static inline void copy_leaf_data(const struct extent_buffer *dst,
122122
* Wrapper around memmove_extent_buffer() that does the math to get the
123123
* appropriate offsets into the leaf from the item numbers.
124124
*/
125-
static inline void memmove_leaf_items(const struct extent_buffer *leaf,
125+
static inline void memmove_leaf_items(struct extent_buffer *leaf,
126126
int dst_item, int src_item, int nr_items)
127127
{
128128
memmove_extent_buffer(leaf, btrfs_item_nr_offset(leaf, dst_item),
@@ -142,7 +142,7 @@ static inline void memmove_leaf_items(const struct extent_buffer *leaf,
142142
* Wrapper around copy_extent_buffer() that does the math to get the
143143
* appropriate offsets into the leaf from the item numbers.
144144
*/
145-
static inline void copy_leaf_items(const struct extent_buffer *dst,
145+
static inline void copy_leaf_items(struct extent_buffer *dst,
146146
const struct extent_buffer *src,
147147
int dst_item, int src_item, int nr_items)
148148
{

kernel-shared/extent_io.c

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -612,50 +612,50 @@ void read_extent_buffer(const struct extent_buffer *eb, void *dst,
612612
memcpy(dst, eb->data + start, len);
613613
}
614614

615-
void write_extent_buffer_fsid(const struct extent_buffer *eb, const void *src)
615+
void write_extent_buffer_fsid(struct extent_buffer *eb, const void *src)
616616
{
617617
write_extent_buffer(eb, src, btrfs_header_fsid(), BTRFS_FSID_SIZE);
618618
}
619619

620-
void write_extent_buffer_chunk_tree_uuid(const struct extent_buffer *eb,
620+
void write_extent_buffer_chunk_tree_uuid(struct extent_buffer *eb,
621621
const void *src)
622622
{
623623
write_extent_buffer(eb, src, btrfs_header_chunk_tree_uuid(eb), BTRFS_FSID_SIZE);
624624
}
625625

626-
void write_extent_buffer(const struct extent_buffer *eb, const void *src,
626+
void write_extent_buffer(struct extent_buffer *eb, const void *src,
627627
unsigned long start, unsigned long len)
628628
{
629629
memcpy((void *)eb->data + start, src, len);
630630
}
631631

632-
void copy_extent_buffer_full(const struct extent_buffer *dst,
632+
void copy_extent_buffer_full(struct extent_buffer *dst,
633633
const struct extent_buffer *src)
634634
{
635635
copy_extent_buffer(dst, src, 0, 0, src->len);
636636
}
637637

638-
void copy_extent_buffer(const struct extent_buffer *dst,
638+
void copy_extent_buffer(struct extent_buffer *dst,
639639
const struct extent_buffer *src,
640640
unsigned long dst_offset, unsigned long src_offset,
641641
unsigned long len)
642642
{
643643
memcpy((void *)dst->data + dst_offset, src->data + src_offset, len);
644644
}
645645

646-
void memcpy_extent_buffer(const struct extent_buffer *dst, unsigned long dst_offset,
646+
void memcpy_extent_buffer(struct extent_buffer *dst, unsigned long dst_offset,
647647
unsigned long src_offset, unsigned long len)
648648
{
649649
memcpy((void *)dst->data + dst_offset, dst->data + src_offset, len);
650650
}
651651

652-
void memmove_extent_buffer(const struct extent_buffer *dst, unsigned long dst_offset,
652+
void memmove_extent_buffer(struct extent_buffer *dst, unsigned long dst_offset,
653653
unsigned long src_offset, unsigned long len)
654654
{
655655
memmove((void *)dst->data + dst_offset, dst->data + src_offset, len);
656656
}
657657

658-
void memset_extent_buffer(const struct extent_buffer *eb, char c,
658+
void memset_extent_buffer(struct extent_buffer *eb, char c,
659659
unsigned long start, unsigned long len)
660660
{
661661
memset((void *)eb->data + start, c, len);

kernel-shared/extent_io.h

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -109,24 +109,24 @@ int memcmp_extent_buffer(const struct extent_buffer *eb, const void *ptrv,
109109
unsigned long start, unsigned long len);
110110
void read_extent_buffer(const struct extent_buffer *eb, void *dst,
111111
unsigned long start, unsigned long len);
112-
void write_extent_buffer_fsid(const struct extent_buffer *eb, const void *src);
113-
void write_extent_buffer_chunk_tree_uuid(const struct extent_buffer *eb,
112+
void write_extent_buffer_fsid(struct extent_buffer *eb, const void *src);
113+
void write_extent_buffer_chunk_tree_uuid(struct extent_buffer *eb,
114114
const void *src);
115-
void write_extent_buffer(const struct extent_buffer *eb, const void *src,
115+
void write_extent_buffer(struct extent_buffer *eb, const void *src,
116116
unsigned long start, unsigned long len);
117-
void copy_extent_buffer_full(const struct extent_buffer *dst,
117+
void copy_extent_buffer_full(struct extent_buffer *dst,
118118
const struct extent_buffer *src);
119-
void copy_extent_buffer(const struct extent_buffer *dst,
119+
void copy_extent_buffer(struct extent_buffer *dst,
120120
const struct extent_buffer *src,
121121
unsigned long dst_offset, unsigned long src_offset,
122122
unsigned long len);
123-
void memcpy_extent_buffer(const struct extent_buffer *dst,
123+
void memcpy_extent_buffer(struct extent_buffer *dst,
124124
unsigned long dst_offset, unsigned long src_offset,
125125
unsigned long len);
126-
void memmove_extent_buffer(const struct extent_buffer *dst,
126+
void memmove_extent_buffer(struct extent_buffer *dst,
127127
const unsigned long dst_offset,
128128
unsigned long src_offset, unsigned long len);
129-
void memset_extent_buffer(const struct extent_buffer *eb, char c,
129+
void memset_extent_buffer(struct extent_buffer *eb, char c,
130130
unsigned long start, unsigned long len);
131131
int extent_buffer_test_bit(const struct extent_buffer *eb, unsigned long start,
132132
unsigned long nr);

0 commit comments

Comments
 (0)