diff options
author | 2020-02-19 15:17:20 +0100 | |
---|---|---|
committer | 2020-03-23 17:01:41 +0100 | |
commit | c17af96554a8a8777cbb0fd53b8497250e548b43 (patch) | |
tree | f97e8666fac3ffbe790f92d18fd151354088d78f /tools/perf/scripts/python/exported-sql-viewer.py | |
parent | btrfs: backref, use correct count to resolve normal data refs (diff) | |
download | linux-dev-c17af96554a8a8777cbb0fd53b8497250e548b43.tar.xz linux-dev-c17af96554a8a8777cbb0fd53b8497250e548b43.zip |
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 <johannes.thumshirn@wdc.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'tools/perf/scripts/python/exported-sql-viewer.py')
0 files changed, 0 insertions, 0 deletions