[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20081017103948.GA23591@hmsreliant.think-freely.org>
Date: Fri, 17 Oct 2008 06:39:48 -0400
From: Neil Horman <nhorman@...driver.com>
To: Stephen Hemminger <shemminger@...tta.com>
Cc: Eric Dumazet <dada1@...mosbay.com>,
David Miller <davem@...emloft.net>, billfink@...dspring.com,
netdev@...r.kernel.org, kuznet@....inr.ac.ru, pekkas@...core.fi,
jmorris@...ei.org, yoshfuji@...ux-ipv6.org, kaber@...sh.net,
johnpol@....mipt.ru
Subject: Re: [PATCH] net: implement emergency route cache rebulds when
gc_elasticity is exceeded
On Thu, Oct 16, 2008 at 10:06:24PM -0700, Stephen Hemminger wrote:
> O
> > +static inline int rt_caching(struct net *net)
> > +{
> > + return net->ipv4.current_rt_cache_rebuild_count <=
> > + net->ipv4.sysctl_rt_cache_rebuild_count;
> > +}
>
> Why not?
>
I assume that you're asking why we use these values to determine if this
namespace should use the route cache? The answer is this thread. Because
ideally we should never rebuild the route cache unless something is giving the
cache an uneven distribution. The consensus is that if we rebuild too many
times, its an indicator that someone knows enough about our hashing algorithm to
maliginantly force the creation of routes in the cache on the same chain
regardless of how many times we rebuild it. In that case the best thing to do
is skip the use of the cache entirely.
Neil
--
/****************************************************
* Neil Horman <nhorman@...driver.com>
* Software Engineer, Red Hat
****************************************************/
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists