netdev
[Top] [All Lists]

Re: net/ipv4/route.c GC patch: is this insane?b4Db3tL

To: Robert Olsson <Robert.Olsson@xxxxxxxxxxx>
Subject: Re: net/ipv4/route.c GC patch: is this insane?b4Db3tL
From: Daniel Stone <daniel@xxxxxxxxxxxxx>
Date: Mon, 31 May 2004 22:26:08 +1000
Cc: netdev@xxxxxxxxxxx
In-reply-to: <16571.9006.703294.10134@xxxxxxxxxxxx>
References: <20040530151434.GE24955@xxxxxxxxxxxxx> <16571.9006.703294.10134@xxxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mutt/1.5.6i
On Mon, May 31, 2004 at 02:21:02PM +0200, Robert Olsson wrote:
> Daniel Stone writes:
>  > Hi guys,
>  > Attached is a patch to net/ipv4/route.c (against 2.4.23, sorry) to
>  > combat the 'dst cache full' issue. Essentially, there's a machine that I
>  > have access to (a router) that will report 'dst cache full', and
>  > immediately cease dealing with any IPv4 traffic. I found the attached
>  > patch against 2.0, and forward-ported it to 2.4. As this issue only
>  > crops up randomly every couple of weeks, I can't tell you whether it's
>  > worked or not. All I know is that it hasn't eaten my firstborn so far:
>  > given a quick spin, it seemed to work OK, but whether or not it solves
>  > the problem is a different matter.
> 
>  Hello!
> 
>  If your traffic is sane and you got dst cache overflow you should 
>  probably try to reduce gc_min_interval so your GC can release a 
>  higher numbers of dst entries per second. Flushing active entries
>  means that they are created again. This decreases your throughput
>  maybe just enough not to reach the limit where you reach overflow.

gc_min_interval was zero - should this increase, or what?.

>  > +  atomic_set(&ipv4_dst_ops.entries, 0);
> 
>  These counters should only be handled by the freeing process rt_free etc.

OK.

Thanks for the pointers!
:) d

-- 
Daniel Stone                                              <daniel@xxxxxxxxxxxxx>
"The programs are documented fully by _The Rise and Fall of a Fooish Bar_,
available by the Info system." -- debian/manpage.sgml.ex, dh_make template

Attachment: signature.asc
Description: Digital signature

<Prev in Thread] Current Thread [Next in Thread>