Re: [PATCH] RFC: vmscan: add min_filelist_kbytes sysctl for protecting the working set

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Minchan Kim
Date: Wednesday, November 3, 2010 - 4:49 pm

Hello.

On Thu, Nov 4, 2010 at 7:40 AM, Mandeep Singh Baines <msb@chromium.org> wrote:

vmscan already have used HZ/10 to calm down congestion of writeback or
something.
(But I don't know why VM used the value and who determined it by any
rationale. It might be a value determined by some experiments.)
If there isn't any good math, we will depend on experiment in this time, too.

Anyway If interval is long, It could make inactive list's size very
shortly in many reclaim workload and then unnecessary OOM kill.
So I hope if inactive list size is very small compared to active list
size, quit the check and refiill the inactive list.

Anyway, the approach makes sense to me.
But need other guy's opinion.

Nitpick :
I expect you will include description of knob in
Documentation/sysctl/vm.txt in your formal patch.

-- 
Kind regards,
Minchan Kim
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] RFC: vmscan: add min_filelist_kbytes sysctl for pr ..., Mandeep Singh Baines, (Thu Oct 28, 12:15 pm)
Re: [PATCH] RFC: vmscan: add min_filelist_kbytes sysctl fo ..., Mandeep Singh Baines, (Thu Oct 28, 3:03 pm)
Re: [PATCH] RFC: vmscan: add min_filelist_kbytes sysctl fo ..., Mandeep Singh Baines, (Thu Oct 28, 3:13 pm)
Re: [PATCH] RFC: vmscan: add min_filelist_kbytes sysctl fo ..., KAMEZAWA Hiroyuki, (Thu Oct 28, 5:04 pm)
Re: [PATCH] RFC: vmscan: add min_filelist_kbytes sysctl fo ..., Mandeep Singh Baines, (Mon Nov 1, 11:24 am)
Re: [PATCH] RFC: vmscan: add min_filelist_kbytes sysctl fo ..., Mandeep Singh Baines, (Mon Nov 1, 12:43 pm)
Re: [PATCH] RFC: vmscan: add min_filelist_kbytes sysctl fo ..., Mandeep Singh Baines, (Wed Nov 3, 3:40 pm)
Re: [PATCH] RFC: vmscan: add min_filelist_kbytes sysctl fo ..., Minchan Kim, (Wed Nov 3, 4:49 pm)
Re: [PATCH] RFC: vmscan: add min_filelist_kbytes sysctl fo ..., Mandeep Singh Baines, (Wed Nov 3, 6:52 pm)
Re: [PATCH] RFC: vmscan: add min_filelist_kbytes sysctl fo ..., Mandeep Singh Baines, (Mon Nov 8, 2:55 pm)