|  | 
lvs-devel
 | 
| To: | Julian Anastasov <ja@xxxxxx> | 
|---|---|
| Subject: | Re: [PATCH ipvs-next] ipvs: Remove rcu_read_unlock();rcu_read_lock(); | 
| Cc: | Simon Horman <horms@xxxxxxxxxxxx>, lvs-devel@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxxxxxx, netfilter-devel@xxxxxxxxxxxxxxx, Wensong Zhang <wensong@xxxxxxxxxxxx> | 
| From: | Pablo Neira Ayuso <pablo@xxxxxxxxxxxxx> | 
| Date: | Fri, 26 Apr 2013 02:59:26 +0200 | 
| Hi Julian, On Thu, Apr 25, 2013 at 11:15:25AM +0300, Julian Anastasov wrote: > > Hello, > > On Thu, 25 Apr 2013, Simon Horman wrote: > > > It is unclear to me that there is any utility in the following: > > > > rcu_read_unlock(); > > rcu_read_lock(); > > I thought it is a good idea for fixed hash table > of IP_VS_TAB_BITS=20. May be if guarded by > > if (!((++idx) & 4095)) > > to reduce its rate to 256 (with idx++ removed from the for loop) ? > > Netfilter has no such logic for nf_conntrack because > it has limit of 16384 rows. We seem to be supporting over that limit via module_param and sysfs: /sys/module/nf_conntrack/parameters/hashsize Regards. -- 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, Simon Horman | 
|---|---|
| Next by Date: | [PATCH 0/2] sched: Add cond_resched_rcu_lock() helper, Simon Horman | 
| 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] |