| Date | Wed, 9 Oct 2013 21:52:50 +0200 | From | Peter Zijlstra <> | Subject | Re: [PATCH 2/6] rcu: Create rcu_sync infrastructure |
| |
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(); > > } > > > > static inline void rcu_sync_enter(struct rcu_sync_struct *xxx) > > { > > synchronize_sched(); > > atomic_dec(&xxx->counter); > > } > > That second rcu_sync_enter should really be rcu_sync_exit. > > Is the "xxx" there to test spamassassin configurations? :)
Uhh, no, its a search-n-replace fail, Oleg's original email on the subject called the entire primitive xxx. I replaced (some) of them with rcu_sync.
|