Messages in this thread | | | Date | Thu, 3 Oct 2013 22:00:02 +0200 | From | Oleg Nesterov <> | Subject | Re: [PATCH 2/3] rcu: Create rcu_sync infrastructure |
| |
On 10/03, Paul E. McKenney wrote: > > On Thu, Oct 03, 2013 at 09:33:19PM +0200, Oleg Nesterov wrote: > > On 10/03, Oleg Nesterov wrote: > > > > > > Because I believe this needs another patch ;) see below, didn't test > > > it yet. > > > ... > > > struct rcu_sync_ops { > > > void (*sync)(void); > > > void (*call)(struct rcu_head *, void (*)(struct rcu_head *)); > > > +#ifdef CONFIG_PROVE_RCU > > > + bool (*held)(void); > > ^^^^ > > OK, it has to return "int". > > I missed this, but the rest looked good. ;-)
OK thanks ;)
So unless Peter objects I'll write the changelogs (always nontrivial task), test, and send these 2 patches + "add ops->barr() / rcu_sync_wait_for_cb" tomorrow.
Oleg.
| |