netdev
[Top] [All Lists]

Re: [4/4] [IPSEC] Store MTU at each xfrm_dst

To: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [4/4] [IPSEC] Store MTU at each xfrm_dst
From: "David S. Miller" <davem@xxxxxxxxxxxxx>
Date: Sun, 6 Mar 2005 21:32:14 -0800
Cc: kuznet@xxxxxxxxxxxxx, jmorris@xxxxxxxxxx, yoshfuji@xxxxxxxxxxxxxx, netdev@xxxxxxxxxxx
In-reply-to: <20050214221607.GC18465@xxxxxxxxxxxxxxxxxxx>
References: <20050214221006.GA18415@xxxxxxxxxxxxxxxxxxx> <20050214221200.GA18465@xxxxxxxxxxxxxxxxxxx> <20050214221433.GB18465@xxxxxxxxxxxxxxxxxxx> <20050214221607.GC18465@xxxxxxxxxxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
On Tue, 15 Feb 2005 09:16:07 +1100
Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> wrote:

> Finally this is the patch that sets the MTU values of each xfrm_dst
> and keeps it up-to-date.
>   
> Signed-off-by: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
> 
> To recap, at this point we've obtained accurate MTU values at each
> xfrm_dst.  The next step would be to start using it as
> dst_pmtu(xfrm_dst).

Applied, but with a bug fix:

+               mtu = dst_pmtu(xdst->route);
+               if (xdst->child_mtu_cached != mtu) {
+                       last = xdst;
+                       xdst->route_mtu_cached = mtu;
+               }

You obviously meant "route_mtu_cached" in the test,
not child_mtu_cached.

<Prev in Thread] Current Thread [Next in Thread>