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-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 Fri, Feb 06,

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,

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 at

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-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 Minchan,

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, On Thu, Feb

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

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 issues are discussed

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

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.

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

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-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 it here. I use

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 would

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 be

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

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

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 over

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

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: >

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:

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,

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,

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)

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

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 minc...@kernel.org Date: Mon, 1 Dec 2014 08:53:55 +0900 Subject: [PATCH] madvise.2: Document MADV_FREE Signed-off-by: Minchan Kim minc...@kernel.org

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

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 17 00:00:00

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

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 minc...@kernel.org Date: Mon, 1 Dec

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

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-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 minc...@kernel.org Date: Mon, 1 Dec 2014 08:53:55 +0900 Subject: [PATCH] madvise.2: Document MADV_FREE Signed-off-by: Minchan Kim minc...@kernel.org

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 minc...@kernel.org Date: Mon, 1 Dec 2014 08:53:55 +0900 Subject: [PATCH] madvise.2:

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

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 been

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

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 discard

[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

[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