diff options
| author |  Colin Ian King <colin.i.king@gmail.com>
| 2023-06-21 14:09:37 +0100 |
|---|---|---|
| committer |  David Sterba <dsterba@suse.com>
| 2023-08-21 14:52:11 +0200 |
| commit | cf4ac2b9049be459dfbaddbeb9322366f1ac2dc6 (patch) | |
| tree | d2b6a5dc7097368e96ca081461b4c927f64f64f8 /fs/btrfs/scrub.c | |
| parent | btrfs: zoned: use vcalloc instead of for vzalloc in btrfs_get_dev_zone_info (diff) | |
| download | linux-cf4ac2b9049be459dfbaddbeb9322366f1ac2dc6.tar.gz linux-cf4ac2b9049be459dfbaddbeb9322366f1ac2dc6.tar.bz2 linux-cf4ac2b9049be459dfbaddbeb9322366f1ac2dc6.zip | |
btrfs: scrub: remove redundant division of stripe_nr
Variable stripe_nr is being divided by map->num_stripes however the
result is never read. The division and assignment are redundant and
can be removed. Cleans up clang scan build warning:
fs/btrfs/scrub.c:1264:3: warning: Value stored to 'stripe_nr' is
never read [deadcode.DeadStores]
The code is a leftover from 6ded22c1bfe6 ("btrfs: reduce div64 calls by
limiting the number of stripes of a chunk to u32") that converted div64
to normal division, it's the same but previous version did not trigger a
warning.
Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/scrub.c')
| -rw-r--r-- | fs/btrfs/scrub.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/btrfs/scrub.c b/fs/btrfs/scrub.c index 7289f5bff397..fe1c5bc7f601 100644 --- a/fs/btrfs/scrub.c +++ b/fs/btrfs/scrub.c @@ -1262,7 +1262,6 @@ static int get_raid56_logic_offset(u64 physical, int num, /* Work out the disk rotation on this stripe-set */ rot = stripe_nr % map->num_stripes; - stripe_nr /= map->num_stripes; /* calculate which stripe this data locates */ rot += i; stripe_index = rot % map->num_stripes; |