f2fs: drop useless initializer and unneeded local variable
authorYangtao Li <frank.li@vivo.com>
Thu, 22 Dec 2022 08:18:55 +0000 (16:18 +0800)
committerJaegeuk Kim <jaegeuk@kernel.org>
Mon, 30 Jan 2023 21:23:51 +0000 (13:23 -0800)
No need to initialize idx twice. BTW, remove the unnecessary cnt variable.

Signed-off-by: Yangtao Li <frank.li@vivo.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/iostat.c

index 91b384b..ed81769 100644 (file)
@@ -87,8 +87,7 @@ int __maybe_unused iostat_info_seq_show(struct seq_file *seq, void *offset)
 
 static inline void __record_iostat_latency(struct f2fs_sb_info *sbi)
 {
-       int io, idx = 0;
-       unsigned int cnt;
+       int io, idx;
        struct f2fs_iostat_latency iostat_lat[MAX_IO_TYPE][NR_PAGE_TYPE];
        struct iostat_lat_info *io_lat = sbi->iostat_io_lat;
        unsigned long flags;
@@ -96,12 +95,11 @@ static inline void __record_iostat_latency(struct f2fs_sb_info *sbi)
        spin_lock_irqsave(&sbi->iostat_lat_lock, flags);
        for (idx = 0; idx < MAX_IO_TYPE; idx++) {
                for (io = 0; io < NR_PAGE_TYPE; io++) {
-                       cnt = io_lat->bio_cnt[idx][io];
                        iostat_lat[idx][io].peak_lat =
                           jiffies_to_msecs(io_lat->peak_lat[idx][io]);
-                       iostat_lat[idx][io].cnt = cnt;
-                       iostat_lat[idx][io].avg_lat = cnt ?
-                          jiffies_to_msecs(io_lat->sum_lat[idx][io]) / cnt : 0;
+                       iostat_lat[idx][io].cnt = io_lat->bio_cnt[idx][io];
+                       iostat_lat[idx][io].avg_lat = iostat_lat[idx][io].cnt ?
+                          jiffies_to_msecs(io_lat->sum_lat[idx][io]) / iostat_lat[idx][io].cnt : 0;
                        io_lat->sum_lat[idx][io] = 0;
                        io_lat->peak_lat[idx][io] = 0;
                        io_lat->bio_cnt[idx][io] = 0;