Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758503AbcDACK3 (ORCPT ); Thu, 31 Mar 2016 22:10:29 -0400 Received: from mail-pa0-f51.google.com ([209.85.220.51]:36136 "EHLO mail-pa0-f51.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753707AbcDACK1 (ORCPT ); Thu, 31 Mar 2016 22:10:27 -0400 From: js1304@gmail.com X-Google-Original-From: iamjoonsoo.kim@lge.com To: Andrew Morton Cc: Rik van Riel , Johannes Weiner , Mel Gorman , Laura Abbott , Minchan Kim , Marek Szyprowski , Michal Nazarewicz , "Aneesh Kumar K.V" , Vlastimil Babka , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Joonsoo Kim Subject: [PATCH 3/4] mm/highmem: make nr_free_highpages() handles all highmem zones by itself Date: Fri, 1 Apr 2016 11:10:09 +0900 Message-Id: <1459476610-31076-3-git-send-email-iamjoonsoo.kim@lge.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1459476610-31076-1-git-send-email-iamjoonsoo.kim@lge.com> References: <1459476610-31076-1-git-send-email-iamjoonsoo.kim@lge.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1095 Lines: 39 From: Joonsoo Kim nr_free_highpages() manually add statistics per each highmem zone and return total value for them. Whenever we add a new highmem zone, we need to consider this function and it's really troublesome. Make it handles all highmem zones by itself. Signed-off-by: Joonsoo Kim --- mm/highmem.c | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/mm/highmem.c b/mm/highmem.c index 123bcd3..50b4ca6 100644 --- a/mm/highmem.c +++ b/mm/highmem.c @@ -112,16 +112,12 @@ EXPORT_PER_CPU_SYMBOL(__kmap_atomic_idx); unsigned int nr_free_highpages (void) { - pg_data_t *pgdat; + struct zone *zone; unsigned int pages = 0; - for_each_online_pgdat(pgdat) { - pages += zone_page_state(&pgdat->node_zones[ZONE_HIGHMEM], - NR_FREE_PAGES); - if (zone_movable_is_highmem()) - pages += zone_page_state( - &pgdat->node_zones[ZONE_MOVABLE], - NR_FREE_PAGES); + for_each_populated_zone(zone) { + if (is_highmem(zone)) + pages += zone_page_state(zone, NR_FREE_PAGES); } return pages; -- 1.9.1