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_scannerd per-zone-killobytes with per-node-count. Signed-off-by: Minchan Kim <[email protected]> --- 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 f80a0e5..7edd311 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), !pgdat_reclaimable(pgdat) ? "yes" : "no"); } @@ -4397,7 +4399,6 @@ void show_free_areas(unsigned int filter) " free_pcp:%lukB" " local_pcp:%ukB" " free_cma:%lukB" - " node_pages_scanned:%lu" "\n", zone->name, K(zone_page_state(zone, NR_FREE_PAGES)), @@ -4415,8 +4416,7 @@ void show_free_areas(unsigned int filter) K(zone_page_state(zone, NR_BOUNCE)), K(free_pcp), K(this_cpu_read(zone->pageset->pcp.count)), - K(zone_page_state(zone, NR_FREE_CMA_PAGES)), - K(node_page_state(zone->zone_pgdat, NR_PAGES_SCANNED))); + K(zone_page_state(zone, NR_FREE_CMA_PAGES))); printk("lowmem_reserve[]:"); for (i = 0; i < MAX_NR_ZONES; i++) printk(" %ld", zone->lowmem_reserve[i]); -- 1.9.1
