Re: Isolated(anon) and Isolated(file)

[Hugh Dickins <hugh.dickins@tiscali.co.uk>]

On Tue, 15 Sep 2009, KOSAKI Motohiro wrote:
> From 7aa6fa2b76ff5d063b8bfa4a3af38c39b9396fd5 Mon Sep 17 00:00:00 2001
> From: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
> Date: Tue, 15 Sep 2009 10:16:51 +0900
> Subject: [PATCH] Kill Isolated field in /proc/meminfo
> 
> Hugh Dickins pointed out Isolated field dislpay 0kB at almost time.
> It is only increased at heavy memory pressure case.
> 
> So, if the system haven't get memory pressure, this field isn't useful.
> And now, we have two alternative way, /sys/device/system/node/node{n}/meminfo
> and /prov/vmstat. Then, it can be removed.
> 
> Reported-by: Hugh Dickins <hugh.dickins@tiscali.co.uk>
> Signed-off-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>

Acked-by: Hugh Dickins <hugh.dickins@tiscali.co.uk>

I should be overjoyed that you agree to hide the Isolateds from my sight:
thank you.  But in fact I'm a little depressed, now you've reminded me of
almost-the-same-but-annoyingly-different /sys/devices/unmemorable/meminfo.

Oh well, since I never see it (I'd need some nodes), I guess I don't
even need to turn a blind eye to it; and it already contains other
stuff I objected to in /proc/meminfo.

I still think your Isolateds make most sense in the OOM display;
and yes, they are there in /proc/vmstat, that's good too.

> ---
>  fs/proc/meminfo.c |    4 ----
>  1 files changed, 0 insertions(+), 4 deletions(-)
> 
> diff --git a/fs/proc/meminfo.c b/fs/proc/meminfo.c
> index 7d46c2e..c7bff4f 100644
> --- a/fs/proc/meminfo.c
> +++ b/fs/proc/meminfo.c
> @@ -65,8 +65,6 @@ static int meminfo_proc_show(struct seq_file *m, void *v)
>  		"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"
> @@ -116,8 +114,6 @@ static int meminfo_proc_show(struct seq_file *m, void *v)
>  		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),
> -- 
> 1.6.2.5

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>