[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20110325.012749.235670347.davem@davemloft.net>
Date: Fri, 25 Mar 2011 01:27:49 -0700 (PDT)
From: David Miller <davem@...emloft.net>
To: steffen.klassert@...unet.com
Cc: netdev@...r.kernel.org
Subject: Re: slow tcp connect when using IPsec
From: Steffen Klassert <steffen.klassert@...unet.com>
Date: Fri, 25 Mar 2011 07:41:16 +0100
> commit 5e2b61f78411be25f0b84f97d5b5d312f184dfd1
> Author: David S. Miller <davem@...emloft.net>
> Date: Fri Mar 4 21:47:09 2011 -0800
>
> ipv4: Remove flowi from struct rtable.
>
> Some time and a lot of trace_printks later I found that we set up
> the flow informations without source _and_ destination address in
> ip_route_newports(). That is because we take the address informations
> from the the rt_key_src and rt_key_dst fields of the rtable here
> and they appear to be empty. If I restore the behaviour before the bisected
> commit by taking the address informations from rt_src and rt_dst the issue
> is gone.
Indeed, it is wrong to use the key values, since they can be
wildcards. Thanks for tracking this down.
> So now I know why it did not behave as expected, but unfortunately I
> still don't know why it magically started to work after 20
> seconds...
After some time, TCP will mark routing path as having trouble, then it
will relookup the route. At this point source and dest will no longer
be wildcarded in the socket, and thus neither will be the resulting
route keys in the relooked-up route.
Look for the dst_negative_advice() paths to see where this happens.
--
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