| To: | Eric Dumazet <dada1@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [BUG] overflow in net/ipv4/route.c rt_check_expire() |
| From: | "David S. Miller" <davem@xxxxxxxxxxxxx> |
| Date: | Wed, 16 Mar 2005 14:09:15 -0800 |
| Cc: | netdev@xxxxxxxxxxx |
| In-reply-to: | <42380EC6.60100@xxxxxxxxxxxxx> |
| References: | <42370997.6010302@xxxxxxxxxxxxx> <20050315103253.590c8bfc.davem@xxxxxxxxxxxxx> <42380EC6.60100@xxxxxxxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
On Wed, 16 Mar 2005 11:47:34 +0100 Eric Dumazet <dada1@xxxxxxxxxxxxx> wrote: > The rt_check_expire() has a serious problem on machines with large > route caches, and a standard HZ value of 1000. > > With default values, ie ip_rt_gc_interval = 60*HZ = 60000 ; > > the loop count : > > for (t = ip_rt_gc_interval << rt_hash_log; t >= 0; > > > overflows (t is a 31 bit value) as soon rt_hash_log is >= 16 (65536 > slots in route cache hash table) ... > I am experimenting some changes that I will share when ready. Good catch, let us know when you have a patch for review. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [15/*] [INET] Fix IPsec calculation in ip_append_data/ip6_append_data, David S. Miller |
|---|---|
| Next by Date: | Re: [PATCH] reduce sizeof(struct inet_peer) from 128 to 64 bytes on 64bits architectures, David S. Miller |
| Previous by Thread: | Re: [PATCH] reduce sizeof(struct inet_peer) from 128 to 64 bytes on 64bits architectures, David S. Miller |
| Next by Thread: | Re: [BUG] overflow in net/ipv4/route.c rt_check_expire(), Eric Dumazet |
| Indexes: | [Date] [Thread] [Top] [All Lists] |