Messages in this thread | | | Date | Tue, 7 Jul 2009 09:01:20 +0900 | From | Minchan Kim <> | Subject | Re: [PATCH 4/5] add isolate pages vmstat |
| |
> ============ CUT HERE =============== > Subject: [PATCH] add isolate pages vmstat > > If the system have plenty threads or processes, concurrent reclaim can > isolate very much pages. > Unfortunately, current /proc/meminfo and OOM log can't show it. > > This patch provide the way of showing this information. > > > reproduce way > ----------------------- > % ./hackbench 140 process 1000 > => couse OOM > > Active_anon:146 active_file:41 inactive_anon:0 > inactive_file:0 unevictable:0 > isolated_anon:49245 isolated_file:113 > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > dirty:0 writeback:0 buffer:49 unstable:0 > free:184 slab_reclaimable:276 slab_unreclaimable:5492 > mapped:87 pagetables:28239 bounce:0 > > > Signed-off-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com> > --- > drivers/base/node.c | 4 ++++ > fs/proc/meminfo.c | 4 ++++ > include/linux/mmzone.h | 2 ++ > mm/page_alloc.c | 10 ++++++++-- > mm/vmscan.c | 5 +++++ > mm/vmstat.c | 3 ++- > 6 files changed, 25 insertions(+), 3 deletions(-) > > Index: b/fs/proc/meminfo.c > =================================================================== > --- a/fs/proc/meminfo.c > +++ b/fs/proc/meminfo.c > @@ -65,6 +65,8 @@ static int meminfo_proc_show(struct seq_ > "Active(file): %8lu kB\n" > "Inactive(file): %8lu kB\n" > "Unevictable: %8lu kB\n" > + "Isolated(anon): %8lu kB\n" > + "Isolated(file): %8lu kB\n" > "Mlocked: %8lu kB\n" > #ifdef CONFIG_HIGHMEM > "HighTotal: %8lu kB\n" > @@ -109,6 +111,8 @@ static int meminfo_proc_show(struct seq_ > K(pages[LRU_ACTIVE_FILE]), > K(pages[LRU_INACTIVE_FILE]), > K(pages[LRU_UNEVICTABLE]), > + K(global_page_state(NR_ISOLATED_ANON)), > + K(global_page_state(NR_ISOLATED_FILE)), > K(global_page_state(NR_MLOCK)), > #ifdef CONFIG_HIGHMEM > K(i.totalhigh), > Index: b/include/linux/mmzone.h > =================================================================== > --- a/include/linux/mmzone.h > +++ b/include/linux/mmzone.h > @@ -100,6 +100,8 @@ enum zone_stat_item { > NR_BOUNCE, > NR_VMSCAN_WRITE, > NR_WRITEBACK_TEMP, /* Writeback using temporary buffers */ > + NR_ISOLATED_ANON, /* Temporary isolated pages from anon lru */ > + NR_ISOLATED_FILE, /* Temporary isolated pages from file lru */ > #ifdef CONFIG_NUMA > NUMA_HIT, /* allocated in intended node */ > NUMA_MISS, /* allocated in non intended node */ > Index: b/mm/page_alloc.c > =================================================================== > --- a/mm/page_alloc.c > +++ b/mm/page_alloc.c > @@ -2116,8 +2116,8 @@ void show_free_areas(void) > } > > printk("Active_anon:%lu active_file:%lu inactive_anon:%lu\n" > - " inactive_file:%lu" > - " unevictable:%lu" > + " inactive_file:%lu unevictable:%lu\n" > + " isolated_anon:%lu isolated_file:%lu\n" > " dirty:%lu writeback:%lu buffer:%lu unstable:%lu\n" > " free:%lu slab_reclaimable:%lu slab_unreclaimable:%lu\n" > " mapped:%lu pagetables:%lu bounce:%lu\n", > @@ -2126,6 +2126,8 @@ void show_free_areas(void) > global_page_state(NR_INACTIVE_ANON), > global_page_state(NR_INACTIVE_FILE), > global_page_state(NR_UNEVICTABLE), > + global_page_state(NR_ISOLATED_ANON), > + global_page_state(NR_ISOLATED_FILE), > global_page_state(NR_FILE_DIRTY), > global_page_state(NR_WRITEBACK), > nr_blockdev_pages(), > @@ -2151,6 +2153,8 @@ void show_free_areas(void) > " active_file:%lukB" > " inactive_file:%lukB" > " unevictable:%lukB" > + " isolated(anon):%lukB" > + " isolated(file):%lukB" > " present:%lukB" > " mlocked:%lukB" > " dirty:%lukB" > @@ -2176,6 +2180,8 @@ void show_free_areas(void) > K(zone_page_state(zone, NR_ACTIVE_FILE)), > K(zone_page_state(zone, NR_INACTIVE_FILE)), > K(zone_page_state(zone, NR_UNEVICTABLE)), > + K(zone_page_state(zone, NR_ISOLATED_ANON)), > + K(zone_page_state(zone, NR_ISOLATED_FILE)), > K(zone->present_pages), > K(zone_page_state(zone, NR_MLOCK)), > K(zone_page_state(zone, NR_FILE_DIRTY)), > Index: b/mm/vmscan.c > =================================================================== > --- a/mm/vmscan.c > +++ b/mm/vmscan.c > @@ -1082,6 +1082,7 @@ static unsigned long shrink_inactive_lis > -count[LRU_ACTIVE_ANON]); > __mod_zone_page_state(zone, NR_INACTIVE_ANON, > -count[LRU_INACTIVE_ANON]); > + __mod_zone_page_state(zone, NR_ISOLATED_ANON + file, nr_taken);
Lumpy can reclaim file + anon anywhere. How about using count[NR_LRU_LISTS]? -- Kind regards, Minchan Kim
| |