| Message ID | 1581568298-45317-1-git-send-email-yang.shi@linux.alibaba.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | mm: vmscan: replace open codings to NUMA_NO_NODE | expand |
On Thu, Feb 13, 2020 at 12:31:38PM +0800, Yang Shi wrote: > The commit 98fa15f34cb3 ("mm: replace all open encodings for > NUMA_NO_NODE") did the replacement across the kernel tree, but we got > some more in vmscan.c since then. > > Cc: Minchan Kim <minchan@kernel.org> > Cc: Anshuman Khandual <anshuman.khandual@arm.com> > Signed-off-by: Yang Shi <yang.shi@linux.alibaba.com> Acked-by: Minchan Kim <minchan@kernel.org> Thanks, Yang!
On Thu, 13 Feb 2020, Yang Shi wrote: > The commit 98fa15f34cb3 ("mm: replace all open encodings for > NUMA_NO_NODE") did the replacement across the kernel tree, but we got > some more in vmscan.c since then. > > Cc: Minchan Kim <minchan@kernel.org> > Cc: Anshuman Khandual <anshuman.khandual@arm.com> > Signed-off-by: Yang Shi <yang.shi@linux.alibaba.com> Acked-by: David Rientjes <rientjes@google.com>
On 02/13/2020 10:01 AM, Yang Shi wrote: > The commit 98fa15f34cb3 ("mm: replace all open encodings for > NUMA_NO_NODE") did the replacement across the kernel tree, but we got > some more in vmscan.c since then. > > Cc: Minchan Kim <minchan@kernel.org> > Cc: Anshuman Khandual <anshuman.khandual@arm.com> > Signed-off-by: Yang Shi <yang.shi@linux.alibaba.com> Reviewed-by: Anshuman Khandual <anshuman.khandual@arm.com>
diff --git a/mm/vmscan.c b/mm/vmscan.c index c05eb9e..567864a 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -2096,7 +2096,7 @@ static void shrink_active_list(unsigned long nr_to_scan, unsigned long reclaim_pages(struct list_head *page_list) { - int nid = -1; + int nid = NUMA_NO_NODE; unsigned long nr_reclaimed = 0; LIST_HEAD(node_page_list); struct reclaim_stat dummy_stat; @@ -2111,7 +2111,7 @@ unsigned long reclaim_pages(struct list_head *page_list) while (!list_empty(page_list)) { page = lru_to_page(page_list); - if (nid == -1) { + if (nid == NUMA_NO_NODE) { nid = page_to_nid(page); INIT_LIST_HEAD(&node_page_list); } @@ -2132,7 +2132,7 @@ unsigned long reclaim_pages(struct list_head *page_list) putback_lru_page(page); } - nid = -1; + nid = NUMA_NO_NODE; } if (!list_empty(&node_page_list)) {
The commit 98fa15f34cb3 ("mm: replace all open encodings for NUMA_NO_NODE") did the replacement across the kernel tree, but we got some more in vmscan.c since then. Cc: Minchan Kim <minchan@kernel.org> Cc: Anshuman Khandual <anshuman.khandual@arm.com> Signed-off-by: Yang Shi <yang.shi@linux.alibaba.com> --- mm/vmscan.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)