Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2015-02-15 Thread Minchan Kim
Hi Shaohua, On Wed, Feb 11, 2015 at 04:14:03PM -0800, Shaohua Li wrote: > On Wed, Feb 11, 2015 at 09:56:20AM +0900, Minchan Kim wrote: > > Hi Shaohua, > > > > On Tue, Feb 10, 2015 at 02:38:26PM -0800, Shaohua Li wrote: > > > On Mon, Feb 09, 2015 at 04:15:53PM +0900, Minchan Kim wrote: > > > > On

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2015-02-11 Thread Shaohua Li
On Wed, Feb 11, 2015 at 09:56:20AM +0900, Minchan Kim wrote: > Hi Shaohua, > > On Tue, Feb 10, 2015 at 02:38:26PM -0800, Shaohua Li wrote: > > On Mon, Feb 09, 2015 at 04:15:53PM +0900, Minchan Kim wrote: > > > On Fri, Feb 06, 2015 at 10:29:18AM -0800, Shaohua Li wrote: > > > > On Fri, Feb 06, 2015

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2015-02-10 Thread Minchan Kim
Hi Shaohua, On Tue, Feb 10, 2015 at 02:38:26PM -0800, Shaohua Li wrote: > On Mon, Feb 09, 2015 at 04:15:53PM +0900, Minchan Kim wrote: > > On Fri, Feb 06, 2015 at 10:29:18AM -0800, Shaohua Li wrote: > > > On Fri, Feb 06, 2015 at 02:51:03PM +0900, Minchan Kim wrote: > > > > Hi Shaohua, > > > > > >

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2015-02-10 Thread Shaohua Li
On Mon, Feb 09, 2015 at 04:15:53PM +0900, Minchan Kim wrote: > On Fri, Feb 06, 2015 at 10:29:18AM -0800, Shaohua Li wrote: > > On Fri, Feb 06, 2015 at 02:51:03PM +0900, Minchan Kim wrote: > > > Hi Shaohua, > > > > > > On Thu, Feb 05, 2015 at 04:33:11PM -0800, Shaohua Li wrote: > > > > > > > > Hi

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2015-02-08 Thread Minchan Kim
On Fri, Feb 06, 2015 at 10:29:18AM -0800, Shaohua Li wrote: > On Fri, Feb 06, 2015 at 02:51:03PM +0900, Minchan Kim wrote: > > Hi Shaohua, > > > > On Thu, Feb 05, 2015 at 04:33:11PM -0800, Shaohua Li wrote: > > > > > > Hi Minchan, > > > > > > Sorry to jump in this thread so later, and if some is

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2015-02-06 Thread Rik van Riel
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/06/2015 01:32 PM, Shaohua Li wrote: > On Fri, Feb 06, 2015 at 01:58:25PM +0100, Michal Hocko wrote: >> On Thu 05-02-15 16:33:11, Shaohua Li wrote: [...] >>> Did you think about move the MADV_FREE pages to the head of >>> inactive LRU, so they can

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2015-02-06 Thread Shaohua Li
On Fri, Feb 06, 2015 at 01:58:25PM +0100, Michal Hocko wrote: > On Thu 05-02-15 16:33:11, Shaohua Li wrote: > [...] > > Did you think about move the MADV_FREE pages to the head of inactive LRU, so > > they can be reclaimed easily? > > Yes this makes sense for pages living on the active LRU list. I

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2015-02-06 Thread Shaohua Li
On Fri, Feb 06, 2015 at 02:51:03PM +0900, Minchan Kim wrote: > Hi Shaohua, > > On Thu, Feb 05, 2015 at 04:33:11PM -0800, Shaohua Li wrote: > > > > Hi Minchan, > > > > Sorry to jump in this thread so later, and if some issues are discussed > > before. > > I'm interesting in this patch, so tried

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2015-02-06 Thread Michal Hocko
On Thu 05-02-15 16:33:11, Shaohua Li wrote: [...] > Did you think about move the MADV_FREE pages to the head of inactive LRU, so > they can be reclaimed easily? Yes this makes sense for pages living on the active LRU list. I would preserve LRU ordering on the inactive list because there is no good

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2015-02-05 Thread Minchan Kim
Hi Shaohua, On Thu, Feb 05, 2015 at 04:33:11PM -0800, Shaohua Li wrote: > > Hi Minchan, > > Sorry to jump in this thread so later, and if some issues are discussed > before. > I'm interesting in this patch, so tried it here. I use a simple test with No problem at all. Interest is always win ov

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2015-02-05 Thread Shaohua Li
Hi Minchan, Sorry to jump in this thread so later, and if some issues are discussed before. I'm interesting in this patch, so tried it here. I use a simple test with jemalloc. Obviously this can improve performance when there is no memory pressure. Did you try setup with memory pressure? In my t

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2015-02-04 Thread Michal Hocko
On Tue 03-02-15 17:39:24, Michael Kerrisk wrote: [...] > If I'm reading the conversation right, the initially proposed text > was from the BSD man page (which would be okay), but most of the > text above seems to have come straight from the page here: > http://www.lehman.cuny.edu/cgi-bin/man-cgi

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2015-02-03 Thread Minchan Kim
Hello, Michael On Tue, Feb 03, 2015 at 05:39:24PM +0100, Michael Kerrisk (man-pages) wrote: > Hello Minchan (and Michal) > > I did not see this patch until just now when Michael explicitly > mentioned it in another discussion because > (a) it was buried in an LMKL thread that started a topic >

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2015-02-03 Thread Michael Kerrisk (man-pages)
Hello Minchan (and Michal) I did not see this patch until just now when Michael explicitly mentioned it in another discussion because (a) it was buried in an LMKL thread that started a topic that was not about a man-pages patch. (b) linux-man@ was not CCed. When resubmitting this patch, could

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2014-12-05 Thread Michal Hocko
On Fri 05-12-14 16:08:16, Minchan Kim wrote: [...] > From cfa212d4fb307ae772b08cf564cab7e6adb8f4fc Mon Sep 17 00:00:00 2001 > From: Minchan Kim > Date: Mon, 1 Dec 2014 08:53:55 +0900 > Subject: [PATCH] madvise.2: Document MADV_FREE > > Signed-off-by: Minchan Kim Reviewed-by: Michal Hocko Than

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2014-12-04 Thread Minchan Kim
On Wed, Dec 03, 2014 at 11:13:29AM +0100, Michal Hocko wrote: > On Wed 03-12-14 09:00:26, Minchan Kim wrote: > > On Tue, Dec 02, 2014 at 11:01:25AM +0100, Michal Hocko wrote: > > > On Mon 01-12-14 08:56:52, Minchan Kim wrote: > > > [...] > > > > From 2edd6890f92fa4943ce3c452194479458582d88c Mon Sep

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2014-12-03 Thread Michal Hocko
On Wed 03-12-14 09:00:26, Minchan Kim wrote: > On Tue, Dec 02, 2014 at 11:01:25AM +0100, Michal Hocko wrote: > > On Mon 01-12-14 08:56:52, Minchan Kim wrote: > > [...] > > > From 2edd6890f92fa4943ce3c452194479458582d88c Mon Sep 17 00:00:00 2001 > > > From: Minchan Kim > > > Date: Mon, 1 Dec 2014 0

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2014-12-02 Thread Minchan Kim
On Tue, Dec 02, 2014 at 11:01:25AM +0100, Michal Hocko wrote: > On Mon 01-12-14 08:56:52, Minchan Kim wrote: > [...] > > From 2edd6890f92fa4943ce3c452194479458582d88c Mon Sep 17 00:00:00 2001 > > From: Minchan Kim > > Date: Mon, 1 Dec 2014 08:53:55 +0900 > > Subject: [PATCH] madvise.2: Document MA

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2014-12-02 Thread Michal Hocko
On Mon 01-12-14 08:56:52, Minchan Kim wrote: [...] > From 2edd6890f92fa4943ce3c452194479458582d88c Mon Sep 17 00:00:00 2001 > From: Minchan Kim > Date: Mon, 1 Dec 2014 08:53:55 +0900 > Subject: [PATCH] madvise.2: Document MADV_FREE > > Signed-off-by: Minchan Kim > --- > man2/madvise.2 | 13

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2014-11-30 Thread Minchan Kim
Hello Michal, On Thu, Nov 27, 2014 at 03:47:25PM +0100, Michal Hocko wrote: > [Late but I didn't get to this soone - I hope this is still up-to-date > version] > > On Mon 20-10-14 19:11:58, Minchan Kim wrote: > > Linux doesn't have an ability to free pages lazy while other OS > > already have bee

Re: [PATCH v17 1/7] mm: support madvise(MADV_FREE)

2014-11-27 Thread Michal Hocko
[Late but I didn't get to this soone - I hope this is still up-to-date version] On Mon 20-10-14 19:11:58, Minchan Kim wrote: > Linux doesn't have an ability to free pages lazy while other OS > already have been supported that named by madvise(MADV_FREE). > > The gain is clear that kernel can disc

[PATCH v17 1/7] mm: support madvise(MADV_FREE)

2014-10-20 Thread Minchan Kim
Linux doesn't have an ability to free pages lazy while other OS already have been supported that named by madvise(MADV_FREE). The gain is clear that kernel can discard freed pages rather than swapping out or OOM if memory pressure happens. Without memory pressure, freed pages would be reused by u