Re: [PATCH tip/core/rcu 03/23] rcu: Move RCU grace-period cleanupinto kthread

From: Paul E. McKenney
Date: Thu Sep 06 2012 - 13:32:56 EST


On Thu, Sep 06, 2012 at 03:34:38PM +0200, Peter Zijlstra wrote:
> On Thu, 2012-08-30 at 11:18 -0700, Paul E. McKenney wrote:
> > static void rcu_report_qs_rsp(struct rcu_state *rsp, unsigned long flags)
> > __releases(rcu_get_root(rsp)->lock)
> > {
> > + raw_spin_unlock_irqrestore(&rcu_get_root(rsp)->lock, flags);
> > + wake_up(&rsp->gp_wq); /* Memory barrier implied by wake_up() path. */
> > }
>
> Could you now also clean up the locking so that the caller releases this
> lock?
>
> I so dislike asymmetric locking like that..

Or I could inline the whole thing at the two callsites...

Thanx, Paul

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/