aboutsummaryrefslogtreecommitdiffstats
path: root/tools/perf/scripts/python/stackcollapse.py
diff options
context:
space:
mode:
authorDavid Sterba <dsterba@suse.com>2020-02-19 15:17:20 +0100
committerDavid Sterba <dsterba@suse.com>2020-03-23 17:01:41 +0100
commitc17af96554a8a8777cbb0fd53b8497250e548b43 (patch)
treef97e8666fac3ffbe790f92d18fd151354088d78f /tools/perf/scripts/python/stackcollapse.py
parentbtrfs: backref, use correct count to resolve normal data refs (diff)
downloadlinux-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/stackcollapse.py')
0 files changed, 0 insertions, 0 deletions