| Message ID | 20250417092422.1333620-1-hao.ge@linux.dev (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | mm/vmscan: Modify the assignment logic of the scan and total_scan variables | expand |
On 17/04/25 2:54 pm, Hao Ge wrote: > From: Hao Ge <gehao@kylinos.cn> > > The scan and total_scan variables can be initialized to 0 > when they are defined, replacing the separate assignment statements. > > Signed-off-by: Hao Ge <gehao@kylinos.cn> Acked-by: Dev Jain <dev.jain@arm.com> > --- > mm/vmscan.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/mm/vmscan.c b/mm/vmscan.c > index b620d74b0f66..bf360ac4f1cf 100644 > --- a/mm/vmscan.c > +++ b/mm/vmscan.c > @@ -1725,13 +1725,11 @@ static unsigned long isolate_lru_folios(unsigned long nr_to_scan, > 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;
diff --git a/mm/vmscan.c b/mm/vmscan.c index b620d74b0f66..bf360ac4f1cf 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -1725,13 +1725,11 @@ static unsigned long isolate_lru_folios(unsigned long nr_to_scan, 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;