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] |