btrfs-progs: remove raid stripe encoding

Remove the not needed encoding and reserved fields in struct
raid_stripe_extent.

This saves 8 bytes per stripe extent.

Note: this is a format change and previously created filesystems with
raid-stripe-tree will not be accessible. Similar patch is needed in
kernel.

Signed-off-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Signed-off-by: David Sterba <dsterba@suse.com>
This commit is contained in:
Johannes Thumshirn 2024-06-20 09:54:55 +02:00 committed by David Sterba
parent d6e23e1d5f
commit 7c549b5f7c
4 changed files with 2 additions and 67 deletions

View File

@ -322,11 +322,8 @@ BTRFS_SETGET_FUNCS(timespec_nsec, struct btrfs_timespec, nsec, 32);
BTRFS_SETGET_STACK_FUNCS(stack_timespec_sec, struct btrfs_timespec, sec, 64); BTRFS_SETGET_STACK_FUNCS(stack_timespec_sec, struct btrfs_timespec, sec, 64);
BTRFS_SETGET_STACK_FUNCS(stack_timespec_nsec, struct btrfs_timespec, nsec, 32); BTRFS_SETGET_STACK_FUNCS(stack_timespec_nsec, struct btrfs_timespec, nsec, 32);
BTRFS_SETGET_FUNCS(stripe_extent_encoding, struct btrfs_stripe_extent, encoding, 8);
BTRFS_SETGET_FUNCS(raid_stride_devid, struct btrfs_raid_stride, devid, 64); BTRFS_SETGET_FUNCS(raid_stride_devid, struct btrfs_raid_stride, devid, 64);
BTRFS_SETGET_FUNCS(raid_stride_offset, struct btrfs_raid_stride, offset, 64); BTRFS_SETGET_FUNCS(raid_stride_offset, struct btrfs_raid_stride, offset, 64);
BTRFS_SETGET_STACK_FUNCS(stack_stripe_extent_encoding,
struct btrfs_stripe_extent, encoding, 8);
BTRFS_SETGET_STACK_FUNCS(stack_raid_stride_devid, struct btrfs_raid_stride, devid, 64); BTRFS_SETGET_STACK_FUNCS(stack_raid_stride_devid, struct btrfs_raid_stride, devid, 64);
static inline struct btrfs_raid_stride *btrfs_raid_stride_nr( static inline struct btrfs_raid_stride *btrfs_raid_stride_nr(

View File

@ -669,42 +669,11 @@ static void print_free_space_header(struct extent_buffer *leaf, int slot)
(unsigned long long)btrfs_free_space_bitmaps(leaf, header)); (unsigned long long)btrfs_free_space_bitmaps(leaf, header));
} }
struct raid_encoding_map {
u8 encoding;
char name[16];
};
static const struct raid_encoding_map raid_map[] = {
{ BTRFS_STRIPE_DUP, "DUP" },
{ BTRFS_STRIPE_RAID0, "RAID0" },
{ BTRFS_STRIPE_RAID1, "RAID1" },
{ BTRFS_STRIPE_RAID1C3, "RAID1C3" },
{ BTRFS_STRIPE_RAID1C4, "RAID1C4" },
{ BTRFS_STRIPE_RAID5, "RAID5" },
{ BTRFS_STRIPE_RAID6, "RAID6" },
{ BTRFS_STRIPE_RAID10, "RAID10" }
};
static const char *stripe_encoding_name(u8 encoding)
{
for (int i = 0; i < ARRAY_SIZE(raid_map); i++) {
if (raid_map[i].encoding == encoding)
return raid_map[i].name;
}
return "UNKNOWN";
}
static void print_raid_stripe_key(struct extent_buffer *eb, static void print_raid_stripe_key(struct extent_buffer *eb,
u32 item_size, struct btrfs_stripe_extent *stripe) u32 item_size, struct btrfs_stripe_extent *stripe)
{ {
int num_stripes; int num_stripes = item_size / sizeof(struct btrfs_raid_stride);
u8 encoding = btrfs_stripe_extent_encoding(eb, stripe);
num_stripes = (item_size - offsetof(struct btrfs_stripe_extent, strides)) /
sizeof(struct btrfs_raid_stride);
printf("\t\t\tencoding: %s\n", stripe_encoding_name(encoding));
for (int i = 0; i < num_stripes; i++) for (int i = 0; i < num_stripes; i++)
printf("\t\t\tstripe %d devid %llu physical %llu\n", i, printf("\t\t\tstripe %d devid %llu physical %llu\n", i,
(unsigned long long)btrfs_raid_stride_devid_nr(eb, stripe, i), (unsigned long long)btrfs_raid_stride_devid_nr(eb, stripe, i),

View File

@ -1721,9 +1721,6 @@ static int check_inode_ref(struct extent_buffer *leaf,
static int check_raid_stripe_extent(const struct extent_buffer *leaf, static int check_raid_stripe_extent(const struct extent_buffer *leaf,
const struct btrfs_key *key, int slot) const struct btrfs_key *key, int slot)
{ {
struct btrfs_stripe_extent *stripe_extent =
btrfs_item_ptr(leaf, slot, struct btrfs_stripe_extent);
if (unlikely(!IS_ALIGNED(key->objectid, leaf->fs_info->sectorsize))) { if (unlikely(!IS_ALIGNED(key->objectid, leaf->fs_info->sectorsize))) {
generic_err(leaf, slot, generic_err(leaf, slot,
"invalid key objectid for raid stripe extent, have %llu expect aligned to %u", "invalid key objectid for raid stripe extent, have %llu expect aligned to %u",
@ -1737,22 +1734,6 @@ static int check_raid_stripe_extent(const struct extent_buffer *leaf,
return -EUCLEAN; return -EUCLEAN;
} }
switch (btrfs_stripe_extent_encoding(leaf, stripe_extent)) {
case BTRFS_STRIPE_RAID0:
case BTRFS_STRIPE_RAID1:
case BTRFS_STRIPE_DUP:
case BTRFS_STRIPE_RAID10:
case BTRFS_STRIPE_RAID5:
case BTRFS_STRIPE_RAID6:
case BTRFS_STRIPE_RAID1C3:
case BTRFS_STRIPE_RAID1C4:
break;
default:
generic_err(leaf, slot, "invalid raid stripe encoding %u",
btrfs_stripe_extent_encoding(leaf, stripe_extent));
return -EUCLEAN;
}
return 0; return 0;
} }

View File

@ -712,21 +712,9 @@ struct btrfs_raid_stride {
__le64 offset; __le64 offset;
} __attribute__ ((__packed__)); } __attribute__ ((__packed__));
/* The stripe_extent::encoding, 1:1 mapping of enum btrfs_raid_types */
#define BTRFS_STRIPE_RAID0 1
#define BTRFS_STRIPE_RAID1 2
#define BTRFS_STRIPE_DUP 3
#define BTRFS_STRIPE_RAID10 4
#define BTRFS_STRIPE_RAID5 5
#define BTRFS_STRIPE_RAID6 6
#define BTRFS_STRIPE_RAID1C3 7
#define BTRFS_STRIPE_RAID1C4 8
struct btrfs_stripe_extent { struct btrfs_stripe_extent {
u8 encoding;
u8 reserved[7];
/* Array of raid strides this stripe is comprised of. */ /* Array of raid strides this stripe is comprised of. */
struct btrfs_raid_stride strides; __DECLARE_FLEX_ARRAY(struct btrfs_raid_stride, strides);
} __attribute__ ((__packed__)); } __attribute__ ((__packed__));
#define BTRFS_FREE_SPACE_EXTENT 1 #define BTRFS_FREE_SPACE_EXTENT 1