[<prev] [next>] [day] [month] [year] [list]
Message-ID: <00df01c71361$c81f04e0$2a01a8c0@liyewang>
Date: Wed, 29 Nov 2006 10:55:07 +0800
From: "Li Yewang" <lyw@...jing-fnst.com>
To: "Herbert Xu" <herbert@...dor.apana.org.au>
Cc: <netdev@...r.kernel.org>
Subject: Re: [PATCH]Fix BUG of ip_rt_send_redirect()
Herbert Xu herbert@...dor.apana.org.au wrote:
> I think there are two problems here:
>
> 1) The first time we hit the check rate_last is zero. We should simply
> proceed with the redirect rather than treating this as a jiffies value.
>
> 2) When a dst is so old that the jiffies have wrapped around. I'm
> not sure whether this is worth solving as it should be extremely rare
> unless your HZ is sufficiently large and you're on a 32-bit platform.
>
> One solution would be to periodically reset the rate_last fields to
> their original states. Perhaps we can combine that with the GC.
>
> Cheers,
Mr Herbert Xu
Sorry for later reply
I have some questions about your advice:
1) If the first time when we send the redirect packets, but the time_after(jiffies, b)==false.
So the redirect packets can not be sended.
This situation happens when the system of 32-bit boot on less than 5 minutes.
This must be the kernel bug.
2) If the router used frequently, the garbage collect function can not free this dst.
Next time when we want this router to send redirect packets, and if the jiffies wraparound
and time_after(jiffies,b)==false, the redirect packets can not be sended.
So we must consider this situation and solve this problem.
> Visit Openswan at http://www.openswan.org/
> Email: Herbert Xu ~{PmV>HI~} <herbert@...dor.apana.org.au>
> Home Page: http://gondor.apana.org.au/~herbert/
> PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
> -
> 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
>
-
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