lvs-devel
|
To: | Simon Horman <horms@xxxxxxxxxxxx> |
---|---|
Subject: | Re: [PATCH ipvs-next] ipvs: Remove rcu_read_unlock();rcu_read_lock(); |
Cc: | Hans Schillstrom <hans@xxxxxxxxxxxxxxx>, Julian Anastasov <ja@xxxxxx>, Pablo Neira Ayuso <pablo@xxxxxxxxxxxxx>, lvs-devel@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxxxxxx, netfilter-devel@xxxxxxxxxxxxxxx, Wensong Zhang <wensong@xxxxxxxxxxxx> |
From: | Eric Dumazet <eric.dumazet@xxxxxxxxx> |
Date: | Thu, 25 Apr 2013 07:04:03 -0700 |
On Thu, 2013-04-25 at 22:36 +0900, Simon Horman wrote: > Ok, leaving it seems reasonable. > Pablo, do you have any objections? I have objections. I would _add_ a cond_resched() there to explicitly do what we want Maybe a macro/inline doing this already exists. static void inline cond_resched_rcu_lock(void) { if (need_resched()) { rcu_read_unlock(); cond_resched(); rcu_read_lock(); } } -- 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 1/1] ip_vs_sip_fill_param() BUG: bad check of return value, Hans Schillstrom |
---|---|
Next by Date: | Re: [PATCH 1/1] ip_vs_sip_fill_param() BUG: bad check of return value, Julian Anastasov |
Previous by Thread: | Re: [PATCH ipvs-next] ipvs: Remove rcu_read_unlock();rcu_read_lock();, Simon Horman |
Next by Thread: | Re: [PATCH ipvs-next] ipvs: Remove rcu_read_unlock();rcu_read_lock();, Julian Anastasov |
Indexes: | [Date] [Thread] [Top] [All Lists] |