Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-19 Thread Michal Hocko
On Wed 18-12-13 14:20:15, Johannes Weiner wrote: > On Wed, Dec 18, 2013 at 05:20:50PM +0100, Michal Hocko wrote: [...] > > Currently we have a per-process (cpuset in fact) flag but this will > > change it to all or nothing. Is this really a good step? > > Btw. I do not mind having PF_SPREAD_PAGE

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-19 Thread Mel Gorman
On Wed, Dec 18, 2013 at 02:48:13PM -0500, Johannes Weiner wrote: > > > > > > Sure about the name? > > > > This is a boolean and "mode" implies it might be a bitmask. That said, I > > recognise that my own naming also sucked because complaining about yours > > I can see that mine also sucks. >

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-19 Thread Mel Gorman
On Wed, Dec 18, 2013 at 02:48:13PM -0500, Johannes Weiner wrote: SNIP Sure about the name? This is a boolean and mode implies it might be a bitmask. That said, I recognise that my own naming also sucked because complaining about yours I can see that mine also sucks. Is it

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-19 Thread Michal Hocko
On Wed 18-12-13 14:20:15, Johannes Weiner wrote: On Wed, Dec 18, 2013 at 05:20:50PM +0100, Michal Hocko wrote: [...] Currently we have a per-process (cpuset in fact) flag but this will change it to all or nothing. Is this really a good step? Btw. I do not mind having PF_SPREAD_PAGE enabled

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Johannes Weiner
On Wed, Dec 18, 2013 at 03:00:38PM +, Mel Gorman wrote: > On Wed, Dec 18, 2013 at 01:17:50AM -0500, Johannes Weiner wrote: > > On Tue, Dec 17, 2013 at 03:02:10PM -0500, Johannes Weiner wrote: > > > Hi Mel, > > > > > > On Tue, Dec 17, 2013 at 04:48:18PM +, Mel Gorman wrote: > > > > This

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Johannes Weiner
On Wed, Dec 18, 2013 at 05:20:50PM +0100, Michal Hocko wrote: > On Wed 18-12-13 10:18:46, Johannes Weiner wrote: > > On Wed, Dec 18, 2013 at 03:51:11PM +0100, Michal Hocko wrote: > > > On Tue 17-12-13 15:02:10, Johannes Weiner wrote: > > > [...] > > > > +pagecache_mempolicy_mode: > > > > + > > > >

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Michal Hocko
On Wed 18-12-13 10:18:46, Johannes Weiner wrote: > On Wed, Dec 18, 2013 at 03:51:11PM +0100, Michal Hocko wrote: > > On Tue 17-12-13 15:02:10, Johannes Weiner wrote: > > [...] > > > +pagecache_mempolicy_mode: > > > + > > > +This is available only on NUMA kernels. > > > + > > > +Per default, the

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Mel Gorman
On Wed, Dec 18, 2013 at 03:00:38PM +, Mel Gorman wrote: > > For what it's worth, this is what I've currently kicked off testes for > > git://git.kernel.org/pub/scm/linux/kernel/git/mel/linux-balancenuma.git > mm-pgalloc-interleave-zones-v4r12 > Pushed a dirty tree by accident. Now

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Johannes Weiner
On Wed, Dec 18, 2013 at 03:51:11PM +0100, Michal Hocko wrote: > On Tue 17-12-13 15:02:10, Johannes Weiner wrote: > [...] > > +pagecache_mempolicy_mode: > > + > > +This is available only on NUMA kernels. > > + > > +Per default, the configured memory policy is applicable to anonymous > > +memory,

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Mel Gorman
On Wed, Dec 18, 2013 at 01:17:50AM -0500, Johannes Weiner wrote: > On Tue, Dec 17, 2013 at 03:02:10PM -0500, Johannes Weiner wrote: > > Hi Mel, > > > > On Tue, Dec 17, 2013 at 04:48:18PM +, Mel Gorman wrote: > > > This series is currently untested and is being posted to sync up > > >

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Michal Hocko
On Tue 17-12-13 15:02:10, Johannes Weiner wrote: [...] > +pagecache_mempolicy_mode: > + > +This is available only on NUMA kernels. > + > +Per default, the configured memory policy is applicable to anonymous > +memory, shmem, tmpfs, etc., whereas pagecache is allocated in an > +interleaving fashion

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Johannes Weiner
On Wed, Dec 18, 2013 at 08:47:45AM -0500, Rik van Riel wrote: > On 12/18/2013 01:17 AM, Johannes Weiner wrote: > > > Updated version with your tmpfs __GFP_PAGECACHE parts added and > > documentation, changelog updated as necessary. I remain unconvinced > > that tmpfs pages should be

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Rik van Riel
On 12/18/2013 01:17 AM, Johannes Weiner wrote: > Updated version with your tmpfs __GFP_PAGECACHE parts added and > documentation, changelog updated as necessary. I remain unconvinced > that tmpfs pages should be round-robined, but I agree with you that it > is the conservative change to do for

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Rik van Riel
On 12/18/2013 01:17 AM, Johannes Weiner wrote: Updated version with your tmpfs __GFP_PAGECACHE parts added and documentation, changelog updated as necessary. I remain unconvinced that tmpfs pages should be round-robined, but I agree with you that it is the conservative change to do for 3.12

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Johannes Weiner
On Wed, Dec 18, 2013 at 08:47:45AM -0500, Rik van Riel wrote: On 12/18/2013 01:17 AM, Johannes Weiner wrote: Updated version with your tmpfs __GFP_PAGECACHE parts added and documentation, changelog updated as necessary. I remain unconvinced that tmpfs pages should be round-robined, but I

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Michal Hocko
On Tue 17-12-13 15:02:10, Johannes Weiner wrote: [...] +pagecache_mempolicy_mode: + +This is available only on NUMA kernels. + +Per default, the configured memory policy is applicable to anonymous +memory, shmem, tmpfs, etc., whereas pagecache is allocated in an +interleaving fashion over

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Mel Gorman
On Wed, Dec 18, 2013 at 01:17:50AM -0500, Johannes Weiner wrote: On Tue, Dec 17, 2013 at 03:02:10PM -0500, Johannes Weiner wrote: Hi Mel, On Tue, Dec 17, 2013 at 04:48:18PM +, Mel Gorman wrote: This series is currently untested and is being posted to sync up discussions on

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Johannes Weiner
On Wed, Dec 18, 2013 at 03:51:11PM +0100, Michal Hocko wrote: On Tue 17-12-13 15:02:10, Johannes Weiner wrote: [...] +pagecache_mempolicy_mode: + +This is available only on NUMA kernels. + +Per default, the configured memory policy is applicable to anonymous +memory, shmem, tmpfs,

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Mel Gorman
On Wed, Dec 18, 2013 at 03:00:38PM +, Mel Gorman wrote: For what it's worth, this is what I've currently kicked off testes for git://git.kernel.org/pub/scm/linux/kernel/git/mel/linux-balancenuma.git mm-pgalloc-interleave-zones-v4r12 Pushed a dirty tree by accident. Now

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Michal Hocko
On Wed 18-12-13 10:18:46, Johannes Weiner wrote: On Wed, Dec 18, 2013 at 03:51:11PM +0100, Michal Hocko wrote: On Tue 17-12-13 15:02:10, Johannes Weiner wrote: [...] +pagecache_mempolicy_mode: + +This is available only on NUMA kernels. + +Per default, the configured memory

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Johannes Weiner
On Wed, Dec 18, 2013 at 05:20:50PM +0100, Michal Hocko wrote: On Wed 18-12-13 10:18:46, Johannes Weiner wrote: On Wed, Dec 18, 2013 at 03:51:11PM +0100, Michal Hocko wrote: On Tue 17-12-13 15:02:10, Johannes Weiner wrote: [...] +pagecache_mempolicy_mode: + +This is available

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-18 Thread Johannes Weiner
On Wed, Dec 18, 2013 at 03:00:38PM +, Mel Gorman wrote: On Wed, Dec 18, 2013 at 01:17:50AM -0500, Johannes Weiner wrote: On Tue, Dec 17, 2013 at 03:02:10PM -0500, Johannes Weiner wrote: Hi Mel, On Tue, Dec 17, 2013 at 04:48:18PM +, Mel Gorman wrote: This series is

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-17 Thread Johannes Weiner
On Tue, Dec 17, 2013 at 03:02:10PM -0500, Johannes Weiner wrote: > Hi Mel, > > On Tue, Dec 17, 2013 at 04:48:18PM +, Mel Gorman wrote: > > This series is currently untested and is being posted to sync up discussions > > on the treatment of page cache pages, particularly the sysv part. I have

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-17 Thread Johannes Weiner
Hi Mel, On Tue, Dec 17, 2013 at 04:48:18PM +, Mel Gorman wrote: > This series is currently untested and is being posted to sync up discussions > on the treatment of page cache pages, particularly the sysv part. I have > not thought it through in detail but postings patches is the easiest way

[RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-17 Thread Mel Gorman
This series is currently untested and is being posted to sync up discussions on the treatment of page cache pages, particularly the sysv part. I have not thought it through in detail but postings patches is the easiest way to highlight where I think a problem might be. Changelog since v2 o Drop

[RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-17 Thread Mel Gorman
This series is currently untested and is being posted to sync up discussions on the treatment of page cache pages, particularly the sysv part. I have not thought it through in detail but postings patches is the easiest way to highlight where I think a problem might be. Changelog since v2 o Drop

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-17 Thread Johannes Weiner
Hi Mel, On Tue, Dec 17, 2013 at 04:48:18PM +, Mel Gorman wrote: This series is currently untested and is being posted to sync up discussions on the treatment of page cache pages, particularly the sysv part. I have not thought it through in detail but postings patches is the easiest way to

Re: [RFC PATCH 0/6] Configurable fair allocation zone policy v3

2013-12-17 Thread Johannes Weiner
On Tue, Dec 17, 2013 at 03:02:10PM -0500, Johannes Weiner wrote: Hi Mel, On Tue, Dec 17, 2013 at 04:48:18PM +, Mel Gorman wrote: This series is currently untested and is being posted to sync up discussions on the treatment of page cache pages, particularly the sysv part. I have not