mm, page_alloc: convert nr_fair_skipped to bool
authorMel Gorman <mgorman@techsingularity.net>
Fri, 20 May 2016 00:13:41 +0000 (17:13 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 20 May 2016 02:12:14 +0000 (19:12 -0700)
The number of zones skipped to a zone expiring its fair zone allocation
quota is irrelevant.  Convert to bool.

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>
mm/page_alloc.c

index 094587a4ed813df4063e6dd802020dc1ba83531b..41d20bf66ac88dbd392acc0b810756ae7e92e56b 100644 (file)
@@ -2680,7 +2680,7 @@ get_page_from_freelist(gfp_t gfp_mask, unsigned int order, int alloc_flags,
        struct zoneref *z;
        struct page *page = NULL;
        struct zone *zone;
-       int nr_fair_skipped = 0;
+       bool fair_skipped;
        bool zonelist_rescan;
 
 zonelist_scan:
@@ -2708,7 +2708,7 @@ zonelist_scan:
                        if (!zone_local(ac->preferred_zone, zone))
                                break;
                        if (test_bit(ZONE_FAIR_DEPLETED, &zone->flags)) {
-                               nr_fair_skipped++;
+                               fair_skipped = true;
                                continue;
                        }
                }
@@ -2801,7 +2801,7 @@ try_this_zone:
         */
        if (alloc_flags & ALLOC_FAIR) {
                alloc_flags &= ~ALLOC_FAIR;
-               if (nr_fair_skipped) {
+               if (fair_skipped) {
                        zonelist_rescan = true;
                        reset_alloc_batches(ac->preferred_zone);
                }