Re: [PATCH v2] srcu: Isolate srcu sections using CONFIG_SRCU

2015-01-04 Thread Paul E. McKenney
On Sun, Jan 04, 2015 at 08:35:52PM +1100, Michael Ellerman wrote: > On Tue, 2014-12-30 at 13:54 -0500, Pranith Kumar wrote: > > On Tue, Dec 30, 2014 at 1:50 PM, Peter Zijlstra > > wrote: > > > On Tue, Dec 30, 2014 at 12:46:22AM -0500, Pranith Kumar wrote: > > >> Isolate the SRCU functions and

Re: [PATCH v2] srcu: Isolate srcu sections using CONFIG_SRCU

2015-01-04 Thread Michael Ellerman
On Tue, 2014-12-30 at 13:54 -0500, Pranith Kumar wrote: > On Tue, Dec 30, 2014 at 1:50 PM, Peter Zijlstra wrote: > > On Tue, Dec 30, 2014 at 12:46:22AM -0500, Pranith Kumar wrote: > >> Isolate the SRCU functions and data structures within CONFIG_SRCU so that > >> there > >> is a compile time

Re: [PATCH v2] srcu: Isolate srcu sections using CONFIG_SRCU

2015-01-04 Thread Michael Ellerman
On Tue, 2014-12-30 at 13:54 -0500, Pranith Kumar wrote: On Tue, Dec 30, 2014 at 1:50 PM, Peter Zijlstra pet...@infradead.org wrote: On Tue, Dec 30, 2014 at 12:46:22AM -0500, Pranith Kumar wrote: Isolate the SRCU functions and data structures within CONFIG_SRCU so that there is a compile

Re: [PATCH v2] srcu: Isolate srcu sections using CONFIG_SRCU

2015-01-04 Thread Paul E. McKenney
On Sun, Jan 04, 2015 at 08:35:52PM +1100, Michael Ellerman wrote: On Tue, 2014-12-30 at 13:54 -0500, Pranith Kumar wrote: On Tue, Dec 30, 2014 at 1:50 PM, Peter Zijlstra pet...@infradead.org wrote: On Tue, Dec 30, 2014 at 12:46:22AM -0500, Pranith Kumar wrote: Isolate the SRCU

Re: [PATCH v2] srcu: Isolate srcu sections using CONFIG_SRCU

2014-12-31 Thread Paul E. McKenney
On Tue, Dec 30, 2014 at 08:07:40PM +0100, Peter Zijlstra wrote: > On Tue, Dec 30, 2014 at 01:54:07PM -0500, Pranith Kumar wrote: > > On Tue, Dec 30, 2014 at 1:50 PM, Peter Zijlstra > > wrote: > > > On Tue, Dec 30, 2014 at 12:46:22AM -0500, Pranith Kumar wrote: > > >> Isolate the SRCU functions

Re: [PATCH v2] srcu: Isolate srcu sections using CONFIG_SRCU

2014-12-31 Thread Paul E. McKenney
On Tue, Dec 30, 2014 at 08:07:40PM +0100, Peter Zijlstra wrote: On Tue, Dec 30, 2014 at 01:54:07PM -0500, Pranith Kumar wrote: On Tue, Dec 30, 2014 at 1:50 PM, Peter Zijlstra pet...@infradead.org wrote: On Tue, Dec 30, 2014 at 12:46:22AM -0500, Pranith Kumar wrote: Isolate the SRCU

Re: [PATCH v2] srcu: Isolate srcu sections using CONFIG_SRCU

2014-12-30 Thread Peter Zijlstra
On Tue, Dec 30, 2014 at 01:54:07PM -0500, Pranith Kumar wrote: > On Tue, Dec 30, 2014 at 1:50 PM, Peter Zijlstra wrote: > > On Tue, Dec 30, 2014 at 12:46:22AM -0500, Pranith Kumar wrote: > >> Isolate the SRCU functions and data structures within CONFIG_SRCU so that > >> there > >> is a compile

Re: [PATCH v2] srcu: Isolate srcu sections using CONFIG_SRCU

2014-12-30 Thread Pranith Kumar
On Tue, Dec 30, 2014 at 1:50 PM, Peter Zijlstra wrote: > On Tue, Dec 30, 2014 at 12:46:22AM -0500, Pranith Kumar wrote: >> Isolate the SRCU functions and data structures within CONFIG_SRCU so that >> there >> is a compile time failure if srcu is used when not enabled. This was decided >> to >>

Re: [PATCH v2] srcu: Isolate srcu sections using CONFIG_SRCU

2014-12-30 Thread Peter Zijlstra
On Tue, Dec 30, 2014 at 12:46:22AM -0500, Pranith Kumar wrote: > Isolate the SRCU functions and data structures within CONFIG_SRCU so that > there > is a compile time failure if srcu is used when not enabled. This was decided > to > be better than waiting until link time for a failure to occur.

Re: [PATCH v2] srcu: Isolate srcu sections using CONFIG_SRCU

2014-12-30 Thread Peter Zijlstra
On Tue, Dec 30, 2014 at 12:46:22AM -0500, Pranith Kumar wrote: Isolate the SRCU functions and data structures within CONFIG_SRCU so that there is a compile time failure if srcu is used when not enabled. This was decided to be better than waiting until link time for a failure to occur. Why?

Re: [PATCH v2] srcu: Isolate srcu sections using CONFIG_SRCU

2014-12-30 Thread Pranith Kumar
On Tue, Dec 30, 2014 at 1:50 PM, Peter Zijlstra pet...@infradead.org wrote: On Tue, Dec 30, 2014 at 12:46:22AM -0500, Pranith Kumar wrote: Isolate the SRCU functions and data structures within CONFIG_SRCU so that there is a compile time failure if srcu is used when not enabled. This was

Re: [PATCH v2] srcu: Isolate srcu sections using CONFIG_SRCU

2014-12-30 Thread Peter Zijlstra
On Tue, Dec 30, 2014 at 01:54:07PM -0500, Pranith Kumar wrote: On Tue, Dec 30, 2014 at 1:50 PM, Peter Zijlstra pet...@infradead.org wrote: On Tue, Dec 30, 2014 at 12:46:22AM -0500, Pranith Kumar wrote: Isolate the SRCU functions and data structures within CONFIG_SRCU so that there is a

[PATCH v2] srcu: Isolate srcu sections using CONFIG_SRCU

2014-12-29 Thread Pranith Kumar
Isolate the SRCU functions and data structures within CONFIG_SRCU so that there is a compile time failure if srcu is used when not enabled. This was decided to be better than waiting until link time for a failure to occur. There are places which include kvm headers and utilize kvm data structures

[PATCH v2] srcu: Isolate srcu sections using CONFIG_SRCU

2014-12-29 Thread Pranith Kumar
Isolate the SRCU functions and data structures within CONFIG_SRCU so that there is a compile time failure if srcu is used when not enabled. This was decided to be better than waiting until link time for a failure to occur. There are places which include kvm headers and utilize kvm data structures