Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-22 Thread Rusty Russell
Dave Hansen writes: > On 04/08/2014 06:32 PM, Madhavan Srinivasan wrote: >>> > In mm/Kconfig, put >>> > >>> > config FAULT_AROUND_ORDER >>> > int >>> > default 1234 if POWERPC >>> > default 4 >>> > >>> > The way you have it now, every single architecture that

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-22 Thread Rusty Russell
Dave Hansen dave.han...@intel.com writes: On 04/08/2014 06:32 PM, Madhavan Srinivasan wrote: In mm/Kconfig, put config FAULT_AROUND_ORDER int default 1234 if POWERPC default 4 The way you have it now, every single architecture that needs to enable

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-10 Thread Madhavan Srinivasan
On Wednesday 09 April 2014 09:18 PM, Dave Hansen wrote: > On 04/09/2014 01:20 AM, Peter Zijlstra wrote: >> This still misses out on Ben's objection that its impossible to get this >> right at compile time for many kernels, since they can boot and run on >> many different subarchs. > > Completely

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-10 Thread Madhavan Srinivasan
On Wednesday 09 April 2014 09:18 PM, Dave Hansen wrote: On 04/09/2014 01:20 AM, Peter Zijlstra wrote: This still misses out on Ben's objection that its impossible to get this right at compile time for many kernels, since they can boot and run on many different subarchs. Completely agree.

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-09 Thread Dave Hansen
On 04/09/2014 01:20 AM, Peter Zijlstra wrote: > This still misses out on Ben's objection that its impossible to get this > right at compile time for many kernels, since they can boot and run on > many different subarchs. Completely agree. The Kconfig-time stuff should probably just be a knob to

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-09 Thread Dave Hansen
On 04/08/2014 06:32 PM, Madhavan Srinivasan wrote: >> > In mm/Kconfig, put >> > >> >config FAULT_AROUND_ORDER >> >int >> >default 1234 if POWERPC >> >default 4 >> > >> > The way you have it now, every single architecture that needs to enable >> > this has

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-09 Thread Peter Zijlstra
On Wed, Apr 09, 2014 at 07:02:02AM +0530, Madhavan Srinivasan wrote: > On Friday 04 April 2014 09:48 PM, Dave Hansen wrote: > > On 04/03/2014 11:27 PM, Madhavan Srinivasan wrote: > >> This patch creates infrastructure to move the FAULT_AROUND_ORDER > >> to arch/ using Kconfig. This will enable

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-09 Thread Peter Zijlstra
On Wed, Apr 09, 2014 at 07:02:02AM +0530, Madhavan Srinivasan wrote: On Friday 04 April 2014 09:48 PM, Dave Hansen wrote: On 04/03/2014 11:27 PM, Madhavan Srinivasan wrote: This patch creates infrastructure to move the FAULT_AROUND_ORDER to arch/ using Kconfig. This will enable architecture

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-09 Thread Dave Hansen
On 04/08/2014 06:32 PM, Madhavan Srinivasan wrote: In mm/Kconfig, put config FAULT_AROUND_ORDER int default 1234 if POWERPC default 4 The way you have it now, every single architecture that needs to enable this has to go put that in their

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-09 Thread Dave Hansen
On 04/09/2014 01:20 AM, Peter Zijlstra wrote: This still misses out on Ben's objection that its impossible to get this right at compile time for many kernels, since they can boot and run on many different subarchs. Completely agree. The Kconfig-time stuff should probably just be a knob to

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-08 Thread Madhavan Srinivasan
On Friday 04 April 2014 11:20 PM, David Miller wrote: > From: Dave Hansen > Date: Fri, 04 Apr 2014 09:18:43 -0700 > >> On 04/03/2014 11:27 PM, Madhavan Srinivasan wrote: >>> This patch creates infrastructure to move the FAULT_AROUND_ORDER >>> to arch/ using Kconfig. This will enable architecture

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-08 Thread Madhavan Srinivasan
On Friday 04 April 2014 09:48 PM, Dave Hansen wrote: > On 04/03/2014 11:27 PM, Madhavan Srinivasan wrote: >> This patch creates infrastructure to move the FAULT_AROUND_ORDER >> to arch/ using Kconfig. This will enable architecture maintainers >> to decide on suitable FAULT_AROUND_ORDER value based

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-08 Thread Madhavan Srinivasan
On Friday 04 April 2014 06:47 PM, Kirill A. Shutemov wrote: > On Fri, Apr 04, 2014 at 11:57:14AM +0530, Madhavan Srinivasan wrote: >> Kirill A. Shutemov with faultaround patchset introduced >> vm_ops->map_pages() for mapping easy accessible pages around >> fault address in hope to reduce number of

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-08 Thread Madhavan Srinivasan
On Friday 04 April 2014 06:47 PM, Kirill A. Shutemov wrote: On Fri, Apr 04, 2014 at 11:57:14AM +0530, Madhavan Srinivasan wrote: Kirill A. Shutemov with faultaround patchset introduced vm_ops-map_pages() for mapping easy accessible pages around fault address in hope to reduce number of minor

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-08 Thread Madhavan Srinivasan
On Friday 04 April 2014 09:48 PM, Dave Hansen wrote: On 04/03/2014 11:27 PM, Madhavan Srinivasan wrote: This patch creates infrastructure to move the FAULT_AROUND_ORDER to arch/ using Kconfig. This will enable architecture maintainers to decide on suitable FAULT_AROUND_ORDER value based on

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-08 Thread Madhavan Srinivasan
On Friday 04 April 2014 11:20 PM, David Miller wrote: From: Dave Hansen dave.han...@intel.com Date: Fri, 04 Apr 2014 09:18:43 -0700 On 04/03/2014 11:27 PM, Madhavan Srinivasan wrote: This patch creates infrastructure to move the FAULT_AROUND_ORDER to arch/ using Kconfig. This will enable

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-07 Thread Benjamin Herrenschmidt
On Fri, 2014-04-04 at 09:18 -0700, Dave Hansen wrote: > On 04/03/2014 11:27 PM, Madhavan Srinivasan wrote: > > This patch creates infrastructure to move the FAULT_AROUND_ORDER > > to arch/ using Kconfig. This will enable architecture maintainers > > to decide on suitable FAULT_AROUND_ORDER value

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-07 Thread Benjamin Herrenschmidt
On Fri, 2014-04-04 at 09:18 -0700, Dave Hansen wrote: On 04/03/2014 11:27 PM, Madhavan Srinivasan wrote: This patch creates infrastructure to move the FAULT_AROUND_ORDER to arch/ using Kconfig. This will enable architecture maintainers to decide on suitable FAULT_AROUND_ORDER value based on

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-04 Thread David Miller
From: Dave Hansen Date: Fri, 04 Apr 2014 09:18:43 -0700 > On 04/03/2014 11:27 PM, Madhavan Srinivasan wrote: >> This patch creates infrastructure to move the FAULT_AROUND_ORDER >> to arch/ using Kconfig. This will enable architecture maintainers >> to decide on suitable FAULT_AROUND_ORDER value

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-04 Thread Dave Hansen
On 04/03/2014 11:27 PM, Madhavan Srinivasan wrote: > This patch creates infrastructure to move the FAULT_AROUND_ORDER > to arch/ using Kconfig. This will enable architecture maintainers > to decide on suitable FAULT_AROUND_ORDER value based on > performance data for that architecture. Patch also

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-04 Thread Kirill A. Shutemov
On Fri, Apr 04, 2014 at 11:57:14AM +0530, Madhavan Srinivasan wrote: > Kirill A. Shutemov with faultaround patchset introduced > vm_ops->map_pages() for mapping easy accessible pages around > fault address in hope to reduce number of minor page faults. > > This patch creates infrastructure to

[PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-04 Thread Madhavan Srinivasan
Kirill A. Shutemov with faultaround patchset introduced vm_ops->map_pages() for mapping easy accessible pages around fault address in hope to reduce number of minor page faults. This patch creates infrastructure to move the FAULT_AROUND_ORDER to arch/ using Kconfig. This will enable architecture

[PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-04 Thread Madhavan Srinivasan
Kirill A. Shutemov with faultaround patchset introduced vm_ops-map_pages() for mapping easy accessible pages around fault address in hope to reduce number of minor page faults. This patch creates infrastructure to move the FAULT_AROUND_ORDER to arch/ using Kconfig. This will enable architecture

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-04 Thread Kirill A. Shutemov
On Fri, Apr 04, 2014 at 11:57:14AM +0530, Madhavan Srinivasan wrote: Kirill A. Shutemov with faultaround patchset introduced vm_ops-map_pages() for mapping easy accessible pages around fault address in hope to reduce number of minor page faults. This patch creates infrastructure to move the

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-04 Thread Dave Hansen
On 04/03/2014 11:27 PM, Madhavan Srinivasan wrote: This patch creates infrastructure to move the FAULT_AROUND_ORDER to arch/ using Kconfig. This will enable architecture maintainers to decide on suitable FAULT_AROUND_ORDER value based on performance data for that architecture. Patch also adds

Re: [PATCH V2 1/2] mm: move FAULT_AROUND_ORDER to arch/

2014-04-04 Thread David Miller
From: Dave Hansen dave.han...@intel.com Date: Fri, 04 Apr 2014 09:18:43 -0700 On 04/03/2014 11:27 PM, Madhavan Srinivasan wrote: This patch creates infrastructure to move the FAULT_AROUND_ORDER to arch/ using Kconfig. This will enable architecture maintainers to decide on suitable