Re: [PATCH 7/7] mm: page_alloc: Default allow file pages to use remote nodes for fair allocation policy

2013-12-17 Thread Mel Gorman
On Fri, Dec 13, 2013 at 05:15:41PM -0500, Johannes Weiner wrote: > On Fri, Dec 13, 2013 at 07:20:14PM +, Mel Gorman wrote: > > On Fri, Dec 13, 2013 at 12:04:43PM -0500, Johannes Weiner wrote: > > > On Fri, Dec 13, 2013 at 02:10:07PM +, Mel Gorman wrote: > > > > Indications from Johannes tha

Re: [PATCH 7/7] mm: page_alloc: Default allow file pages to use remote nodes for fair allocation policy

2013-12-16 Thread Rik van Riel
On 12/13/2013 09:10 AM, Mel Gorman wrote: > Indications from Johannes that he wanted this. Needs some data and/or > justification why > thrash protection needs it plus docs describing how MPOL_LOCAL is now > different before > it should be considered finished. I do not necessarily agree this patc

Re: [PATCH 7/7] mm: page_alloc: Default allow file pages to use remote nodes for fair allocation policy

2013-12-13 Thread Johannes Weiner
On Fri, Dec 13, 2013 at 07:20:14PM +, Mel Gorman wrote: > On Fri, Dec 13, 2013 at 12:04:43PM -0500, Johannes Weiner wrote: > > On Fri, Dec 13, 2013 at 02:10:07PM +, Mel Gorman wrote: > > > Indications from Johannes that he wanted this. Needs some data and/or > > > justification why > > > t

Re: [PATCH 7/7] mm: page_alloc: Default allow file pages to use remote nodes for fair allocation policy

2013-12-13 Thread Mel Gorman
On Fri, Dec 13, 2013 at 12:04:43PM -0500, Johannes Weiner wrote: > On Fri, Dec 13, 2013 at 02:10:07PM +, Mel Gorman wrote: > > Indications from Johannes that he wanted this. Needs some data and/or > > justification why > > thrash protection needs it plus docs describing how MPOL_LOCAL is now

Re: [PATCH 7/7] mm: page_alloc: Default allow file pages to use remote nodes for fair allocation policy

2013-12-13 Thread Johannes Weiner
On Fri, Dec 13, 2013 at 02:10:07PM +, Mel Gorman wrote: > Indications from Johannes that he wanted this. Needs some data and/or > justification why > thrash protection needs it plus docs describing how MPOL_LOCAL is now > different before > it should be considered finished. I do not necessari

[PATCH 7/7] mm: page_alloc: Default allow file pages to use remote nodes for fair allocation policy

2013-12-13 Thread Mel Gorman
Indications from Johannes that he wanted this. Needs some data and/or justification why thrash protection needs it plus docs describing how MPOL_LOCAL is now different before it should be considered finished. I do not necessarily agree this patch is necessary but it's worth punting it out there