| From: Hao Ge <gehao@kylinos.cn> |
| Subject: mm/vmscan: modify the assignment logic of the scan and total_scan variables |
| Date: Thu, 17 Apr 2025 17:24:22 +0800 |
| |
| The scan and total_scan variables can be initialized to 0 when they are |
| defined, replacing the separate assignment statements. |
| |
| Link: https://lkml.kernel.org/r/20250417092422.1333620-1-hao.ge@linux.dev |
| Signed-off-by: Hao Ge <gehao@kylinos.cn> |
| Acked-by: Dev Jain <dev.jain@arm.com> |
| Signed-off-by: Andrew Morton <akpm@linux-foundation.org> |
| --- |
| |
| mm/vmscan.c | 6 ++---- |
| 1 file changed, 2 insertions(+), 4 deletions(-) |
| |
| --- a/mm/vmscan.c~mm-vmscan-modify-the-assignment-logic-of-the-scan-and-total_scan-variables |
| +++ a/mm/vmscan.c |
| @@ -1725,13 +1725,11 @@ static unsigned long isolate_lru_folios( |
| unsigned long nr_taken = 0; |
| unsigned long nr_zone_taken[MAX_NR_ZONES] = { 0 }; |
| unsigned long nr_skipped[MAX_NR_ZONES] = { 0, }; |
| - unsigned long skipped = 0; |
| - unsigned long scan, total_scan, nr_pages; |
| + unsigned long skipped = 0, total_scan = 0, scan = 0; |
| + unsigned long nr_pages; |
| unsigned long max_nr_skipped = 0; |
| LIST_HEAD(folios_skipped); |
| |
| - total_scan = 0; |
| - scan = 0; |
| while (scan < nr_to_scan && !list_empty(src)) { |
| struct list_head *move_to = src; |
| struct folio *folio; |
| _ |