diff options
author | David Sterba <[email protected]> | 2020-02-19 15:17:20 +0100 |
---|---|---|
committer | David Sterba <[email protected]> | 2020-03-23 17:01:41 +0100 |
commit | c17af96554a8a8777cbb0fd53b8497250e548b43 (patch) | |
tree | f97e8666fac3ffbe790f92d18fd151354088d78f /tools/perf/scripts/python/net_dropmonitor.py | |
parent | b25b0b871f206936d5bca02b80d38c05623e27da (diff) |
btrfs: raid56: simplify tracking of Q stripe presence
There are temporary variables tracking the index of P and Q stripes, but
none of them is really used as such, merely for determining if the Q
stripe is present. This leads to compiler warnings with
-Wunused-but-set-variable and has been reported several times.
fs/btrfs/raid56.c: In function ‘finish_rmw’:
fs/btrfs/raid56.c:1199:6: warning: variable ‘p_stripe’ set but not used [-Wunused-but-set-variable]
1199 | int p_stripe = -1;
| ^~~~~~~~
fs/btrfs/raid56.c: In function ‘finish_parity_scrub’:
fs/btrfs/raid56.c:2356:6: warning: variable ‘p_stripe’ set but not used [-Wunused-but-set-variable]
2356 | int p_stripe = -1;
| ^~~~~~~~
Replace the two variables with one that has a clear meaning and also get
rid of the warnings. The logic that verifies that there are only 2
valid cases is unchanged.
Reviewed-by: Johannes Thumshirn <[email protected]>
Signed-off-by: David Sterba <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/net_dropmonitor.py')
0 files changed, 0 insertions, 0 deletions