![]() |
lvs-devel
|
| To: | Simon Horman <horms@xxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH v2 1/2] sched: Add cond_resched_rcu_lock() helper |
| Cc: | Eric Dumazet <eric.dumazet@xxxxxxxxx>, Ingo Molnar <mingo@xxxxxxxxxx>, Peter Zijlstra <peterz@xxxxxxxxxxxxx>, "Paul E. McKenney" <paulmck@xxxxxxxxxxxxxxxxxx>, lvs-devel@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxxxxxx, netfilter-devel@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, Pablo Neira Ayuso <pablo@xxxxxxxxxxxxx>, Dipankar Sarma <dipankar@xxxxxxxxxx> |
| From: | Julian Anastasov <ja@xxxxxx> |
| Date: | Tue, 30 Apr 2013 10:52:38 +0300 (EEST) |
Hello,
On Tue, 30 Apr 2013, Simon Horman wrote:
> > > +static void inline cond_resched_rcu_lock(void)
> > > +{
> > > + if (need_resched()) {
> >
> > Ops, it should be without above need_resched.
>
> Thanks, to clarify, just this:
>
> static void inline cond_resched_rcu_lock(void)
> {
> rcu_read_unlock();
> #ifndef CONFIG_PREEMPT_RCU
> cond_resched();
> #endif
> rcu_read_lock();
> }
Yes, thanks!
--
Julian Anastasov <ja@xxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH v2 1/2] sched: Add cond_resched_rcu_lock() helper, Simon Horman |
|---|---|
| Previous by Thread: | Re: [PATCH v2 1/2] sched: Add cond_resched_rcu_lock() helper, Simon Horman |
| Indexes: | [Date] [Thread] [Top] [All Lists] |