| 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> |
|---|---|---|
| ||
| Previous by Date: | Re: [3/4] [IPSEC] Add route element to xfrm_dst, David S. Miller |
|---|---|
| Next by Date: | Re: [5/*] [IPSEC] Use dst_mtu in xfrm[46]_output, David S. Miller |
| Previous by Thread: | Re: [3/4] [IPSEC] Add route element to xfrm_dst, David S. Miller |
| Next by Thread: | [9/*] [IPSEC] Check dst validity harder in xfrm_bundle_ok, Herbert Xu |
| Indexes: | [Date] [Thread] [Top] [All Lists] |