From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751560AbcGPKOa (ORCPT ); Sat, 16 Jul 2016 06:14:30 -0400 Received: from LGEAMRELO12.lge.com ([156.147.23.52]:55503 "EHLO lgeamrelo12.lge.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751423AbcGPKO3 (ORCPT ); Sat, 16 Jul 2016 06:14:29 -0400 X-Original-SENDERIP: 156.147.1.126 X-Original-MAILFROM: minchan@kernel.org X-Original-SENDERIP: 10.177.223.161 X-Original-MAILFROM: minchan@kernel.org Date: Sat, 16 Jul 2016 19:14:31 +0900 From: Minchan Kim To: Mel Gorman , Andrew Morton Cc: Andrew Morton , Johannes Weiner , Vlastimil Babka , Linux-MM , LKML Subject: Re: [PATCH 4/5] mm: show node_pages_scanned per node, not zone Message-ID: <20160716101431.GA10305@bbox> References: <1468588165-12461-1-git-send-email-mgorman@techsingularity.net> <1468588165-12461-5-git-send-email-mgorman@techsingularity.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1468588165-12461-5-git-send-email-mgorman@techsingularity.net> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jul 15, 2016 at 02:09:24PM +0100, Mel Gorman wrote: > From: Minchan Kim > > The node_pages_scanned represents the number of scanned pages > of node for reclaim so it's pointless to show it as kilobytes. > > As well, node_pages_scanned is per-node value, not per-zone. > > This patch changes node_pages_scanned per-zone-killobytes > with per-node-count. > > Signed-off-by: Minchan Kim > Signed-off-by: Mel Gorman > --- > mm/page_alloc.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/mm/page_alloc.c b/mm/page_alloc.c > index f80a0e57dcc8..7edd311a63f1 100644 > --- a/mm/page_alloc.c > +++ b/mm/page_alloc.c > @@ -4345,6 +4345,7 @@ void show_free_areas(unsigned int filter) > #endif > " writeback_tmp:%lukB" > " unstable:%lukB" > + " pages_scanned:%lu" > " all_unreclaimable? %s" > "\n", > pgdat->node_id, > @@ -4367,6 +4368,7 @@ void show_free_areas(unsigned int filter) > K(node_page_state(pgdat, NR_SHMEM)), > K(node_page_state(pgdat, NR_WRITEBACK_TEMP)), > K(node_page_state(pgdat, NR_UNSTABLE_NFS)), > + node_page_state(zone->zone_pgdat, NR_PAGES_SCANNED), Oops, It should be pgdat, not zone->zone_pgdat. Andrew, please fold it. >>From 0b058f64335764b82439a3c24fad8573cc1474d7 Mon Sep 17 00:00:00 2001 From: Minchan Kim Date: Sat, 16 Jul 2016 19:07:51 +0900 Subject: [PATCH] mm: fix node_pages_scanned Use pgdat for node stat instead of zone->pgdat. Signed-off-by: Minchan Kim --- mm/page_alloc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/page_alloc.c b/mm/page_alloc.c index 7edd311..7547b6b 100644 --- a/mm/page_alloc.c +++ b/mm/page_alloc.c @@ -4368,7 +4368,7 @@ void show_free_areas(unsigned int filter) K(node_page_state(pgdat, NR_SHMEM)), K(node_page_state(pgdat, NR_WRITEBACK_TEMP)), K(node_page_state(pgdat, NR_UNSTABLE_NFS)), - node_page_state(zone->zone_pgdat, NR_PAGES_SCANNED), + node_page_state(pgdat, NR_PAGES_SCANNED), !pgdat_reclaimable(pgdat) ? "yes" : "no"); } -- 1.9.1