Re: [PATCH 2/6] rcu: Create rcu_sync infrastructure

2013-10-17 Thread Srikar Dronamraju
* Peter Zijlstra [2013-10-08 12:25:07]: > From: Oleg Nesterov > > It is functionally equivalent to > > struct rcu_sync_struct { > atomic_t counter; > }; > > static inline bool rcu_sync_is_idle(struct rcu_sync_struct *xxx) > { > r

Re: [PATCH 2/6] rcu: Create rcu_sync infrastructure

2013-10-16 Thread Lai Jiangshan
On 10/08/2013 06:25 PM, Peter Zijlstra wrote: > From: Oleg Nesterov > > It is functionally equivalent to > > struct rcu_sync_struct { > atomic_t counter; > }; > > static inline bool rcu_sync_is_idle(struct rcu_sync_struct *xxx) > { >

Re: [PATCH 2/6] rcu: Create rcu_sync infrastructure

2013-10-09 Thread Peter Zijlstra
On Tue, Oct 08, 2013 at 02:40:45PM -0600, Jonathan Corbet wrote: > OK, so this is a real nit, but...in the changelog: > > > static inline void rcu_sync_enter(struct rcu_sync_struct *xxx) > > { > > atomic_inc(&xxx->counter); > > synchronize_sched(); >

Re: [PATCH 2/6] rcu: Create rcu_sync infrastructure

2013-10-08 Thread Jonathan Corbet
OK, so this is a real nit, but...in the changelog: > static inline void rcu_sync_enter(struct rcu_sync_struct *xxx) > { > atomic_inc(&xxx->counter); > synchronize_sched(); > } > > static inline void rcu_sync_enter(struct rcu_sync_str

[PATCH 2/6] rcu: Create rcu_sync infrastructure

2013-10-08 Thread Peter Zijlstra
From: Oleg Nesterov It is functionally equivalent to struct rcu_sync_struct { atomic_t counter; }; static inline bool rcu_sync_is_idle(struct rcu_sync_struct *xxx) { return atomic_read(&xxx->counter) == 0; } static