btrfs: remove unused btrfs_io_stripe::length
authorDavid Sterba <dsterba@suse.com>
Tue, 22 Apr 2025 15:32:17 +0000 (17:32 +0200)
committerDavid Sterba <dsterba@suse.com>
Thu, 15 May 2025 12:30:47 +0000 (14:30 +0200)
First added (but not effectively used) in 02c372e1f016e5 ("btrfs: add
support for inserting raid stripe extents"). The structure is
initialized to zeros so the only use in btrfs_insert_one_raid_extent()

    u64 length = bioc->stripes[i].length;
    struct btrfs_raid_stride *raid_stride = &stripe_extent->strides[i];

    if (length == 0)
            length = bioc->size;

the 'if' always happens.

Last use in 4016358e852861 ("btrfs: remove unused variable length in
btrfs_insert_one_raid_extent()") was an obvious cleanup. It seems to be
safe to remove, raid-stripe-tree works without using it since 6.6.

This was found by tool https://github.com/jirislaby/clang-struct .

Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/volumes.h

index e247d551da672b982d4d34065f71f6ff3e5c8237..72b8122938eb7555325aca008663b5c89d1ceaad 100644 (file)
@@ -473,7 +473,6 @@ struct btrfs_io_stripe {
        struct btrfs_device *dev;
        /* Block mapping. */
        u64 physical;
-       u64 length;
        bool rst_search_commit_root;
        /* For the endio handler. */
        struct btrfs_io_context *bioc;