lkml.org 
[lkml]   [2009]   [Jul]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [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


    \
     
     \ /
      Last update: 2009-07-07 02:09    [W:0.030 / U:0.212 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site