* [PATCH] net: Don't drop route cache entry in ipv4_negative_advice unless PTMU expired
@ 2010-03-19 14:41 Guenter Roeck
2010-03-22 3:53 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Guenter Roeck @ 2010-03-19 14:41 UTC (permalink / raw
To: David S. Miller; +Cc: netdev, Al Viro, Denis V. Lunev, Guenter Roeck
TCP sessions over IPv4 can get stuck if routers between endpoints
do not fragment packets but implement PMTU instead.
Setup is as follows
MTU1 MTU2 MTU1
A--------B------C------D
with MTU1 > MTU2. A and D are endpoints, B and C are routers. B and C
implement PMTU and drop packets larger than MTU2 (for example because
DF is set on all packets). TCP sessions are initiated between A and D.
There is packet loss between A and D, causing frequent TCP retransmits.
After the number of retransmits on a TCP session reaches tcp_retries1,
tcp calls dst_negative_advice() prior to each retransmit. This results
in route cache entries for the peer to be deleted in ipv4_negative_advice()
if the Path MTU is set.
If the outstanding data on an affected TCP session is larger than MTU2, packets
sent from the endpoints will be dropped by B or C, and ICMP NEEDFRAG will be
returned. A and D receive NEEDFRAG messages and update PMTU.
Before the next retransmit, tcp will again call dst_negative_advice(), causing
the route cache entry (with correct PMTU) to be deleted. The retransmitted
packet will be larger than MTU2, causing it to be dropped again.
This sequence repeats until the TCP session aborts or is terminated.
Problem is fixed by removing route cache entries in ipv4_negative_advice()
only if the PMTU is expired.
Signed-off-by: Guenter Roeck <guenter.roeck@ericsson.com>
---
net/ipv4/route.c | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/net/ipv4/route.c b/net/ipv4/route.c
index d9b4024..7e1c9e4 100644
--- a/net/ipv4/route.c
+++ b/net/ipv4/route.c
@@ -1512,7 +1512,8 @@ static struct dst_entry *ipv4_negative_advice(struct dst_entry *dst)
ip_rt_put(rt);
ret = NULL;
} else if ((rt->rt_flags & RTCF_REDIRECTED) ||
- rt->u.dst.expires) {
+ (rt->u.dst.expires &&
+ time_after_eq(jiffies, rt->u.dst.expires))) {
unsigned hash = rt_hash(rt->fl.fl4_dst, rt->fl.fl4_src,
rt->fl.oif,
rt_genid(dev_net(dst->dev)));
--
1.6.0.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] net: Don't drop route cache entry in ipv4_negative_advice unless PTMU expired
2010-03-19 14:41 [PATCH] net: Don't drop route cache entry in ipv4_negative_advice unless PTMU expired Guenter Roeck
@ 2010-03-22 3:53 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2010-03-22 3:53 UTC (permalink / raw
To: guenter.roeck; +Cc: netdev, viro, den
From: Guenter Roeck <guenter.roeck@ericsson.com>
Date: Fri, 19 Mar 2010 07:41:13 -0700
> TCP sessions over IPv4 can get stuck if routers between endpoints
> do not fragment packets but implement PMTU instead.
This bug actually only applies to a much more specific case.
It only occurs when the router we end up using is the result of
receiving a redirect to it.
If we use the configured route, and do not get redirected, the problem
never happens.
In any case, your patch is correct, and I'll add some clarification to
the commit message when I check this in.
Thanks a lot!
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-03-22 3:53 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-03-19 14:41 [PATCH] net: Don't drop route cache entry in ipv4_negative_advice unless PTMU expired Guenter Roeck
2010-03-22 3:53 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).