From: Mel Gorman <mgorman@techsingularity.net>
Date: Fri, 20 May 2016 00:13:44 +0000 (-0700)
Subject: mm, page_alloc: remove unnecessary local variable in get_page_from_freelist
X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=4dfa6cd8fdb1682586d463bc34888980fe98eb46;p=GitHub%2FLineageOS%2Fandroid_kernel_motorola_exynos9610.git

mm, page_alloc: remove unnecessary local variable in get_page_from_freelist

zonelist here is a copy of a struct field that is used once.  Ditch it.

Signed-off-by: Mel Gorman <mgorman@techsingularity.net>
Acked-by: Vlastimil Babka <vbabka@suse.cz>
Cc: Jesper Dangaard Brouer <brouer@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
---

diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index 41d20bf66ac8..1d705803fc66 100644
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -2676,7 +2676,6 @@ static struct page *
 get_page_from_freelist(gfp_t gfp_mask, unsigned int order, int alloc_flags,
 						const struct alloc_context *ac)
 {
-	struct zonelist *zonelist = ac->zonelist;
 	struct zoneref *z;
 	struct page *page = NULL;
 	struct zone *zone;
@@ -2690,7 +2689,7 @@ zonelist_scan:
 	 * Scan zonelist, looking for a zone with enough free.
 	 * See also __cpuset_node_allowed() comment in kernel/cpuset.c.
 	 */
-	for_each_zone_zonelist_nodemask(zone, z, zonelist, ac->high_zoneidx,
+	for_each_zone_zonelist_nodemask(zone, z, ac->zonelist, ac->high_zoneidx,
 								ac->nodemask) {
 		unsigned long mark;