| To: | Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [3/4] [IPSEC] Add route element to xfrm_dst |
| From: | Patrick McHardy <kaber@xxxxxxxxx> |
| Date: | Tue, 15 Mar 2005 20:05:43 +0100 |
| Cc: | netdev@xxxxxxxxxxx |
| In-reply-to: | <20050314203208.GA15146@xxxxxxxxxxxxxxxxxxx> |
| References: | <20050214221006.GA18415@xxxxxxxxxxxxxxxxxxx> <20050214221200.GA18465@xxxxxxxxxxxxxxxxxxx> <20050214221433.GB18465@xxxxxxxxxxxxxxxxxxx> <42357AF0.4080205@xxxxxxxxx> <20050314203208.GA15146@xxxxxxxxxxxxxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.7.5) Gecko/20050106 Debian/1.7.5-1 |
Herbert Xu wrote: On Mon, Mar 14, 2005 at 12:52:16PM +0100, Patrick McHardy wrote:Since the tunnel dst is not necessarily the last in the bundle anymore, we might miss to initialize some dsts, for example with ipcomp/tunnel + esp/transport. If we have nested tunnels we'll fiddle with entries in the routing cache.Sorry, but I don't get it :) First of all what do you mean by the tunnel dst? If you mean &rt->u.dst then as far as I can see it's still the last child in the bundle. It may also appear in ->route elements earlier on but that does not come into play in this loop. You're right, I must have misread the code somehow. Sorry for the noise. Regards Patrick |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: Constant errors with pktgen, cliff white |
|---|---|
| Next by Date: | Re: RFC: PHY Abstraction Layer II, James Chapman |
| Previous by Thread: | Re: [3/4] [IPSEC] Add route element to xfrm_dst, Herbert Xu |
| Next by Thread: | Re: [4/4] [IPSEC] Store MTU at each xfrm_dst, David S. Miller |
| Indexes: | [Date] [Thread] [Top] [All Lists] |