From: KOSAKI Motohiro <kosaki.motohiro@xxxxxxxxxxxxxx> commit 2244b95a7b (zoned vm counters: basic ZVC (zoned vm counter) implementation) broke protection column. It is a part of "pages" attribute. but not it is showed after vmstats column. This patch restores the right position. <before> pages free 3965 min 32 low 40 high 48 scanned 0 spanned 4080 present 3909 (snip) numa_local 1 numa_other 0 nr_anon_transparent_hugepages 0 protection: (0, 3512, 7867, 7867) <after> pages free 3965 min 32 low 40 high 48 scanned 0 spanned 4080 present 3909 protection: (0, 3504, 7851, 7851) nr_free_pages 3965 nr_inactive_anon 0 Cc: Christoph Lameter <cl@xxxxxxxxx> Signed-off-by: KOSAKI Motohiro <kosaki.motohiro@xxxxxxxxxxxxxx> --- mm/vmstat.c | 15 +++++++-------- 1 files changed, 7 insertions(+), 8 deletions(-) diff --git a/mm/vmstat.c b/mm/vmstat.c index 1bbbbd9..9f5f2a9 100644 --- a/mm/vmstat.c +++ b/mm/vmstat.c @@ -987,19 +987,18 @@ static void zoneinfo_show_print(struct seq_file *m, pg_data_t *pgdat, zone->pages_scanned, zone->spanned_pages, zone->present_pages); - - for (i = 0; i < NR_VM_ZONE_STAT_ITEMS; i++) - seq_printf(m, "\n %-12s %lu", vmstat_text[i], - zone_page_state(zone, i)); - seq_printf(m, "\n protection: (%lu", zone->lowmem_reserve[0]); for (i = 1; i < ARRAY_SIZE(zone->lowmem_reserve); i++) seq_printf(m, ", %lu", zone->lowmem_reserve[i]); - seq_printf(m, - ")" - "\n pagesets"); + seq_printf(m, ")"); + + for (i = 0; i < NR_VM_ZONE_STAT_ITEMS; i++) + seq_printf(m, "\n %-12s %lu", vmstat_text[i], + zone_page_state(zone, i)); + + seq_printf(m, "\n pagesets"); for_each_online_cpu(i) { struct per_cpu_pageset *pageset; -- 1.7.1 -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>