From: duguowei <duguo...@xiaomi.com> Remove the local variable 'run_no', add comments for better reading.
Signed-off-by: duguowei <duguo...@xiaomi.com> --- fs/f2fs/shrinker.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/fs/f2fs/shrinker.c b/fs/f2fs/shrinker.c index dd3c3c7a90ec..7e439496dfca 100644 --- a/fs/f2fs/shrinker.c +++ b/fs/f2fs/shrinker.c @@ -76,18 +76,18 @@ unsigned long f2fs_shrink_scan(struct shrinker *shrink, unsigned long nr = sc->nr_to_scan; struct f2fs_sb_info *sbi; struct list_head *p; - unsigned int run_no; unsigned long freed = 0; spin_lock(&f2fs_list_lock); - do { - run_no = ++shrinker_run_no; - } while (run_no == 0); + shrinker_run_no++; + /* if overflow, next loop needs to skip number 0 */ + if (!shrinker_run_no) + shrinker_run_no = 1; p = f2fs_list.next; while (p != &f2fs_list) { sbi = list_entry(p, struct f2fs_sb_info, s_list); - if (sbi->shrinker_run_no == run_no) + if (sbi->shrinker_run_no == shrinker_run_no) break; /* stop f2fs_put_super */ @@ -97,7 +97,7 @@ unsigned long f2fs_shrink_scan(struct shrinker *shrink, } spin_unlock(&f2fs_list_lock); - sbi->shrinker_run_no = run_no; + sbi->shrinker_run_no = shrinker_run_no; /* shrink extent cache entries */ freed += f2fs_shrink_extent_tree(sbi, nr >> 1); -- 2.36.1 _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel