Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-02-16 Thread Minchan Kim
On Mon, Feb 14, 2011 at 2:33 AM, Balbir Singh wrote: > * MinChan Kim [2011-02-10 14:41:44]: > >> I don't know why the part of message is deleted only when I send you. >> Maybe it's gmail bug. >> >> I hope mail sending is successful in this turn. :) >> >> On Thu, Feb 10, 2011 at 2:33 PM, Minchan K

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-02-14 Thread Balbir Singh
* MinChan Kim [2011-02-10 14:41:44]: > I don't know why the part of message is deleted only when I send you. > Maybe it's gmail bug. > > I hope mail sending is successful in this turn. :) > > On Thu, Feb 10, 2011 at 2:33 PM, Minchan Kim wrote: > > Sorry for late response. > > > > On Fri, Jan 2

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-02-09 Thread Minchan Kim
I don't know why the part of message is deleted only when I send you. Maybe it's gmail bug. I hope mail sending is successful in this turn. :) On Thu, Feb 10, 2011 at 2:33 PM, Minchan Kim wrote: > Sorry for late response. > > On Fri, Jan 28, 2011 at 8:18 PM, Balbir Singh > wrote: >> * MinChan

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-02-09 Thread Minchan Kim
Sorry for late response. On Fri, Jan 28, 2011 at 8:18 PM, Balbir Singh wrote: > * MinChan Kim [2011-01-28 16:24:19]: > >> > >> > But the assumption for LRU order to change happens only if the page >> > cannot be successfully freed, which means it is in some way active.. >> > and needs to be move

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-01-31 Thread Balbir Singh
* KAMEZAWA Hiroyuki [2011-01-31 08:58:53]: > On Fri, 28 Jan 2011 09:20:02 -0600 (CST) > Christoph Lameter wrote: > > > On Fri, 28 Jan 2011, KAMEZAWA Hiroyuki wrote: > > > > > > > I see it as a tradeoff of when to check? add_to_page_cache or when we > > > > > are want more free memory (due to a

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-01-30 Thread KAMEZAWA Hiroyuki
On Fri, 28 Jan 2011 09:20:02 -0600 (CST) Christoph Lameter wrote: > On Fri, 28 Jan 2011, KAMEZAWA Hiroyuki wrote: > > > > > I see it as a tradeoff of when to check? add_to_page_cache or when we > > > > are want more free memory (due to allocation). It is OK to wakeup > > > > kswapd while allocat

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-01-28 Thread Christoph Lameter
On Fri, 28 Jan 2011, KAMEZAWA Hiroyuki wrote: > > > I see it as a tradeoff of when to check? add_to_page_cache or when we > > > are want more free memory (due to allocation). It is OK to wakeup > > > kswapd while allocating memory, somehow for this purpose (global page > > > cache), add_to_page_ca

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-01-28 Thread Balbir Singh
* KAMEZAWA Hiroyuki [2011-01-28 17:17:44]: > On Fri, 28 Jan 2011 13:49:28 +0530 > Balbir Singh wrote: > > > * KAMEZAWA Hiroyuki [2011-01-28 16:56:05]: > > > > BTW, it seems this doesn't work when some apps use huge shmem. > > > How to handle the issue ? > > > > > > > Could you elaborate furt

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-01-28 Thread Balbir Singh
* MinChan Kim [2011-01-28 16:24:19]: > > > > But the assumption for LRU order to change happens only if the page > > cannot be successfully freed, which means it is in some way active.. > > and needs to be moved no? > > 1. holded page by someone > 2. mapped pages > 3. active pages > > 1 is rare

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-01-28 Thread KAMEZAWA Hiroyuki
On Fri, 28 Jan 2011 13:49:28 +0530 Balbir Singh wrote: > * KAMEZAWA Hiroyuki [2011-01-28 16:56:05]: > > BTW, it seems this doesn't work when some apps use huge shmem. > > How to handle the issue ? > > > > Could you elaborate further? > == static inline unsigned long zone_unmapped_file_pages

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-01-28 Thread Balbir Singh
* KAMEZAWA Hiroyuki [2011-01-28 16:56:05]: > On Fri, 28 Jan 2011 16:24:19 +0900 > Minchan Kim wrote: > > > On Fri, Jan 28, 2011 at 3:48 PM, Balbir Singh > > wrote: > > > * MinChan Kim [2011-01-28 14:44:50]: > > > > > >> On Fri, Jan 28, 2011 at 11:56 AM, Balbir Singh > > >> wrote: > > >> > O

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-01-28 Thread KAMEZAWA Hiroyuki
On Fri, 28 Jan 2011 16:24:19 +0900 Minchan Kim wrote: > On Fri, Jan 28, 2011 at 3:48 PM, Balbir Singh > wrote: > > * MinChan Kim [2011-01-28 14:44:50]: > > > >> On Fri, Jan 28, 2011 at 11:56 AM, Balbir Singh > >> wrote: > >> > On Thu, Jan 27, 2011 at 4:42 AM, Minchan Kim > >> > wrote: > >>

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-01-27 Thread Minchan Kim
On Fri, Jan 28, 2011 at 3:48 PM, Balbir Singh wrote: > * MinChan Kim [2011-01-28 14:44:50]: > >> On Fri, Jan 28, 2011 at 11:56 AM, Balbir Singh >> wrote: >> > On Thu, Jan 27, 2011 at 4:42 AM, Minchan Kim wrote: >> > [snip] >> > >> >>> index 7b56473..2ac8549 100644 >> >>> --- a/mm/page_alloc.c >

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-01-27 Thread Balbir Singh
* MinChan Kim [2011-01-28 14:44:50]: > On Fri, Jan 28, 2011 at 11:56 AM, Balbir Singh > wrote: > > On Thu, Jan 27, 2011 at 4:42 AM, Minchan Kim wrote: > > [snip] > > > >>> index 7b56473..2ac8549 100644 > >>> --- a/mm/page_alloc.c > >>> +++ b/mm/page_alloc.c > >>> @@ -1660,6 +1660,9 @@ zonelist_

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-01-27 Thread Balbir Singh
* Christoph Lameter [2011-01-26 10:57:37]: > > Reviewed-by: Christoph Lameter > Thanks for the review! -- Three Cheers, Balbir -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-01-27 Thread Minchan Kim
On Fri, Jan 28, 2011 at 11:56 AM, Balbir Singh wrote: > On Thu, Jan 27, 2011 at 4:42 AM, Minchan Kim wrote: > [snip] > >>> index 7b56473..2ac8549 100644 >>> --- a/mm/page_alloc.c >>> +++ b/mm/page_alloc.c >>> @@ -1660,6 +1660,9 @@ zonelist_scan: >>>                        unsigned long mark; >>>

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-01-27 Thread Balbir Singh
On Thu, Jan 27, 2011 at 4:42 AM, Minchan Kim wrote: [snip] >> index 7b56473..2ac8549 100644 >> --- a/mm/page_alloc.c >> +++ b/mm/page_alloc.c >> @@ -1660,6 +1660,9 @@ zonelist_scan: >>                        unsigned long mark; >>                        int ret; >> >> +                       if (

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-01-26 Thread Minchan Kim
Hi Balbir, On Tue, Jan 25, 2011 at 2:10 PM, Balbir Singh wrote: > Changelog v4 > 1. Add max_unmapped_ratio and use that as the upper limit > to check when to shrink the unmapped page cache (Christoph > Lameter) > > Changelog v2 > 1. Use a config option to enable the code (Andrew Morton) > 2. Expl

Re: [PATCH 3/3] Provide control over unmapped pages (v4)

2011-01-26 Thread Christoph Lameter
Reviewed-by: Christoph Lameter -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html